Skip to content

Make redirection directives less confusing - #2889

Draft
waxlamp wants to merge 2 commits into
masterfrom
redirect-reorg
Draft

Make redirection directives less confusing#2889
waxlamp wants to merge 2 commits into
masterfrom
redirect-reorg

Conversation

@waxlamp

@waxlamp waxlamp commented Aug 14, 2026

Copy link
Copy Markdown
Member

This PR splits apart the four redirects in redirector/netlify.toml into four subdirectories, each handling one redirect. This better matches the structure of our redirection Netlify apps, of which there are four, and which currently rely solely on domain matching to execute the correct redirection rule.

Reasons for doing this:

  • less confusing for devs: each subdirectory handles one redirect and can be matched (by build directory) to one Netlify app
  • less "blast radius" when there's a problem: a single typo anywhere in the unified netlify.toml file could cause up to all four of the redirections to stop working

Reasons not to do this:

  • too many files / too much repetition to replace something that seems to be working
  • we might want instead to unify all the Netlify redirector apps into a single app that has multiple hostnames associated to it (in something of an opposite move from this PR)

I am not sure whether this approach is better, if we should leave well enough alone, or if we should leave this as is and unify the Netlify apps into one. Accepting opinions 🙂

(Note: This leaves the original netlify.toml file in place so as not to disrupt Netlify's live operations at the moment. A later PR can remove that after the Netlify apps have been repointed to the new locations.)

This leaves the original material in place so as not to disrupt
Netlify's live operations at the moment. A later PR will remove those
after the Netlify apps have been repointed to the new locations.
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.

1 participant