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 @fastify/secure-session to v7 [security] #438

Open
wants to merge 1 commit into
base: chore/renovateBaseBranch
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Apr 10, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@fastify/secure-session 6.2.0 -> 7.3.0 age adoption passing confidence

GitHub Vulnerability Alerts

CVE-2024-31999

Impact

At the end of the request handling, it will encrypt all data in the session with a secret key and attach the ciphertext as a cookie value with the defined cookie name. After that, the session on the server side is destroyed. When an encrypted cookie with matching session name is provided with subsequent requests, it will decrypt the ciphertext to get the data. The plugin then creates a new session with the data in the ciphertext. Thus theoretically the web instance is still accessing the data from a server-side session, but technically that session is generated solely from a user provided cookie (which is assumed to be non-craftable because it is encrypted with a secret key not known to the user).

The issue exists in the session removal process. In the delete function of the code, when the session is deleted, it is marked for deletion. However, if an attacker could gain access to the cookie, they could keep using it forever.

Patches

Fixed in 56d66642ecc633cff0606927601e81cdac361370.
Update to v7.3.0.

Workarounds

Include a "last update" field in the session, and treat "old sessions" as expired.
Make sure to configure your cookie as "http only".

References


Release Notes

fastify/fastify-secure-session (@​fastify/secure-session)

v7.3.0

Compare Source

⚠️ Security Release ⚠️

Fixes GHSA-9wwp-q7wq-jx35

What's Changed

New Contributors

Full Changelog: fastify/fastify-secure-session@v7.2.0...v7.3.0

v7.1.0

Compare Source

What's Changed

New Contributors

Full Changelog: fastify/fastify-secure-session@v7.0.0...v7.1.0

v7.0.0

Compare Source

What's Changed

New Contributors

Full Changelog: fastify/fastify-secure-session@v6.2.0...v7.0.0


Configuration

📅 Schedule: Branch creation - "" in timezone Europe/Paris, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


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

This PR has been generated by Mend Renovate. View repository job log here.

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.

None yet

0 participants