fix: Use requests lib for fetching jwks

Author: KShivendu

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changes
 
 - Use `useStaticSigningKey` instead of `use_static_signing_key` in `create_jwt` function. This was a bug in the code.
+- Use request library instead of urllib to fetch JWKS keys ([#344](https://github.com/supertokens/supertokens-python/issues/344))
 
 
 ## [0.14.3] - 2023-06-7

supertokens_python/recipe/session/jwks.py

@@ -1,7 +1,6 @@
 import json
-import urllib.request
+import requests
 from typing import List, Optional
-from urllib.error import URLError
 
 from jwt import PyJWK, PyJWKSet
 from jwt.api_jwt import decode_complete as decode_token  # type: ignore
@@ -46,10 +45,11 @@ def __init__(
     def fetch(self):
         try:
             log_debug_message("Fetching jwk set from the configured uri")
-            with urllib.request.urlopen(self.uri, timeout=self.timeout_sec) as response:
+            with requests.get(self.uri, timeout=self.timeout_sec) as response:
+                response.raise_for_status()
                 self.cached_jwks = PyJWKSet.from_dict(json.load(response))  # type: ignore
                 self.last_fetch_time = get_timestamp_ms()
-        except URLError:
+        except requests.HTTPError:
             raise JWKSRequestError("Failed to fetch jwk set from the configured uri")
 
     def is_cooling_down(self) -> bool: