Skip to content
This repository was archived by the owner on Aug 7, 2026. It is now read-only.

ci: switch dockerhub login to OIDC - #13

Open
ingwarsw wants to merge 1 commit into
masterfrom
ingwar/docker-oidc-login
Open

ci: switch dockerhub login to OIDC#13
ingwarsw wants to merge 1 commit into
masterfrom
ingwar/docker-oidc-login

Conversation

@ingwarsw

@ingwarsw ingwarsw commented Aug 7, 2026

Copy link
Copy Markdown

Replaces DOCKER_HUB_USERNAME/DOCKER_HUB_PASSWORD secrets in the docker job with the docker/oidc-action + docker/login-action OIDC flow, matching the pattern rolled out across other repos (see kentik/ksynth#14, kentik/kagent#1004).

  • Added permissions: contents: read, id-token: write to the docker job.
  • Replaced docker/login-action@v1 using DOCKER_HUB_USERNAME/DOCKER_HUB_PASSWORD with docker/oidc-action + docker/login-action using the kentik OIDC connection.

Copilot AI lite review requested due to automatic review settings August 7, 2026 14:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Docker image publish workflow to authenticate to Docker Hub using GitHub OIDC instead of long-lived Docker Hub username/password secrets.

Changes:

  • Adds id-token: write (and explicit contents: read) permissions to the docker job for OIDC.
  • Replaces docker/login-action secret-based auth with docker/oidc-action token exchange + docker/login-action using the OIDC token.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +35 to +37
permissions:
contents: read
id-token: write
Comment on lines 47 to +49
with:
username: ${{ secrets.DOCKER_HUB_USERNAME }}
password: ${{ secrets.DOCKER_HUB_PASSWORD }}
connection-id: "24b78b4d-ffde-4850-a999-76d7bbaf3214"
expires-in: 1800
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants