From b4b9df0600450de80b7ef936aa7994cfdc032ab8 Mon Sep 17 00:00:00 2001 From: Phuc Tran Truong Date: Mon, 28 Oct 2024 19:07:05 +0100 Subject: [PATCH 1/4] Update completions.py Removed "/chat/completions" endpoint in order to work with internal endpoint url --- src/openai/resources/chat/completions.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/openai/resources/chat/completions.py b/src/openai/resources/chat/completions.py index ceaf3c2fec..5972a74dda 100644 --- a/src/openai/resources/chat/completions.py +++ b/src/openai/resources/chat/completions.py @@ -813,7 +813,7 @@ def create( ) -> ChatCompletion | Stream[ChatCompletionChunk]: validate_response_format(response_format) return self._post( - "/chat/completions", + "", # Removed "/chat/completions" endpoint in order to work with internal endpoint url body=maybe_transform( { "messages": messages, @@ -1631,7 +1631,7 @@ async def create( ) -> ChatCompletion | AsyncStream[ChatCompletionChunk]: validate_response_format(response_format) return await self._post( - "/chat/completions", + "", # Removed "/chat/completions" endpoint in order to work with internal endpoint url body=await async_maybe_transform( { "messages": messages, From 0d554ae7bd305c435eb7f32f2e5af877fe867ea1 Mon Sep 17 00:00:00 2001 From: Phuc Tran Truong Date: Mon, 28 Oct 2024 22:13:27 +0100 Subject: [PATCH 2/4] Update _base_client.py Disable enforce_trailing_slash --- src/openai/_base_client.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/openai/_base_client.py b/src/openai/_base_client.py index e1d4849ae2..04edf72618 100644 --- a/src/openai/_base_client.py +++ b/src/openai/_base_client.py @@ -354,7 +354,8 @@ def __init__( custom_query: Mapping[str, object] | None = None, ) -> None: self._version = version - self._base_url = self._enforce_trailing_slash(URL(base_url)) + # self._base_url = self._enforce_trailing_slash(URL(base_url)) + self._base_url = URL(base_url) # keep base_url as is self.max_retries = max_retries self.timeout = timeout self._limits = limits From 0508d5d5e6b7407b659664049c788112fce676bb Mon Sep 17 00:00:00 2001 From: Phuc Tran Truong Date: Mon, 28 Oct 2024 23:21:25 +0100 Subject: [PATCH 3/4] Convert messages to string in order to work with BAYER LLM --- src/openai/resources/chat/completions.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/openai/resources/chat/completions.py b/src/openai/resources/chat/completions.py index 5972a74dda..9418c1736b 100644 --- a/src/openai/resources/chat/completions.py +++ b/src/openai/resources/chat/completions.py @@ -8,6 +8,7 @@ import httpx import pydantic +import json from ... import _legacy_response from ..._types import NOT_GIVEN, Body, Query, Headers, NotGiven @@ -816,7 +817,7 @@ def create( "", # Removed "/chat/completions" endpoint in order to work with internal endpoint url body=maybe_transform( { - "messages": messages, + "messages": json.dumps(messages), "model": model, "audio": audio, "frequency_penalty": frequency_penalty, @@ -1634,7 +1635,7 @@ async def create( "", # Removed "/chat/completions" endpoint in order to work with internal endpoint url body=await async_maybe_transform( { - "messages": messages, + "messages": json.dumps(messages), "model": model, "audio": audio, "frequency_penalty": frequency_penalty, From c951665f3e66ec90e00a7985ee67755fcaf86e7a Mon Sep 17 00:00:00 2001 From: Phuc Tran Truong Date: Mon, 28 Oct 2024 23:56:06 +0100 Subject: [PATCH 4/4] Removed "Bearer" prefix from Authorization header --- src/openai/_client.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/openai/_client.py b/src/openai/_client.py index d3ee6cf0f1..ad14a9e2b6 100644 --- a/src/openai/_client.py +++ b/src/openai/_client.py @@ -157,7 +157,7 @@ def qs(self) -> Querystring: @override def auth_headers(self) -> dict[str, str]: api_key = self.api_key - return {"Authorization": f"Bearer {api_key}"} + return {"Authorization": f"{api_key}"} # Removed Bearer from value field @property @override