Merge pull request #1186 from letsencrypt/remove-parse
Remove pointless x509.ParseCertificate from ca.IssueCertificate
This commit is contained in:
commit
c689599244
|
|
@ -419,12 +419,7 @@ func (ca *CertificateAuthorityImpl) IssueCertificate(csr x509.CertificateRequest
|
|||
}
|
||||
|
||||
// Submit the certificate to any configured CT logs
|
||||
certObj, err := x509.ParseCertificate(certDER)
|
||||
if err != nil {
|
||||
ca.log.Warning(fmt.Sprintf("Post-Issuance OCSP failed parsing Certificate: %s", err))
|
||||
return cert, nil
|
||||
}
|
||||
go ca.Publisher.SubmitToCT(certObj.Raw)
|
||||
go ca.Publisher.SubmitToCT(certDER)
|
||||
|
||||
// Do not return an err at this point; caller must know that the Certificate
|
||||
// was issued. (Also, it should be impossible for err to be non-nil here)
|
||||
|
|
|
|||
Loading…
Reference in New Issue