Remove version from User-Agent header (#211)

Author: golles

src/tomtom_apis/api.py

@@ -6,7 +6,6 @@
 import socket
 import uuid
 from dataclasses import dataclass, field
-from importlib import metadata
 from types import TracebackType
 from typing import Any, Self
 
@@ -217,14 +216,6 @@ def __init__(
         self.options = options
         self.session = ClientSession(options.base_url, timeout=options.timeout) if session is None else session
         self._close_session = session is None
-        self._version: str | None = None
-
-    @property
-    def version(self) -> str:
-        """Get the version of the TomTom API client."""
-        if self._version is None:
-            self._version = metadata.version(__package__)
-        return self._version
 
     async def _request(  # pylint: disable=too-many-arguments
         self: Self,
@@ -339,7 +330,7 @@ def _prepare_headers(self: Self, headers: dict[str, str] | None, options: ApiOpt
         """
         merged_headers = {
             CONTENT_TYPE: "application/json",
-            USER_AGENT: f"TomTomApiPython/{self.version}",
+            USER_AGENT: "python/tomtom_apis",
             **(headers or {}),
         }
 

tests/test_api.py

@@ -137,7 +137,7 @@ async def test_get_request(base_api: BaseApi, mock_session: AsyncMock) -> None:
         json=None,
         headers={
             "Content-Type": "application/json",
-            "User-Agent": f"TomTomApiPython/{base_api.version}",
+            "User-Agent": "python/tomtom_apis",
         },
     )
     assert isinstance(response, Response)
@@ -157,7 +157,7 @@ async def test_get_request_with_gzip(base_api: BaseApi, mock_session: AsyncMock)
         json=None,
         headers={
             "Content-Type": "application/json",
-            "User-Agent": f"TomTomApiPython/{base_api.version}",
+            "User-Agent": "python/tomtom_apis",
             "Accept-Encoding": "gzip",
         },
     )
@@ -178,7 +178,7 @@ async def test_post_request(base_api: BaseApi, mock_session: AsyncMock) -> None:
         json=data.to_dict(),
         headers={
             "Content-Type": "application/json",
-            "User-Agent": f"TomTomApiPython/{base_api.version}",
+            "User-Agent": "python/tomtom_apis",
         },
     )
     assert isinstance(response, Response)
@@ -197,7 +197,7 @@ async def test_delete_request(base_api: BaseApi, mock_session: AsyncMock) -> Non
         json=None,
         headers={
             "Content-Type": "application/json",
-            "User-Agent": f"TomTomApiPython/{base_api.version}",
+            "User-Agent": "python/tomtom_apis",
         },
     )
     assert isinstance(response, Response)
@@ -217,7 +217,7 @@ async def test_put_request(base_api: BaseApi, mock_session: AsyncMock) -> None:
         json=data.to_dict(),
         headers={
             "Content-Type": "application/json",
-            "User-Agent": f"TomTomApiPython/{base_api.version}",
+            "User-Agent": "python/tomtom_apis",
         },
     )
     assert isinstance(response, Response)