jaredhanson / jaredhanson/passport-http

passReqToCallback support missing from Digest strategy

Open
#56 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
JavaScript
Stars
261
Forks
112
PR merge metrics
No merged PRs in 30d

Description

The **Basic** stragegy implements the **passReqToCallback** option, but support for this is missing in the **Digest** strategy.
Perhaps both callbacks, **secret** and **validate**, should honour this option.

Contributor guide

Open the contributing guide

Research direction

Compare the Digest strategy's callback handling with the Basic strategy's passReqToCallback support. Check how the secret and validate callbacks are invoked, then verify that both honor the option without breaking existing Digest authentication behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
authentication
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.