Skip to content

Pin container base images - #181

Closed
afoote-mitre wants to merge 4 commits into
devfrom
af/runtime-image-hardening
Closed

afoote-mitre wants to merge 4 commits into
devfrom
af/runtime-image-hardening

Conversation

@afoote-mitre

@afoote-mitre afoote-mitre commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Pin the Node dependency-build and Distroless runtime images so rebuilding an unchanged commit cannot silently select a different base image. Document the deliberate refresh process required to receive future security updates.

Changes and Rationale

Change Primary files Why it is needed
Pin builder and runtime images to multi-platform SHA-256 manifest digests Dockerfile, Dockerfile.local Mutable tags can produce different image contents from identical source. Digest pins make base-image selection reproducible while retaining architecture selection, nonroot execution, and runtime-only dependencies. The default AWS build remains free of secret-mount syntax; the local build retains its npm CA secret.
Require digest pins in the existing packaging assertions config/runtime-config.test.js Detect accidental removal of image pins while preserving the existing runtime hardening checks.
Document how to refresh both Dockerfiles and interpret scan results README.md Digest pins do not automatically receive patches. Maintainers need to update the pins deliberately, rebuild, and assess native packages against vendor advisories.

@afoote-mitre afoote-mitre changed the title Pin container base images and document remaining vulnerability status Pin container base images Oct 13, 2026
@afoote-mitre
afoote-mitre marked this pull request as draft October 13, 2026 13:51
@afoote-mitre

Copy link
Copy Markdown
Collaborator Author

Superseded by #193, which pins verified Node.js 24.22.1 builder/runtime images, aligns CI, adds synchronized-input assertions, and updates refresh guidance. This draft's sole unique image-pinning commit is covered by the replacement PR; the earlier dependency-removal changes are already on dev.

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