Skip to content

🔒 fix(security): clear Scorecard OSV and pin image digests - #192

Merged
konih merged 1 commit into
mainfrom
chore/sec-scorecard-docs-osv
Aug 12, 2026
Merged

🔒 fix(security): clear Scorecard OSV and pin image digests#192
konih merged 1 commit into
mainfrom
chore/sec-scorecard-docs-osv

Conversation

@konih

@konih konih commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Bump docs lockfiles so OpenSSF Scorecard OSV findings clear: js-yaml 5.2.2 (GHSA-724g-mxrg-4qvm / GHSA-pm4m-ph32-ghv5), click 8.4.2 (PYSEC-2026-2132), pymdown-extensions 11.0.1 (PYSEC-2026-3609 / PYSEC-2026-3654). mkdocs-material 9.7.7 unblocks the click cap.
  • Pin Dockerfile and the metrics-curl fixture image by manifest-list digest; quote TARGETOS/TARGETARCH (docker:S6570).
  • Contract tests: hack/test/sonar_mk_10_docs_osv_test.sh, hack/test/sonar_mk_11_dockerfile_digest_test.sh.
  • Drop Kubebuilder TODO(user) on metrics certs (Sonar go:S1135 / CWE-tagged leftover).

Test plan

  • bash hack/test/sonar_mk_10_docs_osv_test.sh
  • bash hack/test/sonar_mk_11_dockerfile_digest_test.sh
  • osv-scanner on go.mod + docs lockfiles → no issues
  • mkdocs build --strict with the new requirements
  • CI required checks on this PR head

Scorecard flagged five OSV hits in docs lockfiles (js-yaml DoS, click
injection, pymdown path-traversal/ReDoS) plus unpinned container bases.
Bump the docs toolchain to patched releases, pin FROM lines by
manifest-list digest, and quote Dockerfile ARG expansion.
@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sonarqubecloud

Copy link
Copy Markdown

@konih
konih merged commit 6ce4ec4 into main Aug 12, 2026
22 checks passed
@konih
konih deleted the chore/sec-scorecard-docs-osv branch August 12, 2026 22:38
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