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
192 stars 177 forks source link

redirectURI or redirectURIs? #9

Open fiddur opened 9 years ago

fiddur commented 9 years ago

It seems in the code that an array of redirect_uris can be supplied with options.redirectURIs, but the code tries to join redirectURI. Is it supposed to be two options or one?

https://github.com/jaredhanson/passport-openidconnect/blob/master/lib/registration.js#L40-L42

fiddur commented 9 years ago

Ant by the way; why are they joined by ' '? According to the specification, it should be an "Array of Redirection URI values used by the Client."