Skip to content
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

fix(deps): update dependency passport to v0.6.0 [security] #243

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Sep 25, 2022

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
passport (source) 0.4.1 -> 0.6.0 age adoption passing confidence

GitHub Vulnerability Alerts

CVE-2022-25896

This affects the package passport before 0.6.0. When a user logs in or logs out, the session is regenerated instead of being closed.


Release Notes

jaredhanson/passport (passport)

v0.6.0

Compare Source

Added
  • authenticate(), req#login, and req#logout accept a
    keepSessionInfo: true option to keep session information after regenerating
    the session.
Changed
  • req#login() and req#logout() regenerate the the session and clear session
    information by default.
  • req#logout() is now an asynchronous function and requires a callback
    function as the last argument.
Security
  • Improved robustness against session fixation attacks in cases where there is
    physical access to the same system or the application is susceptible to
    cross-site scripting (XSS).

v0.5.3

Compare Source

Fixed
  • initialize() middleware extends request with login(), logIn(),
    logout(), logOut(), isAuthenticated(), and isUnauthenticated() functions
    again, reverting change from 0.5.1.

v0.5.2

Compare Source

Fixed
  • Introduced a compatibility layer for strategies that depend directly on
    [email protected] or earlier (such as passport-azure-ad), which were
    broken by the removal of private variables in [email protected].

v0.5.1

Compare Source

Added
  • Informative error message in session strategy if session support is not
    available.
Changed
  • authenticate() middleware, rather than initialize() middleware, extends
    request with login(), logIn(), logout(), logOut(), isAuthenticated(),
    and isUnauthenticated() functions.

v0.5.0

Compare Source

Changed
  • initialize() middleware extends request with login(), logIn(),
    logout(), logOut(), isAuthenticated(), and isUnauthenticated()
    functions.
Removed
  • login(), logIn(), logout(), logOut(), isAuthenticated(), and
    isUnauthenticated() functions no longer added to http.IncomingMessage.prototype.
Fixed
  • userProperty option to initialize() middleware only affects the current
    request, rather than all requests processed via singleton Passport instance,
    eliminating a race condition in situations where initialize() middleware is
    used multiple times in an application with userProperty set to different
    values.

  • If you want to rebase/retry this PR, check this box

@codecov
Copy link

codecov bot commented Sep 25, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (402f94f) 54.67% compared to head (50e7568) 54.67%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #243   +/-   ##
=======================================
  Coverage   54.67%   54.67%           
=======================================
  Files         129      129           
  Lines        3111     3111           
  Branches      411      411           
=======================================
  Hits         1701     1701           
  Misses       1403     1403           
  Partials        7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@renovate renovate bot changed the title fix(deps): update dependency passport to v0.6.0 [security] fix(deps): update dependency passport to v0.6.0 [security] - autoclosed Nov 1, 2023
@renovate renovate bot closed this Nov 1, 2023
@renovate renovate bot deleted the renovate/npm-passport-vulnerability branch November 1, 2023 03:09
@renovate renovate bot changed the title fix(deps): update dependency passport to v0.6.0 [security] - autoclosed fix(deps): update dependency passport to v0.6.0 [security] Nov 1, 2023
@renovate renovate bot reopened this Nov 1, 2023
@renovate renovate bot restored the renovate/npm-passport-vulnerability branch November 1, 2023 06:11
@renovate renovate bot force-pushed the renovate/npm-passport-vulnerability branch from 67f4b9e to 50e7568 Compare November 1, 2023 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants