Requests: Cleanup comments. Move request variables to their appropriate location. Hardcode locale.

Author: tehkillerbee

tidalapi/request.py

@@ -57,6 +57,7 @@ class Requests(object):
     def __init__(self, session: "Session"):
         # More Android User-Agents here: https://user-agents.net/browsers/android
         self.user_agent = "Mozilla/5.0 (Linux; Android 12; wv) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/91.0.4472.114 Safari/537.36"
+        self.client_version = "2025.7.16"
         self.session = session
         self.config = session.config
         self.latest_err_response = requests.Response()
@@ -85,7 +86,8 @@ def basic_request(
         if not headers:
             headers = {}
 
-        headers["x-tidal-client-version"] = "2025.7.16"
+        if "x-tidal-client-version" not in headers:
+            headers["x-tidal-client-version"] = self.client_version
 
         if "User-Agent" not in headers:
             headers["User-Agent"] = self.user_agent
@@ -172,18 +174,22 @@ def request(
         return request
 
     def get_latest_err_response(self) -> dict:
-        """Get the latest request Response that resulted in an Exception :return: The
-        request Response that resulted in the Exception, returned as a dict An empty
-        dict will be returned, if no response was returned."""
+        """Get the latest request Response that resulted in an Exception.
+
+        :return: The request Response that resulted in the Exception, returned as a dict
+            An empty dict will be returned, if no response was returned.
+        """
         if self.latest_err_response.content:
             return self.latest_err_response.json()
         else:
             return {}
 
     def get_latest_err_response_str(self) -> str:
-        """Get the latest request response message as a string :return: The contents of
-        the (detailed) error response Response, returned as a string An empty str will
-        be returned, if no response was returned."""
+        """Get the latest request response message as a string.
+
+        :return: The contents of the (detailed) error response, returned as a string An
+            empty str will be returned, if no response was returned.
+        """
         if self.latest_err_response.content:
             resp = self.latest_err_response.json()
             return resp["errors"][0]["detail"]

tidalapi/session.py

@@ -268,9 +268,10 @@ class Session:
     refresh_token: Optional[str] = None
     #: The type of access token, e.g. Bearer
     token_type: Optional[str] = None
-    #: The id for a TIDAL session, you also need this to use load_oauth_session
+    #: The session id for a TIDAL session, you also need this to use load_oauth_session
     session_id: Optional[str] = None
     country_code: Optional[str] = None
+    locale: Optional[str] = None
     #: A :class:`.User` object containing the currently logged in user.
     user: Optional[Union["FetchedUser", "LoggedInUser", "PlaylistCreator"]] = None
 
@@ -282,15 +283,6 @@ def __init__(self, config: Config = Config()):
         self.request = request.Requests(session=self)
         self.genre = genre.Genre(session=self)
 
-        # self.parse_artists = self.artist().parse_artists
-        # self.parse_playlist = self.playlist().parse
-
-        # self.parse_track = self.track().parse_track
-        # self.parse_video = self.video().parse_video
-        # self.parse_media = self.track().parse_media
-        # self.parse_mix = self.mix().parse
-        # self.parse_v2_mix = self.mixv2().parse
-
         self.parse_user = user.User(self, None).parse
         self.page = page.Page(self, "")
         self.parse_page = self.page.parse
@@ -453,6 +445,7 @@ def load_oauth_session(
 
         self.session_id = json["sessionId"]
         self.country_code = json["countryCode"]
+        self.locale = "en_US"  # TODO Get locale from system configuration
         self.user = user.User(self, user_id=json["userId"]).factory()
 
         return True
@@ -719,6 +712,7 @@ def process_auth_token(
         json = session.json()
         self.session_id = json["sessionId"]
         self.country_code = json["countryCode"]
+        self.locale = "en_US"  # TODO Set locale from system configuration
         self.user = user.User(self, user_id=json["userId"]).factory()
         self.is_pkce = is_pkce_token
 
@@ -1094,11 +1088,7 @@ def home(self) -> page.Page:
 
         :return: A :class:`.Page` object with the :class:`.PageCategory` list from the home page
         """
-        params = {}
-        params["deviceType"] = "BROWSER"
-        params["countryCode"] = "IT"
-        params["locale"] = "en_US"
-        params["platform"] = "WEB"
+        params = {"deviceType": "BROWSER", "locale": self.locale, "platform": "WEB"}
 
         json_obj = self.request.request(
             "GET",