Skip to content

fix: dependency security vulnerabilities (DELO-6764, DELO-6786, DELO-6721, DELO-6638, DELO-6639) - #261

Merged
ikrascloudinary merged 2 commits into
masterfrom
DELO-6764/security-fixes
Nov 11, 2026
Merged

ikrascloudinary merged 2 commits into
masterfrom
DELO-6764/security-fixes

Conversation

@ikrascloudinary

@ikrascloudinary ikrascloudinary commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Vulnerabilities resolved

Package Before → After Tickets Advisory
qs 6.16.0 → 6.17.1 DELO-6786 (Improper Encoding, needs ≥6.17.0), DELO-6721 (Uncaught Exception, needs ≥6.16.1) GHSA-w4cr-vjpx-p82q, GHSA-6pq8-jhm2-2vcw
body-parser 1.20.8 → 1.20.9 DELO-6764 (Uncaught Exception on compressed body with trailing bytes) GHSA-27r3-jxmh-p828
http-cache-semantics 4.2.0 → 4.3.0 DELO-6638 (max-stale), DELO-6639 (Vary: *) upstream #56, #57

All three are transitive: qs via express/body-parser/chai-http, body-parser via express@4.22.3 (~1.20.5), http-cache-semantics via got → cacheable-request (^4.2.0).

Snyk lists "no fixed version" for the two http-cache-semantics tickets, but those were filed before 4.3.0 shipped (2026-10-04), which contains "Fix: handle Vary wildcard and inherited headers" (#57) and closes #56.

Dependencies updated

Resolutions

  • qs: raised floor ^6.16.0 → ^6.17.1.
  • Added body-parser: ^1.20.9 and http-cache-semantics: ^4.3.0 so the parent ranges can't re-admit the vulnerable versions.
  • Existing protobufjs, proxy-addr, @grpc/grpc-js pins kept.

Lockfile

  • Only the three entries above change vs master. No other transitive bumps.

Package version

  • Unchanged; release-please handles it.

Test results

  • yarn install --frozen-lockfile clean; yarn check --integrity and yarn outdated run without lockfile errors.
  • yarn test: 13 passing.
  • yarn why confirms single copies: qs@6.17.1, body-parser@1.20.9, http-cache-semantics@4.3.0.

Note (pre-existing, not touched here): patch-package warns that patches/cloudinary+2.9.0.patch targets 2.9.0 while cloudinary@2.10.1 is installed.

Test plan

  • yarn test passes

🤖 Generated with Claude Code

ikrascloudinary and others added 2 commits November 1, 2026 15:10
…6721, DELO-6638, DELO-6639)

Bump transitive qs 6.16.0 -> 6.17.1, body-parser 1.20.8 -> 1.20.9 and
http-cache-semantics 4.2.0 -> 4.3.0, pinned via resolutions so parent
ranges can't re-admit the vulnerable versions.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…arn.lock

The resolutions were added after the lock entries existed, so the
^1.20.9 / ^4.3.0 patterns were missing from the keys and `yarn outdated`
failed with "Outdated lockfile". Versions are unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ikrascloudinary
ikrascloudinary marked this pull request as ready for review November 3, 2026 08:00
@ikrascloudinary
ikrascloudinary merged commit e6884cc into master Nov 11, 2026
5 checks passed
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.

2 participants