Playing deleted beatmap throws `APIException` to user about osu HOT 8 OPEN

I think this probably has to start with osu!web returning an actual error message for when the beatmap isn't found. @ppy/team-web

from osu.

Status code 404 is returned when the beatmap isn't found...

from osu.

Doesn't it seem weird that we have to handle 404s in a very specific way for this one scenario? What if a legitimate 404 actually occurs, like the endpoint becomes inaccessible (for any number of reasons), and we miss it because in that very specific scenario we aren't displaying an error message here?

So there's the question of how should we handle this particular case. In the past we've silently ignored this specific case because it would be too verbose to tell the player that the beatmap doesn't exist online. Curious what @peppy thinks of this.

from osu.

What if a legitimate 404 actually occurs, like the endpoint becomes inaccessible (for any number of reasons), and we miss it because in that very specific scenario we aren't displaying an error message here?

I think this is a reach, 404 is a legitimate response when a resource isn't found in REST. I'm not sure we should be worried about "false positives" that could technically happen if nginx eats dirt or something and the entirety of the endpoint is not there anymore.

from osu.

well, osu-web can return a generic error message ("specified beatmap difficulty (123) couldn't be found"). Currently, most/all requests on missing object return json with null error field.

from osu.

if nginx eats dirt or something

What about if a change in multiplayer happens that results in us not passing the playlist or room to the API, meaning the playlist or room isn't found? My point is, a 404 is a generic catch all term for any resource. It's not specific to a beatmap not being found, and imo we shouldn't treat it that way.

Or what, we add a specific case for multiplayer where it actually does show NotFound because it's actually an error, but we're expecting that to not be an error in solo?

from osu.

It's not specific to a beatmap not being found, and imo we shouldn't treat it that way.

Ok yeah that's fair.

from osu.

I'll add the message I mentioned above but that said, I'm not sure if client should show the generic error message in this case either...

from osu.