Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework auctionhouse Update causes people to be kicked if buy to many items at once #21140

Open
TheSCREWEDSoftware opened this issue Jan 11, 2025 · 3 comments
Labels
Confirmed CORE Related to the core

Comments

@TheSCREWEDSoftware
Copy link
Contributor

TheSCREWEDSoftware commented Jan 11, 2025

Issue linked from CC: chromiecraft/chromiecraft#7657

Triage Notes: chromiecraft/chromiecraft#7657 (comment)

  • I tested this with 2 different accounts.
  • Made sure the "buyer" account had auctionator
  • The "seller" account would have 40 linen clothes (.additem 2589 40) and would make 40 stacks of 1 auctions with 10 copper for bid and 20 bid for buyout (you will need to .modify money 100g)
  • With the buyer try buying 11 linen clothes, you will be kicked but the items will be in your mail still (no error on the the worldserver)

Original Post Below:


What client do you play on?

enUS

Faction

Alliance

Content Phase:

Generic

Current Behaviour

Buying 10 or more items at the same time from AH using Auctionator disconnects the player. Items are succesfully bought and can be found in mailbox after relog.

This problem started happening on Chromiecraft within the last month or so. For some reason server kicks the player. I did the same tests on other 3.3.5a servers with the same addon and there's no such issue.

Expected Blizzlike Behaviour

Server shouldn't kick the player.

Source

No source required.

Steps to reproduce the problem

  1. Install Auctionator addon (from Felbite or somewhere else)
  2. Login to chromiecraft live server
  3. On "Buy" tab, search for an item of which you can buy 10 or more (eg. Light Armor Kit, any potion etc.). It has to be 10 x 1 (10 stacks of 1) and not 1 x 10 (1 stack of 10) (Screenshot: https://imgur.com/a/CGKe6Py)
  4. Enter the amount to buy (10+) and click buy.
  5. You are disconnected.

Extra Notes

There are different working versions of Auctionator but it doesn't matter which one you are using for this test. I will put 2 links below for easy downloading:
1- https://forum.warmane.com/showthread.php?t=412904&p=3030654&viewfull=1#post3030654
2- https://felbite.com/addon/4868-auctionator/

AC rev. hash/commit

chromiecraft@5f020de

Operating system

Ubuntu 22.04

Modules

Customizations

None

Server

ChromieCraft

@TheSCREWEDSoftware TheSCREWEDSoftware added Confirmed CORE Related to the core labels Jan 11, 2025
@TheSCREWEDSoftware
Copy link
Contributor Author

CC: @Takenbacon

@TheSCREWEDSoftware TheSCREWEDSoftware changed the title Auction Update causes people to be kicked if buy to many items at once Rework auctionhouse Update causes people to be kicked if buy to many items at once Jan 11, 2025
@Takenbacon
Copy link
Contributor

AH Query packets were added to the throttler (here: https://github.com/azerothcore/azerothcore-wotlk/blob/master/src/server/game/Server/WorldSession.cpp#L1551-L1553). I'm guessing a buy also seems to send a query request. We can just increase the limits for now, it was noted in the AH PR that the current packet throttler implementation is not ideal for this use case to begin with but that'll need a good bit more work to properly handle.

@skelUA
Copy link
Contributor

skelUA commented Jan 19, 2025

also not woring searching usable items
test on druid
with filter
Image
witout filter

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Confirmed CORE Related to the core
Projects
None yet
Development

No branches or pull requests

3 participants