Skip to content

Handle SSE streams without DONE from iflow domain#29

Merged
kxn merged 1 commit into
masterfrom
codex/add-hack-for-apis.iflow.cn-sse-check
Sep 11, 2025
Merged

Handle SSE streams without DONE from iflow domain#29
kxn merged 1 commit into
masterfrom
codex/add-hack-for-apis.iflow.cn-sse-check

Conversation

@kxn

@kxn kxn commented Sep 11, 2025

Copy link
Copy Markdown
Owner

Summary

  • pass endpoint URL through validation so SSE checks know request origin
  • ignore missing [DONE] marker for OpenAI-style SSE streams from apis.iflow.cn
  • add tests for the iflow domain exception

Testing

  • go test -v ./... (fails: TestConvertAnthropicRequestToOpenAI_SimpleText: MaxCompletionTokens should not be nil)

https://chatgpt.com/codex/tasks/task_e_68c2dd6c3d38832698f8aa9348ac513e

@kxn
kxn merged commit 1c43d69 into master Sep 11, 2025
5 checks passed
@kxn
kxn deleted the codex/add-hack-for-apis.iflow.cn-sse-check branch September 13, 2025 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant