Repository navigation
fix(websocket): apply connection query overrides once - #1283
Draft
markstuart-oai wants to merge 2 commits into
Draft
markstuart-oai wants to merge 2 commits into
markstuart-oai wants to merge 2 commits into
Conversation
Contributor
Castiron custom codeEvaluated main: Trusted main policy: ✅ No new custom-code files detected. 0 mixed files remain; 0 existing customizations changed. Compared A changed generated baseline means this report cannot reliably identify which handwritten lines changed. Inspect the custom-code diffDownload the exact patch produced by this run (requires repository access): gh run download 40464871943 --repo openai/openai-java \
--name castiron-custom-code-40464871943-1 --dir /tmp/castiron-custom-code-40464871943-1
git apply --stat /tmp/castiron-custom-code-40464871943-1/custom-code.patch
cat /tmp/castiron-custom-code-40464871943-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin a565f6e12496d129699f3732d3ead2b9f1d0691e 0e1697a143321ab0ecdee7e8629d5d141407b9b7
python3 scripts/castiron/custom_code_report.py report \
--base a565f6e12496d129699f3732d3ead2b9f1d0691e \
--head 0e1697a143321ab0ecdee7e8629d5d141407b9b7 --target-branch main --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-0e1697a14332
cat /tmp/castiron-custom-code-0e1697a14332/custom-code.patchThis is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Stable Responses and Realtime appended per-connection query parameters after the base URL query. Overrides could reach the server twice. Automatic Azure routing also treated a unified base URL with a query or fragment as a legacy endpoint.
Use the shared WebSocket request builder for both transports. Preserve the path segments and complete query that provider authenticators receive when they sign the handshake. Resolve the Azure API category from the URI path so supported unified endpoints retain their routing and authentication behavior.
Verification
All 648 runnable tests in the OkHttp client suite passed. Two live tests require credentials. The 194 focused tests include stable Responses signing, Realtime lifecycle and Azure compatibility. Twelve Azure probes captured the outgoing requests, and local WebSocket peers verified the signed headers and the encoded query on the wire. The formatter passed.