Skip to content

UID2-7851/UID2-7852: Suppress js-yaml and svgo CVEs - #212

Merged
mcollins-ttd merged 1 commit into
mainfrom
mkc-UID2-7851-7852-jsyaml-svgo
Sep 11, 2026
Merged

UID2-7851/UID2-7852: Suppress js-yaml and svgo CVEs#212
mcollins-ttd merged 1 commit into
mainfrom
mkc-UID2-7851-7852-jsyaml-svgo

Conversation

@mcollins-ttd

Copy link
Copy Markdown
Contributor

Summary

Suppresses two trivy findings via .trivyignore, affecting both React demo apps:

  • web-integrations/google-secure-signals/react-client-side/package-lock.json
  • web-integrations/javascript-sdk/react-client-side/package-lock.json

CVE-2026-84375 — js-yaml (HIGH, installed 3.15.0/4.3.0, fixed 3.15.2/4.3.2)

js-yaml is a transitive dependency of build-tooling only (eslint, react-scripts, test-coverage) in these demo apps. It is not used by served application code.

Jira: https://thetradedesk.atlassian.net/browse/UID2-7851

CVE-2026-84370 — svgo (HIGH, installed 1.3.2/2.8.1/3.3.3, fixed 2.8.4/3.3.5/4.1.0)

svgo runs only at build time via @svgr/plugin-svgo, postcss-svgo, and Create React App's bundled @svgr/webpack, optimizing repo-owned SVGs. Each demo app's Dockerfile runs npm run build and then serves the static build output with a plain Express server — no upload handling, no runtime SVG sanitization of user content.

Jira: https://thetradedesk.atlassian.net/browse/UID2-7852

Test plan

  • .trivyignore is the only file changed — no package.json/package-lock.json edits
  • New entries follow the existing comment/format style, with exp: review dates

… runtime

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@mcollins-ttd
mcollins-ttd force-pushed the mkc-UID2-7851-7852-jsyaml-svgo branch from a181b56 to 35fed67 Compare September 10, 2026 05:57
@mcollins-ttd
mcollins-ttd merged commit 18b335c into main Sep 11, 2026
3 checks passed
@mcollins-ttd
mcollins-ttd deleted the mkc-UID2-7851-7852-jsyaml-svgo branch September 11, 2026 05:24
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