Skip to content

fakespot postmortem redirect rule#406

Merged
pattersonbl2 merged 2 commits intomainfrom
fakespot-redirects
Jul 22, 2025
Merged

fakespot postmortem redirect rule#406
pattersonbl2 merged 2 commits intomainfrom
fakespot-redirects

Conversation

@pattersonbl2
Copy link

@pattersonbl2 pattersonbl2 commented Jul 17, 2025

Refractr PR Checklist

JIRA ticket: [link to relevant JIRA or other system ticket]

When creating a PR for Refractr, confirm you've done the following steps for a smooth CI and CD experience:

  • Have you updated the relevant YAML in the PR?
  • Have you checked the relevant YAML for any possible dupes regarding your domain?
  • Have you checked if there are any TLS cert concerns - e.g. if the domain being redirected already exists, and it is being changed to point at refractr, is a temporary TLS 'outage' while waiting for certification via HTTP challenge okay? If not, add a note to the JIRA ticket.
  • If desired, have you generated the nginx config manually to confirm updates work as expected?

After PR merge, next steps include:

  • A merge to the main branch will automatically deploy refractr's stage environment -- deploying the prod environment requires a GitHub release to be created.
  • Once deployed, refractr's certmap must be updated and DNS entries must be changed -- SRE can help with this. Please pull someone in on the JIRA ticket or ask for help in #sre on Slack.

@pattersonbl2 pattersonbl2 requested a review from a team as a code owner July 17, 2025 19:33
- www.mozilla.org/en-US/about/governance/policies/participation/reporting/:
- cpg-hotline.mozilla.community

- https://blog.mozilla.org/en/mozilla/building-whats-next/:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you can safely strip the https:// prefix here, looking at the other redirects

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

Copy link
Contributor

@jbuck jbuck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feedback in PR

@pattersonbl2
Copy link
Author

Feedback in PR

@jbuck looks like all requested changes are complete.

@pattersonbl2 pattersonbl2 merged commit 0b1c50f into main Jul 22, 2025
@pattersonbl2 pattersonbl2 deleted the fakespot-redirects branch July 22, 2025 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants