ppy / osu

rhythm is just a *click* away!
https://osu.ppy.sh
MIT License
14.75k stars 2.18k forks source link

Derive API response version from game version #28349

Closed bdach closed 2 months ago

bdach commented 2 months ago

(Or local date, in the case of non-deployed builds).

Came up when I was looking at https://github.com/ppy/osu-web/pull/11240 and found that we were still hardcoding this.

Thankfully, this should not cause issues, since there don't seem to be any (documented or [undocumented](https://github.com/search?q=repo%3Appy%2Fosu-web%20api_version()&type=code)) API response version checks for versions newer than 20220705 in osu-web master.

For clarity and possible debugging needs, the API response version is also logged.