Skip to content

chore(deps): bump multer, body-parser, fast-uri, and qs in demos/extractor - #1093

Merged
ericcurtin merged 1 commit into
docker:mainfrom
katsugtgz:dep-multer-2026-10-25
Oct 25, 2026
Merged

ericcurtin merged 1 commit into
docker:mainfrom
katsugtgz:dep-multer-2026-10-25

Conversation

@katsugtgz

Copy link
Copy Markdown
Contributor

Bump multer to 2.5.0 in demos/extractor and refresh three transitive dependencies in the lockfile to address GHSA-q7qp-rjg7-q26m, GHSA-27r3-jxmh-p828, GHSA-v422-hmwv-36x6, GHSA-333x-w4r6-6crp, GHSA-w74x-v6f9-j3xg, GHSA-wjr9-j795-jxcr, GHSA-xhq7-xmqj-769h, GHSA-4mjr-xmp4-gh2g, and GHSA-x5fp-wj9c-mxmx.

Evidence:

  • demos/extractor/package.json pinned multer@2.4.0; the lockfile resolved body-parser 1.20.5, fast-uri 3.1.8, qs 6.15.2
  • osv-scanner reported the nine IDs above for this directory before the update

Changes:

  • multer 2.4.0 -> 2.5.0 (direct pin)
  • body-parser 1.20.5 -> 1.20.9
  • fast-uri 3.1.8 -> 3.1.9
  • qs 6.15.2 -> 6.16.0
  • express resolves to 4.22.3 within the existing ^4.18.2 range so its qs dependency (~6.15.1 in 4.22.2, ~6.16.0 in 4.22.3) dedupes to the patched 6.16.0 instead of nesting a vulnerable 6.15.3

Validation:

  • npm install in demos/extractor completes
  • osv-scanner reports zero advisories for demos/extractor after the update (all nine IDs gone, none added)

Scope: dependency manifest and lockfile update only.

@sourcery-ai sourcery-ai Bot 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.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

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.

🟢 Approval recommended

No blocking issues were identified in the scoped dependency updates, and the reported installation and security scan validations passed.

No open issues 🎉

1 file not reviewed
  • demos/extractor/package-lock.json: Generated file
What changed in this PR

Updates the extractor demo’s dependencies to address nine reported security advisories without changing application code.

Changes:

  • Pins multer to 2.5.0 in the manifest and lockfile.
  • Updates body-parser, fast-uri, and qs to patched versions.
  • Refreshes Express and supporting dependencies so Express can share the patched qs version.
File Description
demos/​extractor/​package.json Updates the direct multer pin to 2.5.0.
demos/​extractor/​package-lock.json Records updated dependency versions, integrity hashes, and dependency requirements.

Review effort: Balanced

@ericcurtin ericcurtin 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.

LGTM. Lockfile hashes verified against npm, npm ci clean, npm audit 0 vulns, multer upload and size limit smoke-tested.

@ericcurtin
ericcurtin merged commit e64726e into docker:main Oct 25, 2026
8 checks passed
@ericcurtin

Copy link
Copy Markdown
Contributor

Merged, thanks! model-runner is being deprecated in favour of https://github.com/llmmanorg/llmman, so please open future PRs and issues there.

Follow-ons: demos/extractor/server.js fails to start on main (pdf-data-extractor is ESM-only, require fails), and the demo has no Dependabot coverage.

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