Skip to content

fix(actions): pass digest environment settings by name - #1107

Merged
Pigbibi merged 1 commit into
mainfrom
fix/digest-environment-config
Nov 10, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
fix/digest-environment-config

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

AI assistance

Codex prepared and validated the change under the repository owner's instruction.

What changed

Pass the seven protected environment configuration names to the fixed Blob adapter and pin its updated commit.

How verified

Actionlint and git diff checks passed. Real dispatches on the previous version confirmed the Azure login received no configured identifiers, before the Blob operation could run. After merge the production read-only and isolated deploy/rollback runs will verify the corrected configuration.

Why

GitHub requires named environment settings to be passed to reusable workflows. This sends only the adapter contract, leaving all unrelated repository secrets outside the call.

Risk

R2, manual release workflow only. No schedule, runtime code, live package or notification behavior is changed.

Co-Authored-By: Codex <noreply@openai.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-10T06:52:39.306640Z ae4ad25 PR opened
🔒 Security Review ✅ Completed 2026-11-10T06:55:32.534278Z ae4ad25 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Pigbibi Pigbibi added the agent-approved An agent cleared this high-risk change on the current head; not an owner confirmation label Nov 10, 2026
@Pigbibi
Pigbibi merged commit 4a5636a into main Nov 10, 2026
19 of 20 checks passed
@Pigbibi
Pigbibi deleted the fix/digest-environment-config branch November 10, 2026 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-approved An agent cleared this high-risk change on the current head; not an owner confirmation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant