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.

Sign up for GitHub

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

Jump to bottom

REQUEST_URI contains arguments in non-Apache servers #146

Closed
gilzow opened this issue May 8, 2019 · 1 comment
Closed

REQUEST_URI contains arguments in non-Apache servers #146

gilzow opened this issue May 8, 2019 · 1 comment
Labels
bug
Milestone
0.6.0

Comments

@gilzow
Copy link

gilzow commented May 8, 2019

See this conversation for detailed information.

TL;DR : in non-apache servers,$_SERVER['REQUEST_URI'] contains both path components and arguments which causes the test at line 271 (in class-wp-saml-auth.php) to always evaluate to true. In this scenario a user is then returned to the login page after successfully authenticating, instead of the dashboard.

Pull request submitted #145
@danielbachhuber danielbachhuber added this to the 0.5.3 milestone May 9, 2019
@danielbachhuber
Copy link
Contributor

Fixed by #145

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
2 participants
@danielbachhuber @gilzow