-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Adding parameter to root URL cause a 404 #1436
Comments
|
Hi @flaviocopes thanks for your answer :) Unfortunately I'm afraid that this behaviour is wrong and Grav should handle that anyway. Because of that I can't use Google PPC (or Adwords). Google is adding some parameters on the current URL, like: This doesn't work and as you explained and returns a 404. If I'm using Optimizely for A/B testing, Optimizely also adds some parameters to the URL, and it returns a 404, I don't have any control on how those tools add parameters unfortunately. I really think Grav should be able to handle it, what's your thoughts on that? Thanks, |
Please try my fix as it should resolve the issue. |
Hi @rhukster , thanks, I'll try it asap. |
@rhukster Thanks, tested and working properly. Really appreciated! |
Tested with Grav: 1.2.2
Platform: OSX El Capitan 10.11.6
Setup is the following one: https://getgrav.org/blog/macos-sierra-apache-multiple-php-versions
Hi, using a new instance of Grav with multi language setup, there is a 404 when adding parameters on the root page only.
Here is how to replicate:
my-page
./grav/user/config/system.yaml
the following lines to enable multi-lang setup:Then, try to fetch the following URLs:
/en
> 200/my-page
> 200/en?foo
> 404/my-page?foo
> 200I also created a video to demonstrate the issue: https://streamable.com/g411p
Thanks
The text was updated successfully, but these errors were encountered: