Skip to content
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

Documentation of use_redirect_parameter_if_present #657

Open
rarog opened this issue Nov 28, 2016 · 0 comments
Open

Documentation of use_redirect_parameter_if_present #657

rarog opened this issue Nov 28, 2016 · 0 comments

Comments

@rarog
Copy link

rarog commented Nov 28, 2016

The behaviour of use_redirect_parameter_if_present doesn't have enough documentation.
I had first to dig through the code to find out, that if the option is enabled, the corresponding parameter via GET should be named "redirect".
I had to further debug with trial and error until I found out, that the redirect parameter may not start with a trailing slash. Especially this should be documented in detail or function routeExists in RedirectCallback.php should do a ltrim($route, '/') to make sure, that a trailing slash is removed and the function is more tolerant.

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

No branches or pull requests

1 participant