passport-oauth2
passport-oauth2 copied to clipboard
`node-oauth` throws intermittent InternalOAuthErrors on fast connections
Hi all, there's an existing bug with passport-google-oauth2 where, due to an upstream issue in node-oauth 0.9.0, OAuthing into Google accounts fails intermittently over fast connections.
Good news! The upstream issue has been fixed and published as node-oauth 0.10.0. However passport-google-oauth2 depends on this package, and this package is pinned at node-oauth 0.9.x. Can you bump this dependency to 0.10.0 and publish a new version?
Here is the change made on node-oauth (named oauth in the dependencies) from the current version used by passport-oauth2 (0.9.x) to the 0.10.0: https://github.com/ciaranj/node-oauth/compare/0.9.15...0.10.0 (made by https://github.com/ciaranj/node-oauth/pull/363)
Once the version will be bumped this will solve several duplicated issues in passport-google-oauth2 as said by @8bitDesigner, such as https://github.com/jaredhanson/passport-google-oauth2/issues/87, https://github.com/jaredhanson/passport-google-oauth2/issues/64, but also may be related to some of the issues of that repo such as involving InternalOAuthError: Failed to obtain access token: https://github.com/jaredhanson/passport-oauth2/issues/155, https://github.com/jaredhanson/passport-oauth2/issues/113
This is a pretty critical issue. Can it please be prioritized? My only recourse right now is to fork and patch. Given the number or forks it seems many others have also done so.
Let me know if I or other community members can help.