Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Add automatic redirect for all /mobile/ app urls to /android/

See original GitHub issue

In #5541 we enabled the user profile but seemed to have missed the mobile app for the redirect.

Looking at https://github.com/mozilla-services/cloudops-deployment/blob/master/projects/amo/puppet/modules/amo_proxy/templates/nginx.addons.conf.erb I see that there are quite a few firefox|android rules which probably should also add |mobile to the expression.

Example: https://addons.mozilla.org/en-US/mobile/user/EnTeQuAk/

@autrilla @bqbn can you take a look at that?

Issue Analytics

State:
Created 5 years ago
Comments:7 (7 by maintainers)

Top GitHub Comments

1reaction

EnTeQuAkcommented, Aug 9, 2018

Yeah, I updated the subject of this issue, let’s rephrase this: Support for the mobile app got removed but we should redirect all users to /android/ automatically. We have some code in addons-server that does that but it seems broken, given that a redirect like that is much more reasonable directly in nginx for performance reasons, can we do that?

Tldr; Redirect all /{locale}/mobile/ URLs to /{locale}/android/ since mobile doesn’t exist anymore (… since 2012 🤣)

0reactions

muffinresearchcommented, Mar 25, 2019

Closing based on https://github.com/mozilla/addons-frontend/issues/5881#issuecomment-412136213