ppy / osu

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

Selected Mods doesn't work on leaderboards #19148

Closed respektive closed 2 years ago

respektive commented 2 years ago

Type

Game behaviour

Bug description

From the logs:

2022-07-16 12:38:10 [verbose]: Performing request osu.Game.Online.API.Requests.GetScoresRequest
2022-07-16 12:38:10 [verbose]: Request to https://osu.ppy.sh/api/v2/beatmaps/2457055/solo-scores?type=global&mode=osu&mods[]=NM failed with System.Net.WebException: InternalServerError.
2022-07-16 12:38:10 [verbose]: Response was: {"error":null}
2022-07-16 12:38:10 [verbose]: Failing request osu.Game.Online.API.Requests.GetScoresRequest (osu.Game.Online.API.APIException: Exception of type 'osu.Game.Online.API.APIException' was thrown.
2022-07-16 12:38:10 [verbose]: ---> System.Net.WebException: InternalServerError
2022-07-16 12:38:10 [verbose]: --- End of inner exception stack trace ---)
2022-07-16 12:38:11 [verbose]: Performing request osu.Game.Online.API.Requests.GetScoresRequest
2022-07-16 12:38:11 [verbose]: Request to https://osu.ppy.sh/api/v2/beatmaps/2457055/solo-scores?type=global&mode=osu&mods[]=NM failed with System.Net.WebException: InternalServerError.
2022-07-16 12:38:11 [verbose]: Response was: {"error":null}
2022-07-16 12:38:11 [verbose]: Failing request osu.Game.Online.API.Requests.GetScoresRequest (osu.Game.Online.API.APIException: Exception of type 'osu.Game.Online.API.APIException' was thrown.
2022-07-16 12:38:11 [verbose]: ---> System.Net.WebException: InternalServerError
2022-07-16 12:38:11 [verbose]: --- End of inner exception stack trace ---)

Screenshots or videos

osu_2022-07-16_14-41-05

Version

2022.716.1-lazer

Logs

database.log network.log performance.log runtime.log

frenzibyte commented 2 years ago

This is already fixed in https://github.com/ppy/osu-web/pull/9110/commits/630730b3db0038850b4c57f913ff0087259f8c29, but seems it's still failing due to the changes not deployed to the score-es-min branch (that branch only has the first two commits of the pull request).

peppy commented 2 years ago

Good catch, fix has been deployed and looks to be working now.