Bug Item owner receives my IP in his history

[madness85]

For some reason on my test site when i change an item from Unique item on/off the item owner has my IP in his history. I've just tested again, this time i only changed the item price and it added my IP. This has also happened when purchasing the item.

This is a fresh xenforo install and there are no other addons installed.

 

[madness85]

@Fillip H. My test server expires today. If you could just confirm before it does i would really appreciate it.

 

[DragonByte Technologies]

Thank you for reporting this issue, it has now been resolved. We are aiming to include any changes that have been made in a future release (6.4.6).

Change log:

Fix: Creating or editing an item will now associate the IP address log with the visitor, not the assigned owner

 

[madness85]

@Fillip H. My live site is using version DragonByte Shop 6.2.3 and users that successfully steal from other users its assigning them the users IP.

I'm not ready for xenforo 2.2 just yet could you please provide a hotfix for this version?

 

[madness85]

@Fillip H.

DragonByte Shop 6.4.5 When creating an item in the shop its assigning my IP to the item owner & can you check purchasing items.

To replicate item purchase IP issue you need to create a new category and item in that category.

 

[madness85]

@Fillip H. Ive been a customer for many years now and I'm very surprised not to get a reply. Is everything OK?

 

[Fillip H.]

Sorry, I've been busy with other things and haven't had a chance to look into this further until now.

I've made a change to v6.2.3 that disables IP logging by default for all purchases. Let me know if that resolves it.

 

[madness85]

Sorry, I've been busy with other things and haven't had a chance to look into this further until now.

I've made a change to v6.2.3 that disables IP logging by default for all purchases. Let me know if that resolves it.

NP and thanks for getting back to me

It was upon a successful steal it was adding the users IP to v6.2.3, its still doing so.

6.4.5 has problems in a few places, creating an item in the shop and purchasing items.

 

[Fillip H.]

I've made an additional change to v6.2.3.

 

[madness85]

I've made an additional change to v6.2.3.

Steal seems to be fixed.

Sadly, I've just created a new item and the item owner got my IP

 

[Fillip H.]

I've applied another hotfix to v6.2.3.

 

[madness85]

Seems all my steal transaction logs has the user and recipient as the same user.

User should be the person they stole from and recipient the user getting the credits.

User
test2
Recipient
test2
Date
Thursday at 10:27 PM
Action
Steal Attempt (Successful)
Transaction info
Lost
4,066 Credits

 

[madness85]

I've applied another hotfix to v6.2.3.

Still does it.

Steps to follow:

Create a new user e.g. test
Create a new category
Create new custom item and place user test as the owner.

Test will now have your IP if you check.

 

[madness85]

Bump.

 

[Fillip H.]

I will post back when I have more information or if I need additional info

 

[Fillip H.]

Update: The issue should now be resolved, I disabled IP logging from the AdminCP. Logging is still in effect in the front-end, since the front-end does not override the "visitor" object with the item owner.

 

[madness85]

Update: The issue should now be resolved, I disabled IP logging from the AdminCP. Logging is still in effect in the front-end, since the front-end does not override the "visitor" object with the item owner.

It seems to be fine now. I do have a question about the log, this is one successful steal attempt.




Should it be listed twice like that for one successful attempt? I'm sure it was just one, user test, recipient admin.

 

[Fillip H.]

Yes, because one is for the person who lost the credits and the other is for the one who gained them. This is intended.

 

[madness85]

Yes, because one is for the person who lost the credits and the other is for the one who gained them. This is intended.

OK thanks for all your work. Will the latest release be coded the same? . I can't stress enough how much of a mess it made mixing users IPs

 

[Fillip H.]

I'll release an update to the XF2.2 version once I'm confident there are no outstanding issues.