Simplify registry authentication logic and update comments #11

Merged
coissac merged 1 commits from push-wzvpxqqoyvkr into master 2026-05-04 13:18:46 +00:00
+3 -5
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}"