PalmStoneGames / kube-cert-manager

Manage Lets Encrypt certificates for a Kubernetes cluster.
Apache License 2.0
541 stars 102 forks source link

Add error checking to pem.Decode call #87

Closed ean closed 7 years ago

ean commented 7 years ago

I have seen the line with x509.ParseCertificate() panic because of nil pointer dereference a few times. Have not been able to track down why acmeCert.Cert sometime is populated with a json null value, but this will at least stop kube-cert-manager from dying.