Skip to content

chore(docker): move runtime images off end-of-life alpine 3.19 - #571

Open
0xrlawrence wants to merge 1 commit into
canopy-network:mainfrom
0xrlawrence:chore/bump-eol-alpine-base
Open

chore(docker): move runtime images off end-of-life alpine 3.19#571
0xrlawrence wants to merge 1 commit into
canopy-network:mainfrom
0xrlawrence:chore/bump-eol-alpine-base

Conversation

@0xrlawrence

Copy link
Copy Markdown

Description

Both runtime stages pin alpine:3.19, which has passed end of community support and no longer receives security patches, so the shipped image accumulates unfixed CVEs in its base layer.

Changes Made

  • Dockerfile and .docker/Dockerfile: alpine:3.19 -> alpine:3.22.

The runtime layer only installs pigz and ca-certificates, so there is no version-sensitive surface here.

Not changed — needs a maintainer decision

Neither Dockerfile declares a USER, so the node runs as root. That is worth fixing, but it is not a one-line change: .docker/compose.yaml mounts chain data at /root/.canopy, so the data directory would have to move and every volume mapping would have to move with it. I left it out rather than bundling an untested permissions change here — happy to open a follow-up if you want it.

Caveat

I could not build these images locally (no Docker in my environment), so this is an inspection-only change. Worth a CI build before merge.

🤖 Generated with Claude Code

Both runtime stages pin alpine:3.19, which passed end of community support
and no longer receives security patches, so the shipped image accumulates
unfixed CVEs in the base layer. Bumped to alpine:3.22, which is still
supported. The runtime layer only installs pigz and ca-certificates, so
there is no version-sensitive surface here.

Not changed, but worth a maintainer decision: neither Dockerfile declares a
USER, so the node runs as root. Fixing that is not a one-line change,
because .docker/compose.yaml mounts the chain data at /root/.canopy — the
data directory would have to move and every volume mapping would have to
move with it. Left as a follow-up rather than bundled here.

Caveat: I could not build these images locally (no Docker in my
environment), so this is an inspection-only change. Worth a CI build before
merge.
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