Store redirects independint of trailing slash about wpcom-legacy-redirector HOT 3 OPEN

@jeremeylduvall Those pesky trailing slashes never go away :)

Thanks for scoping this out and for sharing the provided sample code. I was able to test, validate, and implement automatically supporting both scenarios (with or without the trailing slash). I also included query string compatibility as well and I'll be forking + committing code soon.

from wpcom-legacy-redirector.

@GaryJones Do we want to allow the creation of redirects ($from_url) for both trailing slash URL variants through the UI? It is currently allowed. This decision impacts how the most optimal solution is implemented for v1.4.0 and supported moving forward.

i.e...
https://example.com/redir to https://example.com/page1
https://example.com/redir/ to https://example.com/page2

Possible Solutions:
A) Consistent trailing slash agnostic support across the CLI, UI, and internally within the plugin. We'd uniformly consider the example above as one redirect route. Only the CLI can create a trailing slash variant with --skip-validation. We can apply the trailing slash policy based on the site's current permalink structure.
B) Allow these to operate as separate redirect routes as is allowed today. We'd add logic to maybe_do_redirect() to support a fallback check on the second possible URL - the trailing slash variant as outlined in this issue. We'd further optimize for performance.

Thoughts? Thank you, sir!

from wpcom-legacy-redirector.

+1 to this. We've had to hack the core plugin code for this use case.

from wpcom-legacy-redirector.