Skip to content

Support propagates timeout parameter from request options for OpenAi models#6648

Open
guanxuc wants to merge 1 commit into
spring-projects:mainfrom
guanxuc:openai-runtime-timeout
Open

Support propagates timeout parameter from request options for OpenAi models#6648
guanxuc wants to merge 1 commit into
spring-projects:mainfrom
guanxuc:openai-runtime-timeout

Conversation

@guanxuc

@guanxuc guanxuc commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Description

Support propagates timeout parameter from request options for OpenAi models.

Related issue

Relate to #6604

@guanxuc guanxuc changed the title Support propagates timeout parameter from request options Support propagates timeout parameter from request options for OpenAi models Jul 20, 2026
…i models

Signed-off-by: guanxu <1510424541@qq.com>
@seeun0210

Copy link
Copy Markdown

Thanks for taking this on @guanxuc! 🙌

This is the OpenAI counterpart to the same root cause behind #6604 — in #6605 (the Anthropic fix) I flagged that OpenAiChatModel.internalCall()/internalStream() had the identical issue (calling .create(request)/.createStreaming(request) with no RequestOptions), so it's great to see it covered here.

For reference, #6605 fixes the Anthropic side by passing the resolved timeout through the SDK's two-arg create(params, RequestOptions) overload — might be worth keeping the two approaches consistent so reviewers can look at them together. Nice work! 👍

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.

4 participants