Skip to content

Pin the base image to a digest#99

Merged
larsoner merged 2 commits into
codespell-project:masterfrom
LotemKahana:pin-base-image-digest
Jul 4, 2026
Merged

Pin the base image to a digest#99
larsoner merged 2 commits into
codespell-project:masterfrom
LotemKahana:pin-base-image-digest

Conversation

@LotemKahana

Copy link
Copy Markdown
Contributor

python:3.13-alpine is a mutable tag that gets updated whenever upstream ships Alpine/Python patches, so a build today could pull different content than a build tomorrow with no change to this file. Pin to the digest instead.

python:3.13-alpine is a mutable tag that gets updated whenever
upstream ships Alpine/Python patches, so a build today could pull
different content than a build tomorrow with no change to this file.
Pin to the digest instead.
@larsoner

larsoner commented Jul 1, 2026

Copy link
Copy Markdown
Member

This one I'm not sure about... what's the advantage here? I think the risk of using their incrementally updated version is sufficiently low it's not worth the maintenance overhead of us having to update this periodically

@LotemKahana

Copy link
Copy Markdown
Contributor Author

This is a security best practice against supply chain attacks, docker docs
To reduce maintenance dependabot has the capability to bump the digest - example pr

@larsoner

larsoner commented Jul 2, 2026

Copy link
Copy Markdown
Member

Can you adjust the dependabot config as part of this PR?

Requested in review: since the Dockerfile now pins the base image by
digest, dependabot needs a docker-ecosystem entry to keep that digest
current.
@LotemKahana

Copy link
Copy Markdown
Contributor Author

Added a weekly update. not so sure if it should be weekly or daily, happy to change if you think daily is better.
my main reasoning again is in case of a supply chain attack having it running weekly is the closest thing i found to delaying version upgrades

@larsoner larsoner merged commit 0194c8f into codespell-project:master Jul 4, 2026
6 checks passed
@larsoner

larsoner commented Jul 4, 2026

Copy link
Copy Markdown
Member

Thanks!

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