Skip to content

AST-172563 Use Docker Hub OIDC federation in release pipeline - #1546

Merged
cx-lior-poterman merged 1 commit into
mainfrom
ast-172563-add-docker-oidc-to-release-pipeline
Aug 26, 2026
Merged

AST-172563 Use Docker Hub OIDC federation in release pipeline#1546
cx-lior-poterman merged 1 commit into
mainfrom
ast-172563-add-docker-oidc-to-release-pipeline

Conversation

@cx-lior-poterman

@cx-lior-poterman cx-lior-poterman commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Login to Docker Hub in the release workflow used a static username/password pair, via docker-login-action v4.1.0 which predates OIDC support.

The release workflow's Docker Hub login step used static
username/password credentials via step-security/docker-login-action
v4.1.0, which predates OIDC support. Bump the action to v4.5.1 and
switch to OIDC federation using vars.DOCKERHUB_USERNAME and
secrets.DOCKERHUB_OIDC_CONNECTIONID, which are already configured on
the repo's release GitHub Environment. The job already declares
environment: release and id-token: write, so no permissions changes
are needed. This removes the last static Docker Hub credential from
the release pipeline.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@cx-lior-poterman
cx-lior-poterman requested a review from a team August 24, 2026 12:25
@cx-sumit-morchhale
cx-sumit-morchhale self-requested a review August 25, 2026 07:15

@cx-sumit-morchhale cx-sumit-morchhale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@cx-lior-poterman
cx-lior-poterman merged commit e718c78 into main Aug 26, 2026
18 of 26 checks passed
cx-lior-poterman added a commit that referenced this pull request Aug 26, 2026
…rets (#1549)

The Secrets Manager step used the secret's ARN as both the secret-id and
implicit alias, which produced env vars like
CHECKMARX_PUBLIC_SECRETS_APPLE_DEVELOPER_CERTIFICATE_P12_BASE64 instead of
the un-prefixed names the rest of the workflow expects, breaking Apple
code-signing cert import. Use a blank alias (leading comma) so env var
names match the secret's JSON keys directly.

Also removes the now-unused DOCKER_PASSWORD/DOCKER_USERNAME workflow_call
secrets (Docker Hub now uses OIDC per #1546) and PERSONAL_ACCESS_TOKEN /
GO_BOT_TOKEN, whose only consumer (the Homebrew tap publisher) is
commented out in .goreleaser.yml.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

4 participants