cisagov / admiral

Distributed certificate transparency log harvester
Creative Commons Zero v1.0 Universal
14 stars 3 forks source link

Fix crash on save #46

Closed king-alexander closed 1 year ago

king-alexander commented 1 year ago

๐Ÿ—ฃ Description

This is a simple change to extract the log_id from the cert_by_issuance task. The line of code broke when we switched APIs.

๐Ÿ’ญ Motivation and context

๐Ÿงช Testing

I didn't catch this bug earlier because I tested without saving certificates to the database. This time I did. I ran the load_certs script on the opensource.gov domain and successfully saved 100 certificates to the development database.

โœ… Pre-approval checklist

โœ… Pre-merge checklist

โœ… Post-merge checklist