Skip to content

Bumped timeout to 30 seconds for openai since calls may take awhile#266

Merged
sfreeman422 merged 2 commits into
masterfrom
feature/bump-timeout
Jun 29, 2026
Merged

Bumped timeout to 30 seconds for openai since calls may take awhile#266
sfreeman422 merged 2 commits into
masterfrom
feature/bump-timeout

Conversation

@sfreeman422

@sfreeman422 sfreeman422 commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

This pull request makes a small configuration change to the OpenAI client settings. The default timeout value for OpenAI API requests has been increased from 10,000ms (10 seconds) to 30,000ms (30 seconds) in the getOpenAIClientConfig function.

  • Increased the default value of timeoutMs in getOpenAIClientConfig from 10,000ms to 30,000ms to allow more time for OpenAI API requests to complete. (packages/backend/src/config/openai.ts)

Copilot AI review requested due to automatic review settings June 29, 2026 15:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the backend’s default resilient OpenAI client configuration to allow longer-running OpenAI calls by increasing the default request timeout.

Changes:

  • Increase the default OPENAI_TIMEOUT_MS fallback from 10,000ms to 30,000ms in getOpenAIClientConfig.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


export const getOpenAIClientConfig = (): OpenAIClientConfig => ({
timeoutMs: Math.max(1, parseIntWithDefault(process.env.OPENAI_TIMEOUT_MS, 10_000)),
timeoutMs: Math.max(1, parseIntWithDefault(process.env.OPENAI_TIMEOUT_MS, 30_000)),
@sfreeman422 sfreeman422 merged commit 1151bc1 into master Jun 29, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants