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
190 stars 175 forks source link

Registration HTTP Status does not need to be 200 #10

Closed fiddur closed 9 years ago

fiddur commented 9 years ago

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

The spec says "A successful response SHOULD use the HTTP 201 Created", so checking for status 200 is wrong.

fiddur commented 9 years ago

I see this is fixed in igi64's fork…
https://github.com/igi64/passport-openidconnect