Skip to content

fix: authenticate local helper and protocol proxy#1416

Open
superman2003 wants to merge 1 commit into
BigPizzaV3:mainfrom
superman2003:codex/fix-local-helper-auth
Open

fix: authenticate local helper and protocol proxy#1416
superman2003 wants to merge 1 commit into
BigPizzaV3:mainfrom
superman2003:codex/fix-local-helper-auth

Conversation

@superman2003

Copy link
Copy Markdown

Summary

  • require the local protocol proxy to receive the active relay bearer token before forwarding requests
  • protect renderer-only helper routes with a per-process token injected into Codex
  • remove wildcard CORS from protocol responses and reject proxy preflights

Test plan

  • cargo fmt --all -- --check
  • focused launcher/helper authorization tests
  • cargo check -p codex-plus-core -j 1
  • node --check assets/inject/renderer-inject.js

Environment note

The full launcher integration target compiled and reached the new tests, but the Windows test process later aborted with a host memory-allocation failure. All focused security regressions passed serially.

Protect relay credentials and renderer-only endpoints from cross-origin callers while preserving authenticated desktop fallbacks.

Co-authored-by: Cursor <cursoragent@cursor.com>
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