Skip to content

Fix critical vulnerabilities from daily dependency scan#21402

Merged
geropl merged 2 commits intomainfrom
gero/clc-2242-fix-critical-vulnerabilities-from-daily-dependency-scan
Apr 17, 2026
Merged

Fix critical vulnerabilities from daily dependency scan#21402
geropl merged 2 commits intomainfrom
gero/clc-2242-fix-critical-vulnerabilities-from-daily-dependency-scan

Conversation

@geropl
Copy link
Copy Markdown
Member

@geropl geropl commented Apr 17, 2026

Description

Fix critical vulnerabilities detected by the daily dependency scan on main.

  • handlebars 4.7.7 → 4.7.9 (CVE-2026-33937, JS injection via AST type confusion): Added yarn resolution since upstream grpc_tools_node_protoc_ts pins 4.7.7 with no fixed release.
  • protobufjs 7.4.0 → 7.5.5 (arbitrary code execution): Lockfile-only bump — existing semver ranges in @grpc/proto-loader and ts-proto already accept 7.5.5.

The typeorm SQL injection (CVE-2022-33171) is a known issue already in the exclusion list.

Related Issue(s)

Fixes CLC-2242

How to test

yarn audit --level critical should only report the excluded typeorm vulnerabilities.

geropl and others added 2 commits April 17, 2026 06:44
grpc_tools_node_protoc_ts pins handlebars 4.7.7 and has no fixed
release. Use yarn resolutions to force 4.7.9 which patches the
JavaScript injection via AST type confusion vulnerability.

Co-authored-by: Ona <no-reply@ona.com>
Lockfile-only change. Both @grpc/proto-loader (^7.2.5) and ts-proto
(^7.2.4) already accept 7.5.5 via semver, so no package.json or
resolution changes needed.

Co-authored-by: Ona <no-reply@ona.com>
@geropl geropl marked this pull request as ready for review April 17, 2026 06:55
@geropl geropl enabled auto-merge (squash) April 17, 2026 06:55
@geropl geropl merged commit 3d124ef into main Apr 17, 2026
14 checks passed
@geropl geropl deleted the gero/clc-2242-fix-critical-vulnerabilities-from-daily-dependency-scan branch April 17, 2026 07:42
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