-
-
Notifications
You must be signed in to change notification settings - Fork 130
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
Avoid Error 429 Too Many Requests when loading large inventory #265
Comments
If you're getting rate-limited, it's because you're requesting too many inventories from the same IP. Requesting multiple pages of the same inventory would not cause a rate-limit (except perhaps hugely massive inventories), since obviously nobody would be able to load the inventory from their browser. |
I load only 1 inventory and on ~15 page hit 429 error |
Please try getUserInventoryContents instead. |
few days ago Steam still uses |
Parts of Steam break all the time. getUserInventoryContents is more stable than getUserInventory since it's not rate-limiting you when you try to load a large inventory. If the GC goes down, then naturally getUserInventoryContents isn't going to work when it tries to contact that GC. The old endpoint is likely just serving up its stale cache when it can't contact the GC, which means it's giving you outdated information without any indication that it's outdated. If you can tell me exactly what delay prevents you from getting rate-limited, and if it's a reasonable length of time, then I'll consider adding it. I'm not going to spend any of my personal time updating a deprecated, unstable endpoint though. |
Steam changed once again rate-limits, see existing PR on node-steam-tradeoffer-manager |
SteamCommunity.prototype.getUserInventory:
node-steamcommunity/components/users.js
Line 384 in 2bbdbf3
maybe it is worth adding a delay? I don't know the exact rate limit to avoid 429
The text was updated successfully, but these errors were encountered: