Skip to content

ci: Add id-token permissions to support WIF - #118

Merged
thced merged 1 commit into
masterfrom
ci/wif-id-token-permissions
Aug 10, 2026
Merged

ci: Add id-token permissions to support WIF#118
thced merged 1 commit into
masterfrom
ci/wif-id-token-permissions

Conversation

@thced

@thced thced commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Grant workflow-level id-token: write in release.yaml and pull_request.yaml so the jobs calling extenda/actions/* can request an OIDC token from GitHub. The release job already re-declared it alongside contents: write; job-level blocks replace rather than merge, so that one stays as-is.

The extenda wrappers pick their auth mode from the contents of the service-account-key value -- either a legacy private_key JSON key or a workload_identity_provider federation config. SECRET_AUTH currently holds a legacy key, so this permission is groundwork only. The migration completes when SECRET_AUTH is swapped to a WIF cred-config on the GCP side, which also needs the WIF principal to hold the Secret Manager roles the current service account has.

Grant workflow-level `id-token: write` in release.yaml and
pull_request.yaml so the jobs calling extenda/actions/* can request an
OIDC token from GitHub. The release job already re-declared it
alongside `contents: write`; job-level blocks replace rather than merge,
so that one stays as-is.

The extenda wrappers pick their auth mode from the contents of the
service-account-key value -- either a legacy `private_key` JSON key or a
`workload_identity_provider` federation config. SECRET_AUTH currently
holds a legacy key, so this permission is groundwork only. The migration
completes when SECRET_AUTH is swapped to a WIF cred-config on the GCP
side, which also needs the WIF principal to hold the Secret Manager
roles the current service account has.

The commonpom 4.0.3+ prerequisite does not apply here: this repo has no
io.extenda.maven:commonpom parent since #86, and the Maven deploy
authenticates to Maven Central with the MAVEN_CENTRAL_* and GPG secrets
rather than to GCP.
@sonarqubecloud

Copy link
Copy Markdown

@thced
thced merged commit 5688e8d into master Aug 10, 2026
5 checks passed
@thced
thced deleted the ci/wif-id-token-permissions branch August 10, 2026 15:11
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.

1 participant