Repository navigation
fix(realtime): validate beta WebSocket authentication profiles - #4218
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
jbeckwith-oai
left a comment
There was a problem hiding this comment.
Reviewed exact commit 2669ef6. The sync/async beta preflight rejects unsupported implicit provider credentials before connection setup, keeps X.509 blocked, and preserves caller-owned authorization and case-insensitive omission/override behavior. Coordinator and independent static review found no actionable issues. Hosted checks are successful or explicitly skipped. No candidate code or tests executed.
Automated Release PR --- ## [3.42.1](openai/openai-python@v3.42.0...v3.42.1) (2026-11-01) ### Bug Fixes * **api:** preserve Responses WebSocket connection behavior ([openai#4221](openai#4221)) ([38075d8](openai@38075d8)) * **realtime:** validate beta WebSocket authentication profiles ([openai#4218](openai#4218)) ([3622a97](openai@3622a97)) ### Refactors * **websocket:** add SDK-owned Responses policy helpers ([openai#4219](openai#4219)) ([83ed269](openai@83ed269)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: openai-sdks[bot] <284451331+openai-sdks[bot]@users.noreply.github.com>
A bare workload-identity or Bedrock client could attempt a beta Realtime handshake without applying its configured authentication. Reject those unsupported profiles locally before credential refresh or connection, and enforce the documented X.509 WebSocket exclusion.
Preserve explicit
Authorizationsettings for other clients, including default and connection headers, case-insensitive overrides, and removal withomit. Document this compatibility boundary without adding provider token exchange or signing.Validation: