2 Commits

Author SHA1 Message Date
coissac 6a2f207d8d Merge pull request 'Simplify registry authentication logic and update comments' (#11) from push-wzvpxqqoyvkr into master
Reviewed-on: #11
2026-05-04 13:18:45 +00:00
Eric Coissac b79bb06130 Simplify registry authentication logic and update comments
Updates the script comment to reference skopeo and clarifies the pre-build authentication check. Removes the conditional docker login wrapper and intermediate prompts, now executing docker login directly. The script will fail fast with a clear error message if authentication fails.
2026-05-04 15:17:47 +02: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}"