Skip to content

Conversation

@lionello
Copy link

@lionello lionello commented Feb 9, 2017

This PR fixes the return_url when using the OpenIDConnect middleware from a router under a sub-path. For example,

const OauthRoutes = require('./routes/oauth')
app.use('/oauth', OauthRoutes)//"login"

Without baseUrl, the return_url will be /authorize?response_type=..., but it should be /oauth/authorize?....

@lionello lionello force-pushed the fix_redir_baseUrl branch from a0df306 to c4347a4 Compare April 14, 2017 16:11
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

Successfully merging this pull request may close these issues.

1 participant