Skip to content

Support additional session request headers - #69

Merged
bc-anam merged 1 commit into
mainfrom
codex/http-request-headers
Aug 5, 2026
Merged

Support additional session request headers#69
bc-anam merged 1 commit into
mainfrom
codex/http-request-headers

Conversation

@bc-anam

@bc-anam bc-anam commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add optional ClientOptions.request_headers for preview/proxy HTTP headers
  • ensure SDK-owned authorization and content-type headers cannot be overridden
  • cover defaults and header precedence in tests

Verification

  • uv run ruff format --check .
  • uv run ruff check .
  • uv run pytest -q (71 passed)

uv run mypy src still reports the existing aiortc/av typing errors also present on main.


Summary by cubic

Adds support for extra HTTP headers on session start via ClientOptions.request_headers. SDK-owned Authorization and Content-Type remain enforced and cannot be overridden.

  • New Features

    • Merges request_headers into session request headers while preserving SDK values for Authorization and Content-Type.
    • Tests cover default None and header precedence (e.g., x-vercel-protection-bypass).
  • Dependencies

    • Bump anam in uv.lock to 0.9.0a2.

Written for commit 6514741. Summary will update on new commits.

Review in cubic

@bc-anam bc-anam added the low-risk Low risk changes that can be auto-approved label Aug 5, 2026

@anam-push-bot anam-push-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Auto-approved with label(s): low-risk

@bc-anam
bc-anam merged commit a6d5947 into main Aug 5, 2026
9 of 10 checks passed
@bc-anam
bc-anam deleted the codex/http-request-headers branch August 5, 2026 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

low-risk Low risk changes that can be auto-approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant