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

VPN download link on stage points to incorrect domain #13312

Closed
jswinarton opened this issue Jun 26, 2023 · 5 comments
Closed

VPN download link on stage points to incorrect domain #13312

jswinarton opened this issue Jun 26, 2023 · 5 comments
Assignees
Labels
Bug 🐛 Something's not working the way it should

Comments

@jswinarton
Copy link

Description

As part of the FPN sunset initiative we have cleaned up the domains that the VPN project uses. Previously, the VPN used stage-vpn.guardian.nonprod.cloudops.mozgcp.net, but it now uses stage.guardian.nonprod.cloudops.mozgcp.net (i.e. without the -vpn). This is now breaking download links on stage bedrock pages.

Steps to reproduce

  1. Visit https://www-dev.allizom.org/en-US/products/vpn/download/
  2. Click the "Get Mozilla VPN" link under "Based on your current device"

Expected result

The link should direct to the new domain.

Actual result

The link directs to the old domain. (This still works at the moment but will stop working soon. We plan to take this down on June 27.)

Environment

Firefox/Ubuntu

@jswinarton
Copy link
Author

Hi @craigcook, thank you for making this change! I just went to verify that this is live, but the links on https://www-dev.allizom.org/en-US/products/vpn/download/ still point to the old domain. I checked on the What's Deployed status to verify that this commit is live and it seems to be -- maybe I am missing something?

@alexgibson
Copy link
Member

@jswinarton hi thanks for the update. It looks like there might be some download links that we're missed initially. I'll take a look

@alexgibson
Copy link
Member

Hmm, code looks ok - which leads me to wonder if there's an environment variable that needs updating

@alexgibson
Copy link
Member

Tried removing the variables here: mozmeao/www-config#525

If this doesn't work I'll ping Brett (our SRE) to take a look tomorrow

@alexgibson alexgibson reopened this Jul 4, 2023
reemhamz pushed a commit to reemhamz/bedrock that referenced this issue Jul 18, 2023
@alexgibson
Copy link
Member

Looks like this got fixed by our SRE follks, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐛 Something's not working the way it should
Projects
None yet
Development

No branches or pull requests

3 participants