Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Idea: Show random skin instead of steve when rate limited #146

Open
connor4312 opened this issue Jan 6, 2015 · 6 comments
Open

Idea: Show random skin instead of steve when rate limited #146

connor4312 opened this issue Jan 6, 2015 · 6 comments

Comments

@connor4312
Copy link
Member

Sometimes showing a "steve skin" is unavoidable due to rate limiting and such... What about, rather than showing a steve, show a random skin stored in the cache? This would make temporary unavailability less noticeable and provide the aesthetic functionality which minotar is often used for.

@LukeHandle
Copy link
Member

A pretty interesting idea. I think it would probably draw attention away from the issue - which would be good.

Slight potential for an inappropriate skin popping up which might be bad...

We would need to adjust caching headers to prevent upstream Varnish/Nginx/Browser keeping them too long - we should probably already do this when we return chars :)

@LukeHandle LukeHandle changed the title Idea... Idea: Show random skin instead of steve when rate limited Jan 6, 2015
@connor4312
Copy link
Member Author

mhm. What might work is to make some hash (Adler is fast to compute) of the username and have 100 built-in usernames. Map the first/last two digits in the adler hash of the username to the built-in avatars. So that way each person would have a "consistent" picture, and we could serve even when Minecraft's texture servers are completely unavailable (due to outages, for example).

@gumptionthomas
Copy link

I am not a fan of this idea. I'm already anticipating the support calls I'll get from confused users when they get someone else's random skin. Our users have already been conditioned that getting a steve is a temporary condition.

@lenovouser
Copy link

You could provide a config option to enable or disable this feature. Because I personally think this is a good idea.

@Foxtrek64
Copy link

Seeing this is still open, I thought I'd throw in my two cents. What if instead of picking a random skin across all of everyone, we instead grabbed Mojang staff skins (Notch, Jeb_, etc.) and displayed a random from this pool?

This would still give us the randomization we're looking for while also cutting out the issue of random inappropriate skins. Also, the skins are immediately recognizable in most cases, so I think it would be fairly easily to document the fact that "if you get a Mojang staff-member skin, there was probably a temporary issue with the service." This alone could lead to fewer heat tickets, which @gumptionthomas is rightly concerned for. Plus, with the small pool size, it wouldn't be too hard to include these skins (along with Steve/Alex) in a directory on-server, giving us instant access to them.

@lenovouser brings up a good point about having this toggleable - use Steve/Alex or use random skins from the pool. We may also be able to include functionality for users running their own imgd servers to place their own skin files within the directory mentioned above, in case they want to display a custom skin on error.

@MisterFixx
Copy link

I aupport with @Foxtrek64 suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants