GHSA-9wwp-q7wq-jx35
Suggest an improvement- Source
- https://github.com/advisories/GHSA-9wwp-q7wq-jx35
- Import Source
- https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2024/04/GHSA-9wwp-q7wq-jx35/GHSA-9wwp-q7wq-jx35.json
- JSON Data
- https://api.test.osv.dev/v1/vulns/GHSA-9wwp-q7wq-jx35
- Aliases
- Related
- Published
- 2024-04-10T17:15:50Z
- Modified
- 2024-04-11T14:31:13Z
- Severity
-
- 7.4 (High) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:H/I:N/A:N CVSS Calculator
- Summary
- @fastify/secure-session: Reuse of destroyed secure session cookie
- Details
-
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
- https://hackerone.com/reports/2374253
- Database specific
{ "nvd_published_at": "2024-04-10T22:15:07Z", "cwe_ids": [ "CWE-613" ], "severity": "HIGH", "github_reviewed": true, "github_reviewed_at": "2024-04-10T17:15:50Z" }- References
Affected packages
npm / @fastify/secure-session
Package
- Name
- @fastify/secure-session
- View open source insights on deps.dev
- Purl
- pkg:npm/%40fastify/secure-session