fix: reset user agent after request has been made

GIScience · Dec 21, 2023 · 712f71f · 712f71f
1 parent 48b1251
commit 712f71f
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/ORStools/common/client.py b/ORStools/common/client.py
@@ -75,13 +75,11 @@ def __init__(self, provider=None, agent=None):
             "Authorization": provider["key"],
         }
 
-        settings = QgsSettings()
+        self.settings = QgsSettings()
         # Read the current value
-        user_agent = settings.value("qgis/networkAndProxy/userAgent")
+        self.user_agent = self.settings.value("qgis/networkAndProxy/userAgent")
         # Set a new value
-        settings.setValue("qgis/networkAndProxy/userAgent", agent)
-        # Reset to old value
-        settings.setValue("qgis/networkAndProxy/userAgent", user_agent)
+        self.settings.setValue("qgis/networkAndProxy/userAgent", agent)
 
         # Save some references to retrieve in client instances
         self.url = None
@@ -191,6 +189,9 @@ def request(self, url, params, first_request_time=None, retry_counter=0, post_js
                     env_var, response.headers.get(self.ENV_VARS[env_var], "None")
                 )
 
+        # Reset to old value
+        self.settings.setValue("qgis/networkAndProxy/userAgent", self.user_agent)
+
         return json.loads(content.decode("utf-8"))
 
     def _check_status(self):