From 34ff23f2b949b1750e31268f738a54c5bcc89367 Mon Sep 17 00:00:00 2001 From: Alexander Bokovoy Date: Nov 05 2025 07:56:00 +0000 Subject: Use requests.Session class directly and inherit default headers requests.session() was deprecated in requests v1.0.0. By using requests.Session class directly we also rely on the default headers supplied by the requests.Session class and extend them rather than overriding when token was specified: >>> import requests >>> requests.Session().headers {'User-Agent': 'python-requests/2.32.4', 'Accept-Encoding': 'gzip, deflate, br, zstd', 'Accept': '*/*', 'Connection': 'keep-alive'} Fixes: https://pagure.io/libpagure/issue/52 Signed-off-by: Alexander Bokovoy --- diff --git a/libpagure/libpagure.py b/libpagure/libpagure.py index 11965b9..d12edfa 100644 --- a/libpagure/libpagure.py +++ b/libpagure/libpagure.py @@ -56,7 +56,7 @@ class Pagure(object): self.username = fork_username self.namespace = namespace self.instance = instance_url - self.session = requests.session() + self.session = requests.Session() adapter = requests.adapters.HTTPAdapter(max_retries=5) @@ -67,7 +67,7 @@ class Pagure(object): self.session.mount("https://", adapter) if self.token: - self.session.headers = {"Authorization": "token " + self.token} + self.session.headers["Authorization"] = "token " + self.token def _call_api(self, url, method="GET", params=None, data=None): """ Method used to call the API.