Skip to content

Bump protobufjs from 7.2.6 to 7.5.5#222

Open
poorvapotnis wants to merge 1 commit intomainfrom
bump-protobufjs-7.5.5
Open

Bump protobufjs from 7.2.6 to 7.5.5#222
poorvapotnis wants to merge 1 commit intomainfrom
bump-protobufjs-7.5.5

Conversation

@poorvapotnis
Copy link
Copy Markdown

Why

Fixes Dependabot alert #134 — arbitrary code execution in protobufjs < 7.5.5 (GHSA advisory — attackers who can control protobuf definitions can inject arbitrary JS in the type field that executes during decode).

Originating Slack thread: https://replit.slack.com/archives/D0AH702HZHQ/p1776388262869839

protobufjs is pulled in transitively via @replit/protocol@0.3.16 → protobufjs@^7.2.4, which was resolving to 7.2.6 in the lockfile.

What changed

  • yarn.lock: bumped protobufjs from 7.2.67.5.5. No changes to package.json — the existing ^7.2.4 constraint in @replit/protocol already permits this version, we just re-resolved the lockfile entry. 3 lines changed.

Test plan

  • yarn build (tsc) — passes.
  • yarn lint — passes.
  • yarn test targeted (sortByPriority) — passes. (Full yarn test hangs on pre-existing open handles unrelated to this change; CI will exercise the full suite.)

Revertibility

Safe to revert. Lockfile-only change; reverting restores the previous protobufjs@7.2.6 resolution.

~ written by Zerg 👾 (ravenous-scout-d71e)

Fixes Dependabot alert #134 (arbitrary code execution in protobufjs).
protobufjs is pulled in transitively via @replit/protocol and
versions <7.5.5 allow attackers who can control protobuf definitions
to execute arbitrary JS during object decoding.

The @replit/protocol package declares protobufjs "^7.2.5" so no
downstream version constraints change; only the lockfile is updated.
@poorvapotnis poorvapotnis added the zergling-authored Authored by a Zergling label Apr 17, 2026
@poorvapotnis poorvapotnis requested a review from ryantm April 17, 2026 01:51
@poorvapotnis poorvapotnis marked this pull request as ready for review April 17, 2026 01:52
@poorvapotnis poorvapotnis requested review from jackyzha0 and removed request for ryantm April 17, 2026 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

zergling-authored Authored by a Zergling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants