Skip to content

security: override js-yaml to 4.3.2 in the docs image - #1292

Merged
michalharakal merged 1 commit into
developfrom
security/docs-image-js-yaml
Sep 20, 2026
Merged

michalharakal merged 1 commit into
developfrom
security/docs-image-js-yaml

Conversation

@michalharakal

Copy link
Copy Markdown
Contributor

Follow-up to #1291. Committing docs/.docker/package-lock.json made the docs image's npm graph visible to Dependabot for the first time, which surfaced js-yaml 4.1.1 (pulled by @antora/content-aggregator, @antora/playbook-builder, @antora/ui-loader, cosmiconfig) — alerts #130, #131, #132, #135. Same package versions were in the image before; they were just not tracked.

Fix: npm overridesjs-yaml: 4.3.2 in docs/.docker/package.json; lockfile regenerated inside the pinned node:20-alpine digest (npm install --package-lock-only), only the js-yaml entry changes.

Verified: docker build --no-cache green; the real site builds with the image (Antora parses its YAML playbook through js-yaml): exit 0, 0 errors, 72 pages, 10 with rendered diagrams — same as before.

Not addressed here: extract-zip 2.0.1 (#133, #134) has no patched release; it is only reachable through @puppeteer/browsers' browser download, which the image disables (PUPPETEER_SKIP_DOWNLOAD=true, Alpine Chromium is used instead).

Committing the docs image lockfile made its npm graph visible to Dependabot: Antora and
cosmiconfig resolve js-yaml 4.1.1 (GHSA-2883-xcg3-v3hh and three earlier quadratic-CPU
advisories). An npm override pins 4.3.2; only that lockfile entry changes. Image rebuilt
from scratch and the docs site renders unchanged.
@github-actions

Copy link
Copy Markdown

📖 Documentation Preview

The documentation has been built successfully for this PR.

Generated Files:

  • Operator documentation: docs/modules/operators/_generated_/
  • JSON schema output: operators.json

Artifacts:

  • Download the documentation-preview-1292 artifact to view the complete documentation locally.

This comment will be updated automatically when the PR is updated.

@michalharakal
michalharakal merged commit 88af4a4 into develop Sep 20, 2026
18 checks passed
@michalharakal
michalharakal deleted the security/docs-image-js-yaml branch September 20, 2026 16:28
@michalharakal michalharakal mentioned this pull request Sep 20, 2026
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.

1 participant