Adding a header parameters to the client constructor

Author: alallema

meilisearch/_httprequests.py

@@ -19,7 +19,7 @@ def __init__(self, config: Config) -> None:
         self.config = config
         self.headers = {
             "Authorization": f"Bearer {self.config.api_key}",
-            "User-Agent": qualified_version(),
+            "User-Agent": qualified_version() + (";" + ";".join(config.client_agents) if config.client_agents else ""),
         }
 
     def send_request(

meilisearch/client.py

@@ -29,7 +29,7 @@ class Client:
     """
 
     def __init__(
-        self, url: str, api_key: Optional[str] = None, timeout: Optional[int] = None
+        self, url: str, api_key: Optional[str] = None, timeout: Optional[int] = None, client_agents: Optional[List[str]] = None
     ) -> None:
         """
         Parameters
@@ -39,7 +39,7 @@ def __init__(
         api_key:
             The optional API key for Meilisearch
         """
-        self.config = Config(url, api_key, timeout=timeout)
+        self.config = Config(url, api_key, timeout=timeout, client_agents=client_agents)
 
         self.http = HttpRequests(self.config)
 

meilisearch/config.py

@@ -1,6 +1,6 @@
 from __future__ import annotations
 
-from typing import Optional
+from typing import Optional, List
 
 
 class Config:
@@ -35,7 +35,7 @@ class Paths:
         swap = "swap-indexes"
 
     def __init__(
-        self, url: str, api_key: Optional[str] = None, timeout: Optional[int] = None
+        self, url: str, api_key: Optional[str] = None, timeout: Optional[int] = None, client_agents: Optional[List[str]] = None
     ) -> None:
         """
         Parameters
@@ -49,4 +49,5 @@ def __init__(
         self.url = url
         self.api_key = api_key
         self.timeout = timeout
+        self.client_agents = client_agents
         self.paths = self.Paths()

tests/client/test_http_requests.py

@@ -11,3 +11,11 @@ def test_get_headers_from_http_requests_instance():
 
     assert http.headers["Authorization"] == f"Bearer {MASTER_KEY}"
     assert http.headers["User-Agent"] == qualified_version()
+
+def test_get_headers_with_multiple_user_agent():
+    """Tests getting defined headers from instance in HttpRequests."""
+    config = Config(BASE_URL, MASTER_KEY, timeout=None, client_agents=["Meilisearch Package1 (v1.1.1)", "Meilisearch Package2 (v2.2.2)"])
+    http = HttpRequests(config=config)
+
+    assert http.headers["Authorization"] == f"Bearer {MASTER_KEY}"
+    assert http.headers["User-Agent"] == qualified_version() + ";Meilisearch Package1 (v1.1.1);Meilisearch Package2 (v2.2.2)"