summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthew Holt <mholt@users.noreply.github.com>2020-03-14 15:20:04 -0600
committerMatthew Holt <mholt@users.noreply.github.com>2020-03-14 15:20:04 -0600
commitacf4dde1dd1fa2d30b76c19009e5a7019cce0b2b (patch)
treea8d4e61054297706fa3217fbf1031539ab3d48a5
parent7a4548c5823e85bab0a2e2f40a3ea00f64ce8552 (diff)
pki: Don't treat cert installation failure as error
See https://caddy.community/t/fail-to-start-caddy2-not-nss-security-databases-found/7223?u=matt
-rw-r--r--modules/caddypki/pki.go6
1 files changed, 5 insertions, 1 deletions
diff --git a/modules/caddypki/pki.go b/modules/caddypki/pki.go
index 373b4cb..562fae2 100644
--- a/modules/caddypki/pki.go
+++ b/modules/caddypki/pki.go
@@ -92,7 +92,11 @@ func (p *PKI) Start() error {
truststore.WithJava(),
)
if err != nil {
- return fmt.Errorf("adding root certificate to trust store: %v", err)
+ // could be some system dependencies that are missing;
+ // shouldn't totally prevent startup, but we should log it
+ p.log.Error("failed to install root certificate",
+ zap.Error(err),
+ zap.String("certificate_file", ca.rootCertPath))
}
}