Skip to content

Lowercase ghcr owner for docker image name#58

Merged
xakep666 merged 1 commit into
xakep666:masterfrom
CatMe0w:fix-docker-image-casing
Jun 15, 2026
Merged

Lowercase ghcr owner for docker image name#58
xakep666 merged 1 commit into
xakep666:masterfrom
CatMe0w:fix-docker-image-casing

Conversation

@CatMe0w

@CatMe0w CatMe0w commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

This is not a problem for the upstream repo, since the repo owner name is already in all lowercase. But in my fork it causes Docker images to fail to publish.

  • publishing
    • docker images (v2)
      • creating images                              id=ps3netsrv-go
        images=
        │ ghcr.io/CatMe0w/ps3netsrv-go:latest
        │ ghcr.io/CatMe0w/ps3netsrv-go:v0.4
        │ ghcr.io/CatMe0w/ps3netsrv-go:v0.4.1-fork.1
  ⨯ release failed after 1m43s                       error=docker images (v2): failed to publish artifacts: exit status 1 message=could not build docker image args=docker buildx build --platform linux/amd64,linux/arm64 -t ghcr.io/CatMe0w/ps3netsrv-go:latest -t ghcr.io/CatMe0w/ps3netsrv-go:v0.4 -t ghcr.io/CatMe0w/ps3netsrv-go:v0.4.1-fork.1 --push --attest=type=sbom --iidfile=id.txt --label org.opencontainers.image.created=2026-06-15T06:51:57Z --label org.opencontainers.image.licenses=MIT --label org.opencontainers.image.revision=c40458918f1ab33fa586c5e889c1d3444406f188 --label org.opencontainers.image.source=https://github.com/CatMe0w/ps3netsrv-go --label org.opencontainers.image.title=ps3netsrv-go --label org.opencontainers.image.version=v0.4.1-fork.1 . id=ps3netsrv-go
    output=
    │ ERROR: failed to build: invalid tag "ghcr.io/CatMe0w/ps3netsrv-go:latest": repository name must be lowercase
Error: The process '/opt/hostedtoolcache/goreleaser-action/2.16.0/x64/goreleaser' failed with exit code 1

Add tolower as an explicit guard.

@xakep666
xakep666 merged commit 3106d5f into xakep666:master Jun 15, 2026
7 checks passed
@CatMe0w
CatMe0w deleted the fix-docker-image-casing branch June 23, 2026 03:21
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