passport-oauth2

passport-oauth2 copied to clipboard

Arity check does not work. Therefore Id token will never work.

4

The google client code, returns an accessToken, refreshToken, and the result itself. Which now returns the idToken, or at least in my case. The way to get that back id...

arturohernandez10

Removes [growl](https://github.com/tj/node-growl). It's no longer used after updating ancestor dependency [mocha](https://github.com/mochajs/mocha). These dependencies need to be updated together. Removes `growl` Updates `mocha` from 2.5.3 to 10.2.0 Release notes Sourced from...

dependabot[bot]

Removes [minimist](https://github.com/minimistjs/minimist). It's no longer used after updating ancestor dependency [mocha](https://github.com/mochajs/mocha). These dependencies need to be updated together. Removes `minimist` Updates `mocha` from 2.5.3 to 10.2.0 Release notes Sourced from...

dependabot[bot]

`req.connection`, an alias of `req.socket` can be null, causing a null pointer exception when reconstructing a request URL. See https://github.com/jaredhanson/passport-oauth2/issues/183

mserranom

`req.connection` (deprecated alias of `req.socket`, see https://github.com/jaredhanson/passport-oauth2/issues/178) can be `null`, causing a null pointer exception when reconstructing a request URL. See upstream discussion in https://github.com/nodejs/undici/issues/1115 and https://github.com/nodejs/node/issues/41011.

mserranom

Hi, I found a pending deprecation hier: https://github.com/jaredhanson/passport-oauth2/blob/ea9e99adda82dff67502654347589866fea80eb2/lib/strategy.js#L103 I used This NODE_OPTIONS: ```bash NODE_OPTIONS="--pending-deprecation --throw-deprecation --trace-deprecation" ``` Output: ``` + node server.mjs node:internal/process/warning:162 throw warning; ^ DeprecationWarning: `url.parse()` behavior is...

CodeIter

rachmadideni

Support userInfo endpoint

1

Hi @jaredhanson, I see that the `profile` params are empty for me, would be possible to populate that with the `userInfo` endpoint? Most of OAuth2 providers (keycloak for example) supports...

ThallesP

Still use request.connection that deprecated by nodejs since v13.0.0

1

i found my project show deprecation warning with stack trace come here , look at the line of 27  https://nodejs.org/api/http.html#http_request_connection, why this lib still use request.connection?

LuckyArdhika

Without redirecting to azuread login page, i need to get the token using API itself

sugandansa