Adding a header parameters to the client constructor

Author: alallema

meilisearch/_httprequests.py

@@ -19,7 +19,8 @@ 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,11 @@ 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 +43,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 List, Optional
 
 
 class Config:
@@ -35,7 +35,11 @@ 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 +53,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,20 @@ 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)"
+    )