feat(response_type): response_type in options

jaredhanson / passport-oauth2

OAuth 2.0 authentication strategy for Passport and Node.js.

https://www.passportjs.org/packages/passport-oauth2/?utm_source=github&utm_medium=referral&utm_campaign=passport-oauth2&utm_content=about

MIT License

607 stars 343 forks source link

feat(response_type): response_type in options #107

Closed brunosantanarj closed 5 years ago

brunosantanarj commented 5 years ago

response_type is fixed by limiting the authentication options. What I did was check if there is response_type in the options, if it has, I hedge response_type, if not, code.

coveralls commented 5 years ago

Coverage remained the same at 100.0% when pulling 8c884033ed65ee6218e0ab182bbdeb34a2864441 on brunosantanarj:feat/response_type into a9480960be04ac5312f9eae2a6d06d0e8d0e5dda on jaredhanson:master.

jaredhanson commented 5 years ago

What response_type type do you need to set other than code? The authorization code flow is the one specified by the OAuth 2.0 specification, and any other response_type is likely to have a far different protocol (and thus not be compatible anyway).