@marcel91200 / caddy Mirror · read-only / commits / acf4dde1
Fork 0

Fast and extensible multi-platform HTTP/1-2-3 web server with automatic HTTPS

Mirrored from https://github.com/Slazee/caddy.git

Synced Nov 14, 05:15 UTC

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

Matthew Holt committed Mar 14, 2020 at 15:20 UTC acf4dde1dd1fa2d30b76c19009e5a7019cce0b2b
1 file changed +5 -1
modules/caddypki/pki.go
+5 -1
@@ -92,7 +92,11 @@ func (p *PKI) Start() error {
92 truststore.WithJava(),
93 )
94 if err != nil {
95 - return fmt.Errorf("adding root certificate to trust store: %v", err)
95 + // could be some system dependencies that are missing;
96 + // shouldn't totally prevent startup, but we should log it
97 + p.log.Error("failed to install root certificate",
98 + zap.Error(err),
99 + zap.String("certificate_file", ca.rootCertPath))
100 }
101 }
102