Skip to content

Conversation

@gcoxmoz
Copy link
Contributor

@gcoxmoz gcoxmoz commented Jan 9, 2026

A request for an uglifier www. as a prepended domain came in after the initial commit, sorry for the revisit.

Refractr PR Checklist

JIRA ticket: IO-3544 / #417

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.

@gcoxmoz gcoxmoz requested a review from a team as a code owner January 9, 2026 08:24
Copy link
Member

@bkochendorfer bkochendorfer left a comment

Choose a reason for hiding this comment

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

Is there a link to the request? We avoid adding www subdomains on to an existing subdomain unless this is existing or previous art that we need to accommodate for.

@gcoxmoz
Copy link
Contributor Author

gcoxmoz commented Jan 9, 2026

Yes, in IO-3544. Even including -my- suggesting not to. :)

Copy link
Member

@bkochendorfer bkochendorfer left a comment

Choose a reason for hiding this comment

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

If that domain does not already exist let's not create it.

@gcoxmoz
Copy link
Contributor Author

gcoxmoz commented Jan 14, 2026

I accept this Uno Reverse.

@gcoxmoz gcoxmoz closed this Jan 14, 2026
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.

2 participants