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

Fix handling of redirect URI with --skip-provider-button #395

Closed
wants to merge 1 commit into from
Closed

Fix handling of redirect URI with --skip-provider-button #395

wants to merge 1 commit into from

Conversation

bastianeicher
Copy link

Fixes #327

@jehiah jehiah added the bug label May 22, 2017
@ploxiln
Copy link
Contributor

ploxiln commented Sep 30, 2017

You might need to add a bit more of the logic which the sign_in page uses for this: https://github.com/bitly/oauth2_proxy/blob/v2.2/oauthproxy.go#L367-L373

@jehiah jehiah added this to the v2.3 milestone Oct 23, 2017
systemizer added a commit to segmentio/oauth2_proxy that referenced this pull request Dec 3, 2017
applies fix indicated here: bitly#395
@rmoriz
Copy link

rmoriz commented Mar 2, 2018

@ploxiln
So this PR does not yet support the HTTP header "X-Auth-Request-Redirect" and therefore isn't merged, right?

@adrianchifor
Copy link

Any progress on this?

@ploxiln
Copy link
Contributor

ploxiln commented Jan 21, 2019

this repo is abandoned, there are a couple successors, see #628 (comment)

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

skip-provider-button seems to force final redirection url to be /
5 participants