TrueCloudLab/certificates
17
0
Fork 2
Code Issues 1 Pull requests 1 Releases Activity Actions

Add comment about the missing error check.

Browse source
This commit is contained in:
Mariano Cano 2020-10-05 17:39:44 -07:00
parent 8381e9bd17
commit d64427487d
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
authority/tls.go
View file

@ -354,7 +354,9 @@ func (a *Authority) Revoke(ctx context.Context, revokeOpts *RevokeOptions) error
err = a.db.RevokeSSH(rci) err = a.db.RevokeSSH(rci)
} else { } else {
// Revoke an X.509 certificate using CAS. If the certificate is not // Revoke an X.509 certificate using CAS. If the certificate is not
// provided we will try to read it from the db. // provided we will try to read it from the db. If the read fails we
// won't throw an error as it will be responsability of the CAS
// implementation to require a certificate.
var revokedCert *x509.Certificate var revokedCert *x509.Certificate
if revokeOpts.Crt != nil { if revokeOpts.Crt != nil {
revokedCert = revokeOpts.Crt revokedCert = revokeOpts.Crt