-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update passport-oauth to ^1.0.0 #8
Comments
FYI, [email protected] just got released which applies a general patch. So I removed the "Critical" in the title. |
@analog-nico I've created related PR #5 about 6 months ago, but they don't accept my PR until now. I think this repository is NOT maintained anymore. Sad. FYI for someone who looking for similar issues. |
Oh-oh-oh. they just merged my PR. Yay! @analog-nico FYI, Your issue has been resolved ;) Please update status of this issue. Thanks for your support! |
Unless this library upgrades to passport-oauth ^1.0.0 it will crash
req.logIn(...)
when the app is using the latest version of Passport. This is caused by introducing lazy session initialization in passport 0.3.0. See the technical details and Jared's explanation.Let me know if I can help.
The text was updated successfully, but these errors were encountered: