Merge pull request #63 from cisagov/improvement/update-search-requests

Update CT Search request

examples/load_certs.py

@@ -140,7 +140,7 @@ def group_update_domain(domain, max_expired_date, verbose=False, dry_run=False):
 
     # create x509 certificates from the results
     for task, result in tasks_to_results:
-        data = base64.b64decode(result["data"])  # encoded in ASN.1 DER
+        data = base64.b64decode(result)  # encoded in ASN.1 DER
         pem = ssl.DER_cert_to_PEM_cert(data)
         cert, is_precert = Cert.from_pem(pem)
         cert.log_id = task.get("args")[0]["id"]  # get log_id from task

src/admiral/_version.py

@@ -1,2 +1,2 @@
 """This file defines the version of this module."""
-__version__ = "1.3.2"
+__version__ = "1.3.4"

src/admiral/certs/tasks.py

@@ -53,7 +53,7 @@ def summary_by_domain(domain, subdomains=True):
     logger.info(f"Fetching certs from CT log for: {domain}")
     url = (
         f"https://api.certspotter.com/v1/issuances?domain={domain}&include_subdomains={subdomains}"
-        f"&expand=dns_names&expand=cert"
+        f"&expand=dns_names&expand=cert_der"
     )
     req = requests.get(
         url,
@@ -82,4 +82,4 @@ def cert_by_issuance(issuance):
     id = issuance["id"]
     logger.info(f"Fetching cert data from CT log for id: {id}.")
 
-    return issuance["cert"]
+    return issuance["cert_der"]