fix(api): implement verify_api_key utility for API key validation

Author: Programmer-RD-AI

Diff for: src/openai/_client.py

@@ -19,11 +19,7 @@
     ProxiesTypes,
     RequestOptions,
 )
-from ._utils import (
-    is_given,
-    is_mapping,
-    get_async_library,
-)
+from ._utils import is_given, is_mapping, get_async_library, verify_api_key
 from ._version import __version__
 from .resources import files, images, models, batches, embeddings, completions, moderations
 from ._streaming import Stream as Stream, AsyncStream as AsyncStream
@@ -104,8 +100,8 @@ def __init__(
         - `organization` from `OPENAI_ORG_ID`
         - `project` from `OPENAI_PROJECT_ID`
         """
-        if api_key is None:
-            api_key = os.environ.get("OPENAI_API_KEY")
+        api_key = verify_api_key(api_key, "OPENAI_API_KEY")
+
         if api_key is None:
             raise OpenAIError(
                 "The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable"
@@ -331,8 +327,7 @@ def __init__(
         - `organization` from `OPENAI_ORG_ID`
         - `project` from `OPENAI_PROJECT_ID`
         """
-        if api_key is None:
-            api_key = os.environ.get("OPENAI_API_KEY")
+        api_key = api_key = verify_api_key(api_key, "OPENAI_API_KEY")
         if api_key is None:
             raise OpenAIError(
                 "The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable"

Diff for: src/openai/_utils/__init__.py

@@ -34,6 +34,7 @@
     maybe_coerce_boolean as maybe_coerce_boolean,
     maybe_coerce_integer as maybe_coerce_integer,
     is_async_azure_client as is_async_azure_client,
+    verify_api_key as verify_api_key,
 )
 from ._typing import (
     is_list_type as is_list_type,

Diff for: src/openai/_utils/_utils.py

@@ -428,3 +428,10 @@ def is_async_azure_client(client: object) -> TypeGuard[AsyncAzureOpenAI]:
     from ..lib.azure import AsyncAzureOpenAI
 
     return isinstance(client, AsyncAzureOpenAI)
+
+
+def verify_api_key(api_key: str, env_name: str) -> str | None:
+    if api_key is None or api_key.strip() == "":
+        env_api_key = os.environ.get(env_name)
+        api_key = env_api_key if env_api_key and env_api_key.strip() != "" else None
+    return api_key

Diff for: src/openai/lib/azure.py

@@ -8,7 +8,7 @@
 import httpx
 
 from .._types import NOT_GIVEN, Omit, Query, Timeout, NotGiven
-from .._utils import is_given, is_mapping
+from .._utils import is_given, is_mapping, verify_api_key
 from .._client import OpenAI, AsyncOpenAI
 from .._compat import model_copy
 from .._models import FinalRequestOptions
@@ -163,8 +163,7 @@ def __init__(
             azure_deployment: A model deployment, if given sets the base client URL to include `/deployments/{azure_deployment}`.
                 Note: this means you won't be able to use non-deployment endpoints. Not supported with Assistants APIs.
         """
-        if api_key is None:
-            api_key = os.environ.get("AZURE_OPENAI_API_KEY")
+        api_key = verify_api_key(api_key, "AZURE_OPENAI_API_KEY")
 
         if azure_ad_token is None:
             azure_ad_token = os.environ.get("AZURE_OPENAI_AD_TOKEN")
@@ -425,8 +424,7 @@ def __init__(
             azure_deployment: A model deployment, if given sets the base client URL to include `/deployments/{azure_deployment}`.
                 Note: this means you won't be able to use non-deployment endpoints. Not supported with Assistants APIs.
         """
-        if api_key is None:
-            api_key = os.environ.get("AZURE_OPENAI_API_KEY")
+        api_key = verify_api_key(api_key, "AZURE_OPENAI_API_KEY")
 
         if azure_ad_token is None:
             azure_ad_token = os.environ.get("AZURE_OPENAI_AD_TOKEN")