Note: there is a bug atm where sometimes the /players route includes duplicates. Not sure why but FYI.
This PR does a few things:
Changes some API methods to use my map-monitor API over nadeo's (The answer to 'why?' is at the end)
Adds support for up to 1000 club members
Warns the user when the club is over 1k members (so only the first 1k will be tracked)
Fixes some deprecation warnings
Simplifies some code
Some end-of-line spaces removed automatically
Okay, so why should you change the API endpoint?
Basically plugins are causing Nadeo enough load to be a problem, notably COTD HUD and COTD Stats have been asked to reduce their load (in that order). This is somewhat natural considering COTD HUD made an order of magnitude more requests than COTD Stats.
[6:36 PM]magnetik: @XertroV I think your plugin COTD hub is spamming like crazy during COTD
[6:37 PM]magnetik: around 1000 req/s feels like a little too much
[7:15 PM]magnetik: Hey COTDStats plugin is spamming our API during COTD way too much
COTD Stats is transitioning to my endpoint atm. I've just improve it's performance too.
It seems reasonable that, since COTD Stats has been asked to reduce load, COTD Live Qualifying might be asked to soon, too.
Using the map-monitor API will immediately help with reducing load.
The APIs this PR calls are new, and it's hard for me to test them since main COTD is at 3am for me. So I suggest you test this yourself before merging. Hopefully it works okay.
Note: there is a bug atm where sometimes the /players route includes duplicates. Not sure why but FYI.
This PR does a few things:
Okay, so why should you change the API endpoint?
Basically plugins are causing Nadeo enough load to be a problem, notably COTD HUD and COTD Stats have been asked to reduce their load (in that order). This is somewhat natural considering COTD HUD made an order of magnitude more requests than COTD Stats.
COTD Stats is transitioning to my endpoint atm. I've just improve it's performance too.
It seems reasonable that, since COTD Stats has been asked to reduce load, COTD Live Qualifying might be asked to soon, too. Using the map-monitor API will immediately help with reducing load.
The APIs this PR calls are new, and it's hard for me to test them since main COTD is at 3am for me. So I suggest you test this yourself before merging. Hopefully it works okay.