Skip to content

fix: authenticate registry before image attestation - #162

Merged
mldangelo-oai merged 1 commit into
mainfrom
mdangelo/codex/fix-container-attestation-login-20260730
Jul 30, 2026
Merged

fix: authenticate registry before image attestation#162
mldangelo-oai merged 1 commit into
mainfrom
mdangelo/codex/fix-container-attestation-login-20260730

Conversation

@mldangelo-oai

Copy link
Copy Markdown
Collaborator

Summary

  • authenticate to GHCR before pushing the verified image provenance attestation
  • preserve the existing protected release, OIDC signing, immutable digest, and stable-tag promotion gates

Evidence

  • production container-release run 30582841027 passed both native publication jobs and multiarchitecture manifest verification, then failed with Error: No credentials found for registry ghcr.io
  • validated workflow YAML and Prettier formatting
  • regression check confirms GHCR authentication executes before provenance signing
  • git diff --check passes

@mldangelo-oai

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 91c5e1f06f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 91c5e1f06f

View security finding report

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@mldangelo-oai
mldangelo-oai merged commit 780c207 into main Jul 30, 2026
14 checks passed
@mldangelo-oai
mldangelo-oai deleted the mdangelo/codex/fix-container-attestation-login-20260730 branch July 30, 2026 21:38
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.

2 participants