From 8aecf7a033016f85bf265f182d3b464b6a4dc398 Mon Sep 17 00:00:00 2001
From: theanupllm <theanupdas@protonmail.com>
Date: Wed, 5 Feb 2025 12:08:53 +0530
Subject: [PATCH] fix: Improve Pinecone base URL configuration logic

- Refactor base URL assignment to prioritize environment variable
- Ensure consistent base URL configuration for Pinecone index
- Maintain existing logging behavior for remote API usage
---
 semantic_router/index/pinecone.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/semantic_router/index/pinecone.py b/semantic_router/index/pinecone.py
index 7241328e..663b9814 100644
--- a/semantic_router/index/pinecone.py
+++ b/semantic_router/index/pinecone.py
@@ -153,7 +153,11 @@ class PineconeIndex(BaseIndex):
 
         if base_url is not None or os.getenv("PINECONE_API_BASE_URL"):
             logger.info("Using pinecone remote API.")
-            self.base_url = base_url or os.getenv("PINECONE_API_BASE_URL")
+            if os.getenv("PINECONE_API_BASE_URL"):
+                self.base_url = os.getenv("PINECONE_API_BASE_URL")
+            else:
+                self.base_url = base_url
+
 
         if self.base_url and "api.pinecone.io" in self.base_url:
             self.headers["X-Pinecone-API-Version"] = "2024-07"
-- 
GitLab