-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
Redirect URL with query strings #339
Comments
Same thing with an old wordpress path :
Kirby 4.1.2 |
What I understood after I opened this ticket was that Kirby's routing doesn't support URLs with querystring. There's no reply from the dev, so maybe Retour will not add support for it. You may consider Redirects instead: https://github.com/bnomei/kirby3-redirects. Last month they updated it to fix compatibility with Kirby 4. |
This is a bit tricky for me: I found a way to support something like |
Not sure if I can offer support or https://github.com/bnomei/kirby3-redirects is indeed your better choice. The difference is that Retour is build on Kirby's routing - which doesn't support queries for route patterns. @bnomei has built their own matching function to see if a redirect should be applied to the current request. |
I have an old URL that looks like this:
read.php?article=1-welcome-to-website
In the Failures tab, I can see Retour found the error in path
read.php
, but without the query stringI've tried to explicitly add a new redirect from the Routes tab:
read.php?article=1-welcome-to-website
But when trying to access the old URL, the page only redirected to Kirby's site error page; and I can see that the failures Hit count for
news.php
has increased. So it seems that Retour is ignoring it.Is there any way to make this work?
The text was updated successfully, but these errors were encountered: