fix: bugs creating Certificates from URI

pki_tools/types/certificates.py

@@ -2,7 +2,6 @@
 from functools import lru_cache
 from typing import List, Type, T
 
-import httpx
 from cryptography import x509
 from loguru import logger
 
@@ -12,6 +11,7 @@
 from pki_tools.exceptions import OcspIssuerFetchFailure
 from pki_tools.types.certificate import Certificate
 from pki_tools.types.crypto_parser import CryptoParser
+from pki_tools.utils import HTTPX_CLIENT
 
 CACHE_TIME_SECONDS = 60 * 60 * 24 * 30  # 1 month
 
@@ -100,7 +100,7 @@ def from_uri(
     @classmethod
     @lru_cache(maxsize=None)
     def _from_uri(cls: T, uri: str, ttl=None) -> T:
-        ret = httpx.get(uri)
+        ret = HTTPX_CLIENT.get(uri)
 
         if ret.status_code != 200:
             logger.bind(status=ret.status_code).error(
@@ -110,7 +110,7 @@ def _from_uri(cls: T, uri: str, ttl=None) -> T:
                 f"Issuer URI fetch failed. Status: {ret.status_code}"
             )
 
-        return cls(certificates=x509.load_pem_x509_certificate(ret.content))
+        return cls.from_pem_string(ret.text)
 
     @property
     def pem_string(self):

test/test_types_cetificates.py

@@ -1,6 +1,5 @@
 import os
 
-
 from conftest import CURRENT_DIR
 from pki_tools import Certificates
 
@@ -21,3 +20,10 @@ def test_certificates_save_and_read_file(cert_pem_string):
     os.remove(file_path)
 
     assert certs.pem_string == new_certs.pem_string
+
+
+def test_certificates_from_uri(mocked_requests_get, cert_pem_string):
+    mocked_requests_get.return_value.status_code = 200
+    mocked_requests_get.return_value.text = cert_pem_string
+
+    Certificates.from_uri("http://TEST_URI")