My gigantic avatars are not working!

AI-generated summary

The discussion revolves around the issue of gigantic avatars not working on iPads. Rainbowresidue initially reported the problem, stating that avatars show up on other devices but not on iPads. sam discovered that the issue was due to a hardcoded limit of 500px in the avatars controller, which was causing a redirect. sam suggested that Rainbowresidue could temporarily fix the issue by requesting smaller avatar sizes. However, Rainbowresidue explained that the larger size was requested by members and had worked before the last update. sam agreed to lift the 500px restriction, and zogstrip subsequently pushed a fix to resolve the issue.

Dear Gentleman, @codinghorror @sam
We are having an issue on iPads too. The avatars won’t show on iPads, but show everywhere else (phones, laptops, PCs).

Here’s a snapshot from my iPad Air 2. (My friend also can’t see avatars on his iPad either, so it’s not just me)

Thanks in advance for looking into this.
Have a pleasant day,
Trish
Edit to add: Before the last update, I used my iPad and the avatars showed just fine, if that helps.

Yeah, you have a site customization there that is asking for gigantic avatars.

https://global.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/360/5243_1.png

works but

https://global.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/600/5243_1.png

@zogstrip there is a hardcoded

in avatars controller, perhaps we should allow up to 1000 ? It just results in a redirect anyway

return render_blank if size < 8 || size > 500

@Rainbowresidue one easy fix on your side would be to stop asking for 600px sized avatars :), I doubt anyone is uploading that resolution.

Also, when you ask for an avatar in an unsupported size you get a redirect, which hurts perf.

Thanks Sam,

But the avatars were requested by members. Lots of forums use this size. (We have to keep up with the competition.)
Quite frankly, it worked before without any issues, only stopped showing on iPads after the last update.

I am fine to lift the 500px restriction

Thank you very much, I think that would help our I.T. gents. :slight_smile:

Just pushed a fix :rocket:

https://github.com/discourse/discourse/commit/c104e4c0225b15b09fd9dfd49c6f27bb7f11e763