Merge pull request 'Simplify registry authentication logic and update comments' (#11) from push-wzvpxqqoyvkr into master

Reviewed-on: #11
This commit was merged in pull request #11.
This commit is contained in:
2026-05-04 13:18:45 +00:00
+8 -10
View File
@@ -266,17 +266,15 @@ publish_images() {
version=$(read_version) version=$(read_version)
# docker buildx --push uses Docker's own credential store, independent of # docker buildx --push uses Docker's own credential store, independent of
# skopeo. Verify auth early to get a clear error before a long build. # skopeo. Prompt once before the (long) build so the user isn't surprised
echo -e "${BLUE}Checking registry authentication...${NC}" # by an auth failure at the very end.
local registry_host="${REGISTRY%%/*}" local registry_host="${REGISTRY%%/*}"
if ! docker login "$registry_host" >/dev/null 2>&1; then echo -e "${BLUE}Authenticating to ${registry_host} (required to push)...${NC}"
echo -e "${YELLOW}Not logged in to ${registry_host}. Running docker login...${NC}" docker login "$registry_host" || {
docker login "$registry_host" || { echo "Error: authentication to ${registry_host} failed." >&2
echo "Error: authentication to ${registry_host} failed." >&2 echo "Run: docker login ${registry_host}" >&2
echo "Run: docker login ${registry_host}" >&2 exit 1
exit 1 }
}
fi
echo "" echo ""
echo -e "${BLUE}Publishing images (version ${version}) to ${REGISTRY}${NC}" echo -e "${BLUE}Publishing images (version ${version}) to ${REGISTRY}${NC}"