TrackerNetwork / DestinyStatus

Destiny Status code base
https://destinystatus.com
MIT License
100 stars 29 forks source link

Cache Name -> Id lookup #17

Closed bladefist closed 8 years ago

iBotPeaches commented 8 years ago

Every profile hit, we send 2 requests to Bungie.

destiny/stats/account/1/4611686018430553461/?lc=en&groups%5B0%5D=General (545.71ms)
destiny/1/account/4611686018430553461/summary/?lc=en (558.4ms)

They are relatively clunky and don't need to be re-used. The connection between platform/gamertag/membershipId can be cached.

We can check cache pool, if hit. Return membershipId and whatever else it needs.

If not, hit api. No harm no foul. Repeat visitors will have n-2 requests.

iBotPeaches commented 8 years ago

speed

The merge of dev into master fixes this. Properly applies cache to search/lookup.