Skip to content

Commit 977de5f

Browse files
authored
fix(docker-tag): force lowercase for repository and tag inputs (#375)
## Summary - The `docker-tag` composite action compared `repository` and `upstream_repository` case-sensitively, so dispatching prysm with the actual GitHub-cased `OffchainLabs/prysm` did not match the `offchainlabs/prysm` upstream and produced an `OffchainLabs-` prefix (e.g. `OffchainLabs-glamsterdam-devnet-3-minimal`). - Lowercase all four inputs (`input`, `repository`, `upstream_repository`, `docker_tag`) before comparing and emitting the tag. Prefix detection becomes case-insensitive and tags stay fully lowercase across every workflow that uses this action. ## Test plan - [ ] Dispatch `Build prysm docker image` with `repository=OffchainLabs/prysm` and confirm no `offchainlabs-` prefix is added to the resulting tag. - [ ] Dispatch with a fork (e.g. `MarcoPolo/prysm`) and confirm the prefix is still applied as `marcopolo-`. - [ ] Spot-check another client workflow (e.g. lighthouse) to confirm normal upstream dispatches are unchanged.
1 parent 4f3fb56 commit 977de5f

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

.github/actions/docker-tag/action.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,16 +24,16 @@ runs:
2424
id: docker_tag
2525
shell: bash
2626
run: |
27-
docker_tag="${{ inputs.input }}"
27+
docker_tag="$(echo "${{ inputs.input }}" | tr '[:upper:]' '[:lower:]')"
2828
if [ -z "$docker_tag" ]; then
2929
echo "Cannot generate tag for empty input"
3030
exit 1
3131
fi
3232
3333
# Check if repository is provided and is not the upstream repo
34-
repository="${{ inputs.repository }}"
35-
upstream_repo="${{ inputs.upstream_repository }}"
36-
docker_tag_override="${{ inputs.docker_tag }}"
34+
repository="$(echo "${{ inputs.repository }}" | tr '[:upper:]' '[:lower:]')"
35+
upstream_repo="$(echo "${{ inputs.upstream_repository }}" | tr '[:upper:]' '[:lower:]')"
36+
docker_tag_override="$(echo "${{ inputs.docker_tag }}" | tr '[:upper:]' '[:lower:]')"
3737
prefix=""
3838
3939
# If upstream_repo is not provided, we don't add any prefix

0 commit comments

Comments
 (0)