Skip to content

custom BASE_PATH #72

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

BeatlesCoder
Copy link

Allow custom BASE_PATH, which are passed in when new OpenAIAPI.

@srcnalt
Copy link
Owner

srcnalt commented Jul 17, 2023

Hi @BeatlesCoder,

Since this package is specifically for OpenAI, I do not see a use case where a custom base URL would be used. Could you elaborate on the need for having this? For special cases where a developer using the same API structure accessed from a custom path could work on a package fork, I assume this is not a common situation.

@srcnalt
Copy link
Owner

srcnalt commented Jul 19, 2023

Hi @BeatlesCoder, could #76 be a use case for this?

@redclock
Copy link

In fact, customizing the base URL is very useful for situations where direct access to the official OpenAI address is not possible or where a proxy is needed, such as in China.

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