Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

rd param allows other domain #547

Closed
wants to merge 1 commit into from
Closed

rd param allows other domain #547

wants to merge 1 commit into from

Conversation

icchy
Copy link

@icchy icchy commented Feb 14, 2018

rd param at /oauth2/start currently allows only the path starts with /.
Otherwise the request is redirected to / even if rd param is required to allow other domain URI.
This commit will work on following nginx snippet:

location /secret {
    satisfy any;
    auth_request /oauth2/auth;
    error_page 401 = /oauth2/start?rd=$scheme://$host$uri;
}

which is useful for multiple path required to authenticated.

@icchy icchy closed this Feb 14, 2018
@icchy
Copy link
Author

icchy commented Feb 14, 2018

seems to be conflicted with #464 and #544

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant