Skip to content

[GHSA-8wpr-639p-ccrj] Nest has a Fastify URL Encoding Middleware Bypass (TOCTOU) - #9632

Open
nikpivkin wants to merge 1 commit into
nikpivkin/advisory-improvement-9632from
nikpivkin-GHSA-8wpr-639p-ccrj
Open

nikpivkin wants to merge 1 commit into
nikpivkin/advisory-improvement-9632from
nikpivkin-GHSA-8wpr-639p-ccrj

Conversation

@nikpivkin

Copy link
Copy Markdown

Updates

  • Affected products
  • CVSS v4
  • Severity

Comments
The range should be < 11.1.11, not < 11.1.10. The advisory text says "Patched in @nestjs/platform-fastify@11.1.11".

The fix nestjs/nest@c4cedda is in tag v11.1.11 and not in tag v11.1.10.

So version 11.1.10 is still vulnerable but is outside the current range.

The form does not accept the E:U metric in the CVSS 4.0 vector (see #5357), so I had to remove it. Please keep E:U if possible, I did not mean to change the severity.

@github

github commented Sep 19, 2026

Copy link
Copy Markdown
Collaborator

Hi there @kamilmysliwiec! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

Copilot AI balanced review requested due to automatic review settings September 19, 2026 15:17
@github-actions
github-actions Bot changed the base branch from main to nikpivkin/advisory-improvement-9632 September 19, 2026 15:18
@nikpivkin

Copy link
Copy Markdown
Author

The severity change from MODERATE to CRITICAL is not intended. The form does not accept E:U in the CVSS 4.0 vector (#5357), so I had to remove it, and the score went up. Please keep the original vector CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:H/VI:H/VA:N/SC:N/SI:N/SA:N/E:U and severity MODERATE. The only change I meant is the range.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The valid E:U metric and corresponding moderate severity must be restored.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 High severity

Open (1)
What changed in this PR

Updates the NestJS Fastify advisory’s affected range and severity metadata.

Changes:

  • Removes the stale < 11.1.10 range override, leaving 11.1.11 as the fix.
  • Removes CVSS E:U and changes severity to critical; both are unintended.
File Description
GHSA-8wpr-639p-ccrj.json Updates affected-version and severity metadata.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

{
"type": "CVSS_V4",
"score": "CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:H/VI:H/VA:N/SC:N/SI:N/SA:N/E:U"
"score": "CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:H/VI:H/VA:N/SC:N/SI:N/SA:N"
"CWE-367"
],
"severity": "MODERATE",
"severity": "CRITICAL",

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"severity": "CRITICAL",
"severity": "MODERATE",

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.

3 participants