jaredhanson / jaredhanson/passport-readability

Critical: Update passport-oauth to ^1.0.0

Open
#2 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
4
Forks
0
PR merge metrics
No merged PRs in 30d

Description

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. Of course you know what I am talking about. ;)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the package's dependency declaration and the authentication entry point that calls req.logIn(...). Update the passport-oauth requirement to ^1.0.0, then verify that the strategy no longer crashes when used with Passport 0.3.0 or its latest version.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js
Domain
authentication
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.