Skip to content

[DOC]: Pin docker image with hash - #846

Merged
MacOS merged 1 commit into
SKaiNET-developers:developfrom
MacOS:docker-image/pin-image-with-hash
Jul 21, 2026
Merged

MacOS merged 1 commit into
SKaiNET-developers:developfrom
MacOS:docker-image/pin-image-with-hash

Conversation

@MacOS

@MacOS MacOS commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

This PR pins the docker image used in docs with a hash to increase security.

The OpenSSF Score should increase after this PR has been merged.

The PR is linked to #594 and #814 and #815. I did not open an issue for this PR because the change is a little to small, and #815 is similar in spirit as it pins actions with the hash.

@MacOS
MacOS requested review from Copilot and michalharakal July 20, 2026 13:19
@MacOS MacOS self-assigned this Jul 20, 2026
@MacOS MacOS added documentation Improvements or additions to documentation (DARC: D) enhancement New feature or request labels Jul 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR increases supply-chain security for the documentation build container by pinning the node:20-alpine base image to an immutable SHA256 digest in docs/.docker/Dockerfile, aligning with the broader OpenSSF hardening work referenced in the PR description.

Changes:

  • Pin node:20-alpine base image to a specific sha256 digest in the docs Antora Dockerfile.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@michalharakal michalharakal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tnank you @MacOS

Note: the failing preview-documentation check is not related to this change — it's a known limitation where fork PRs get a read-only GITHUB_TOKEN, so the "post preview comment" step can't run. Docs build fine and the artifact uploads correctly. Safe to ignore here; a fix to skip that step on fork PRs is coming separately.

@MacOS
MacOS merged commit e58b8d0 into SKaiNET-developers:develop Jul 21, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation (DARC: D) enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants