Skip to content

deps: @hono/node-server 2.1.5 in the bridge and fieldpass (GHSA-rmxm-3fg6-px4f) - #233

Merged
askalf merged 1 commit into
masterfrom
hono-node-server-2.1.3
Oct 29, 2026
Merged

askalf merged 1 commit into
masterfrom
hono-node-server-2.1.3

Conversation

@askalf

@askalf askalf commented Oct 28, 2026

Copy link
Copy Markdown
Owner

Fixes Dependabot alerts 55 and 56 (GHSA-rmxm-3fg6-px4f, medium): before 2.1.3, @hono/node-server's serveStatic decodes the request path a second time, so a static path can bypass middleware.

  • The overrides floor for @hono/node-server moves from ^2.0.5 to ^2.1.3 in package.json and policy/package.json.
  • Both lockfiles resolve 2.1.5 at the top level. No other package moves.
  • Tests on a clean npm ci: bridge 171 pass, fieldpass 176 pass, 0 fail.

Alert 54 (deepmerge, low, prototype poisoning in mergeObject) stays open. No patched release exists, and the only caller is puppeteer-extra merging plugin options this repo writes itself, never request data.

…3fg6-px4f)

serveStatic decoded the request path twice before 2.1.3, which lets a
static path slip past middleware. The override floor moves from ^2.0.5
to ^2.1.3 in both package.json files; both lockfiles resolve 2.1.5 at
the top level, nothing else moves.
@github-actions github-actions Bot added policy fieldpass policy layer (policy/) size/XS Under 10 hand-written lines labels Oct 28, 2026

@sprayberry-redline sprayberry-redline left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Verdict: approve. Both manifests require @hono/node-server ^2.1.3, and both lockfiles consistently resolve 2.1.5. The update remains compatible with the SDK's dependency range and the projects' Node requirements; the expanded peer range still accepts Hono 4.

@askalf
askalf merged commit d3728d6 into master Oct 29, 2026
21 checks passed
@askalf
askalf deleted the hono-node-server-2.1.3 branch October 29, 2026 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

policy fieldpass policy layer (policy/) size/XS Under 10 hand-written lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants