ff14-advanced-market-search / saddlebag-with-pockets

This is the Frontend for the FF14 Marketplace

Home Page:https://saddlebagexchange.com/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Reduce the calls for `${address}/api/wow/itemnames`

cohenaj194 opened this issue · comments

After doing some DDOS testing I found an issue is that we have a database connection limit of 256 at a time. This is a big issue with the wow item name api.

https://github.com/ff14-advanced-market-search/saddlebag-with-pockets/blob/master/app/requests/WoWGetItems.ts

Normally this is not a problem at all its a very small and fast api request. The problem is this seems to trigger every time someone visits the website. So thousands of requests from a ton of new users or a DDOS attack will trigger this to the point that it will clog up the database and make us hit the DB connection limit.

Maybe we should build a static file list like we do for FFXIV or something. Now that we have been collecting the wow names for a while it is very rare a new one appears outside of patch times.

another method is having the api provide a static file instead of the database call, which is less work on the fe