Skip to content

fix(openai): honor realtime base URL env fallback#1650

Merged
chenghao-mou merged 1 commit into
mainfrom
fix/openai-realtime-base-url-env
May 29, 2026
Merged

fix(openai): honor realtime base URL env fallback#1650
chenghao-mou merged 1 commit into
mainfrom
fix/openai-realtime-base-url-env

Conversation

@rosetta-livekit-bot
Copy link
Copy Markdown
Contributor

@rosetta-livekit-bot rosetta-livekit-bot Bot commented May 29, 2026

Summary

The OpenAI realtime model ignored the OPENAI_BASE_URL environment variable. When no base_url was passed (and not using Azure), it fell back to a hardcoded OPENAI_BASE_URL = "https://api.openai.com/v1" constant rather than reading the env var.

This was inconsistent with:

  • The LLM path (llm.py), which forwards base_url=None to openai.AsyncClient, letting the OpenAI SDK read OPENAI_BASE_URL.
  • The STT/TTS path (utils.get_base_url), which falls back to os.getenv("OPENAI_BASE_URL", ...).
  • The realtime constructor's own docstring, which states: "If not provided, uses OPENAI_BASE_URL for OpenAI."

Change

-                base_url_val = OPENAI_BASE_URL
+                base_url_val = os.getenv("OPENAI_BASE_URL", OPENAI_BASE_URL)

The passed base_url still takes precedence when given (is_given(base_url)); the env var is only used as the fallback, matching the other code paths. The hardcoded constant remains the final default.

🤖 Generated with Claude Code

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 29, 2026

🦋 Changeset detected

Latest commit: 43523a9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 33 packages
Name Type
@livekit/agents-plugin-openai Patch
@livekit/agents-plugin-anam Patch
@livekit/agents-plugin-cartesia Patch
@livekit/agents-plugin-cerebras Patch
@livekit/agents-plugin-elevenlabs Patch
@livekit/agents-plugin-fishaudio Patch
@livekit/agents-plugin-google Patch
@livekit/agents-plugin-hume Patch
@livekit/agents-plugin-inworld Patch
@livekit/agents-plugin-neuphonic Patch
@livekit/agents-plugin-perplexity Patch
@livekit/agents-plugin-rime Patch
@livekit/agents-plugin-sarvam Patch
@livekit/agents-plugin-xai Patch
@livekit/agents Patch
@livekit/agents-plugin-assemblyai Patch
@livekit/agents-plugin-baseten Patch
@livekit/agents-plugin-bey Patch
@livekit/agents-plugin-deepgram Patch
@livekit/agents-plugin-hedra Patch
@livekit/agents-plugin-lemonslice Patch
@livekit/agents-plugin-liveavatar Patch
@livekit/agents-plugin-livekit Patch
@livekit/agents-plugin-minimax Patch
@livekit/agents-plugin-mistral Patch
@livekit/agents-plugin-mistralai Patch
@livekit/agents-plugin-phonic Patch
@livekit/agents-plugin-resemble Patch
@livekit/agents-plugin-runway Patch
@livekit/agents-plugin-silero Patch
@livekit/agents-plugin-tavus Patch
@livekit/agents-plugin-trugen Patch
@livekit/agents-plugins-test Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@chenghao-mou chenghao-mou merged commit f71ab2d into main May 29, 2026
9 checks passed
@chenghao-mou chenghao-mou deleted the fix/openai-realtime-base-url-env branch May 29, 2026 20:54
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.

1 participant