Skip to content

fix: Use OPENAI_BASE_URL parameter in OpenAI service#3

Open
oobug wants to merge 1 commit into
tjtanjin:mainfrom
oobug:fix/issue1
Open

fix: Use OPENAI_BASE_URL parameter in OpenAI service#3
oobug wants to merge 1 commit into
tjtanjin:mainfrom
oobug:fix/issue1

Conversation

@oobug
Copy link
Copy Markdown

@oobug oobug commented Aug 5, 2025

Description

Use the configured OPENAI_BASE_URL parameter instead of the hard-coded OpenAI API URL.

Closes #1

What change does this PR introduce?

Please select the relevant option(s).

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (changes to docs/code comments)

What is the proposed approach?

Use the configuration setting in services/openai.ts.

Checklist:

  • The commit message follows our adopted guidelines
  • Testing has been done for the change(s) added (for bug fixes/features)
  • Relevant comments/docs have been added/updated (for bug fixes/features)

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.

[Bug] OpenAI Base URL from environment variable is not used in the proxy

1 participant