Letter avatars broken in 1.5b5?


(Stephen) #1

We’ve just upgraded to 1.5b5(33) and letter avatars aren’t working.

They link to:

/letter_avatar_proxy/v2/letter/l/a6a055/45.png

Our site setting is still:

/letter_avatar_proxy/v2/letter/{first_letter}/{color}/{size}.png

While I notice that avatars here on meta are being served from avatars.discourse.org

Any ideas?


User pictures broken after latest update (v1.5.0.beta5)
(mrosas) #2

Ditto for us.

Looks like there were changes that went in recently impacting this code here:


(Bryan Weissman) #3

At least until they fix it, changing the external system avatars URL to “https://avatars.discourse.org/v2/letter/{first_letter}/{color}/{size}.png” fixed the problem for me.


User pictures broken after latest update (v1.5.0.beta5)
(Stephen) #4

Looks like this requires a launcher rebuild to resolve, avatars are working once again.

Presumably grabbing the latest build via rebuild either creates or grabs local copies of these images. All you’re doing above is using the versions hosted on another site.

@sam if a beta upgrade isn’t going to work correctly until a rebuild has happened, isn’t it a bit silly to prompt the user with an upgrade button in the back end?


(Sam Saffron) #5

Confirmed, you must rebuild to get the fix, I will see if I can add a big red warning on /admin/upgrade page


(Stephen) #6

That’ll do it. Thanks!


(Alan Tan) #7

The avatars breaks in development mode though since nginx is not running on top of the rails web server. :grin:


(Sam Saffron) #8

Yeah I noticed that, maybe we should add a route for it, mainly for dev @mpalmer


(Matt Palmer) #9

Would that be a route for /letter_avatar_proxy to feed back into the internal avatar generation code? I think that’d be reasonable.


(Sam Saffron) #11

I think we sorted this out a while back, closing, flag to reopen if still an issue.


(Sam Saffron) #12