Skip to content

update token#173

Merged
kdhillon-stripe merged 1 commit into
mainfrom
update-token-used
Mar 24, 2026
Merged

update token#173
kdhillon-stripe merged 1 commit into
mainfrom
update-token-used

Conversation

@kdhillon-stripe
Copy link
Copy Markdown
Collaborator

Summary

How to test (optional)

Related

  • Closes #

Thanks for contributing ❤️

@kdhillon-stripe kdhillon-stripe marked this pull request as ready for review March 24, 2026 04:12
Copilot AI review requested due to automatic review settings March 24, 2026 04:12
Copy link
Copy Markdown

Copilot AI left a comment

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 GitHub Actions Docker image publishing workflow to use a different Docker Hub token secret for authentication.

Changes:

  • Switch Docker Hub login from secrets.DOCKERHUB_TOKEN to secrets.DOCKERHUB_TOKEN_SDB in the Docker image workflow.

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

with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_TOKEN }}
password: ${{ secrets.DOCKERHUB_TOKEN_SDB }}
Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

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

This workflow now references secrets.DOCKERHUB_TOKEN_SDB. If that secret isn’t configured in the repo/org secrets, the Docker Hub login step will fail at runtime. Consider either (a) adding a short inline comment here documenting the required secret name and purpose, or (b) supporting a temporary fallback to the previous secret name during the transition to avoid breaking builds if the secret hasn’t been created everywhere yet.

Suggested change
password: ${{ secrets.DOCKERHUB_TOKEN_SDB }}
password: ${{ secrets.DOCKERHUB_TOKEN_SDB }}
# Requires DOCKERHUB_TOKEN_SDB secret: Docker Hub access token for pushing stripe/sdb images

Copilot uses AI. Check for mistakes.
@kdhillon-stripe kdhillon-stripe merged commit 0ac523a into main Mar 24, 2026
11 of 12 checks passed
@kdhillon-stripe kdhillon-stripe deleted the update-token-used branch March 24, 2026 04:29
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.

3 participants