Skip to content

Fix Codex OpenAI API key proxy transform#730

Open
gakonst wants to merge 1 commit into
mainfrom
centaur/fix-codex-placeholder-auth-1782261446
Open

Fix Codex OpenAI API key proxy transform#730
gakonst wants to merge 1 commit into
mainfrom
centaur/fix-codex-placeholder-auth-1782261446

Conversation

@gakonst

@gakonst gakonst commented Jun 24, 2026

Copy link
Copy Markdown
Member

Summary

  • change Codex OpenAI API-key proxy auth from header injection to placeholder replacement on Authorization
  • assert Codex API-key auth contributes the OPENAI_API_KEY placeholder env

Evidence

  • Live failure: sandbox pod logged repeated 401s to wss://api.openai.com/v1/responses while OPENAI_API_KEY was set to the literal placeholder
  • Proxy auth logs showed no separate proxy-side secret-resolution failure

Tests

  • cargo test -p centaur-iron-proxy harness_auth_fragments_are_baked_in
  • cargo test -p centaur-api-server codex_app_server_env_template_injects_auth_mode_and_placeholder
  • cargo fmt --check

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