Skip to content

chore(deps): bump body-parser from 1.20.4 to 1.20.6#2728

Open
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/body-parser-1.20.6
Open

chore(deps): bump body-parser from 1.20.4 to 1.20.6#2728
dependabot[bot] wants to merge 1 commit into
mainfrom
dependabot/npm_and_yarn/body-parser-1.20.6

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jul 23, 2026

Copy link
Copy Markdown
Contributor

Bumps body-parser from 1.20.4 to 1.20.6.

Release notes

Sourced from body-parser's releases.

1.20.6

Important: Security

What's Changed

Full Changelog: expressjs/body-parser@1.20.5...1.20.6

v1.20.5

What's Changed

The reason for this release is a fix to the extended urlencoded parser returning objects instead of arrays for large array inputs (> 100) on qs@6.14.2+. (expressjs/body-parser#716)

New Contributors

Special thanks to triager @​krzysdz for keeping this on our radar and effectively triaging the specific issue!

Full Changelog: expressjs/body-parser@1.20.4...1.20.5

Changelog

Sourced from body-parser's changelog.

1.20.6 / 2026-07-09

  • Security fix for GHSA-v422-hmwv-36x6
  • fix: improve limit option validation (#698)
    • Invalid limit values (e.g. unparseable strings or NaN) now throw instead of being silently ignored, which previously disabled size limit enforcement
    • null and undefined fall back to the default 100kb limit

1.20.5 / 2026-04-24

  • refactor(json): simplify strict mode error string construction
  • fix: extended urlencoded parsing of arrays with >100 elements (#716)
  • deps: qs@~6.15.1
Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

@dependabot
chore(deps): bump body-parser from 1.20.4 to 1.20.6
1bcd2b5 
Bumps [body-parser](https://github.com/expressjs/body-parser) from 1.20.4 to 1.20.6.
- [Release notes](https://github.com/expressjs/body-parser/releases)
- [Changelog](https://github.com/expressjs/body-parser/blob/master/HISTORY.md)
- [Commits](expressjs/body-parser@1.20.4...1.20.6)

---
updated-dependencies:
- dependency-name: body-parser
  dependency-version: 1.20.6
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Jul 23, 2026
@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

PR author is in the excluded authors list.

@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1bcd2b5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel

vercel Bot commented Jul 23, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
hyperdx-oss Ignored Ignored Preview Jul 23, 2026 10:53pm
hyperdx-storybook Ignored Ignored Preview Jul 23, 2026 10:53pm

Request Review

@github-actions github-actions Bot added the review/tier-1 Trivial — auto-merge candidate once CI passes label Jul 23, 2026
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

🟢 Tier 1 — Trivial

Docs, images, lock files, or a dependency bump. No functional code changes detected.

Why this tier:

  • Bot author: dependabot[bot]

Review process: Auto-merge once CI passes. No human review required.
SLA: Resolves automatically.

Stats
  • Production files changed: 0
  • Production lines changed: 0
  • Branch: dependabot/npm_and_yarn/body-parser-1.20.6
  • Author: dependabot[bot]

To override this classification, remove the review/tier-1 label and apply a different review/tier-* label. Manual overrides are preserved on subsequent pushes.

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Deep Review

✅ No critical issues found.

This PR is a Dependabot bump of the transitive dependency body-parser from 1.20.4 to 1.20.6, changing only the root yarn.lock. There are no source or package.json changes. body-parser is resolved transitively through express (selector ~1.20.3); the bump also pulls qs to ~6.15.1 and raw-body to ~2.5.3. The 1.20.6 release is a security patch, so the update is net-positive for risk posture.

The one behavioral change in 1.20.6 that could affect this repo is stricter limit validation: an invalid limit (unparseable string or NaN) now throws at parser construction instead of being silently ignored. Every body-parser call site in the app passes a valid hardcoded string:

  • packages/api/src/api-app.ts:55-57 — express.json, express.text, and express.urlencoded all use limit: '32mb', which bytes parses cleanly.
  • packages/api/src/opamp/app.ts:12-18 — express.raw uses limit: '10mb', also valid.

Because no limit value is dynamic, env-derived, or non-numeric, the stricter validation does not introduce a startup failure here. The change is safe to merge.

Reviewers (1): manual-focused-review (correctness + dependency-impact lens).

The automated multi-agent persona fan-out could not execute: the sandbox blocked git and grep/glob for the orchestrator and every subagent (bubblewrap /home/.mcp.json permission error on all invocations), so persona reviewers had no diff or search access. This review was performed by reading the resolved lockfile entry and tracing the affected Express entry points directly.

Testing gaps:

  • Body-parser call-site verification relied on tracing known Express entry points (api-app.ts, opamp/app.ts, mcp/app.ts, external-v2 router) rather than an exhaustive packages/** sweep, since grep/glob were unavailable; if another app passes a dynamically computed limit, it should be confirmed to resolve to a valid byte string before relying on this bump.

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

E2E Test Results

✅ All tests passed • 242 passed • 1 skipped • 898s

Status Count
✅ Passed 242
❌ Failed 0
⚠️ Flaky 1
⏭️ Skipped 1

Tests ran across 4 shards in parallel.

View full report →

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code review/tier-1 Trivial — auto-merge candidate once CI passes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

0 participants