Fix OpenAI optional header casing in locale-sensitive environments#12578
Fix OpenAI optional header casing in locale-sensitive environments#12578william-xue wants to merge 1 commit into
Conversation
|
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
|
I have read the CLA Document and I hereby sign the CLA recheck |
4d3cf9c to
5ebdb79
Compare
|
Updated the commit author/committer to my GitHub noreply address so the CLA check can associate the commit with my GitHub account. recheck |
|
@continuedev maintainers: I updated the only PR commit so both author and committer now resolve to my GitHub account Could you please re-run/refresh the CLA assistant or advise if there is another required signature format? |
|
recheck |
Summary
Context
Fixes #12568. In Turkish locales, the OpenAI SDK's normalization of
OpenAI-Organization/OpenAI-Projectcan produce dotless-i header names. Passing lowercase keys withundefinedprevents the optional SDK headers from being injected/normalized for users who have not configured them.Validation
npm test -- src/test/main.test.tsnpm run buildSummary by cubic
Explicitly disable optional OpenAI org/project headers to prevent locale-sensitive casing issues (e.g., Turkish dotless-i) that broke requests when those headers weren’t configured. This stabilizes OpenAI request handling in
openai-adapters.defaultHeaderswith lowercase "openai-organization" and "openai-project" toundefinedinOpenAIApi.Written for commit 5ebdb79. Summary will update on new commits.