search

jaredhanson / passport-openidconnect

OpenID Connect authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-openidconnect/?utm_source=github&utm_medium=referral&utm_campaign=passport-openidconnect&utm_content=about
MIT License
191 stars 177 forks source link

Fix for passing options into the utils.originalURL function #48

Closed PhilipSkinner closed 2 years ago

PhilipSkinner commented 7 years ago

I found an issue when attempting to use this library behind a proxy and found that it was not passing my main options object through to the utils.originalURL function so it could determine if it should trust the x-forwarded-host and x-forwarded-proto headers in the request.

I have added some rudimentary tests over the originalURL function aswell to make sure its functionality cannot be changed without being noticed.

jaredhanson commented 2 years ago

Merged, thanks!