refactor: use empty pool rather than failing if system cert load fails

TylerGillson · TylerGillson · commit 5c96fd2536a7 · 2024-07-22T14:38:01.000-06:00
Signed-off-by: Tyler Gillson &lt;tyler.gillson@gmail.com&gt;
diff --git a/internal/sinks/alertmanager.go b/internal/sinks/alertmanager.go
@@ -84,7 +84,8 @@ func (s *AlertmanagerSink) Configure(c Client, config map[string][]byte) error {
 	if ok {
 		caCertPool, err = x509.SystemCertPool()
 		if err != nil {
-			return errors.Wrap(err, "invalid Alertmanager config: failed to get system cert pool")
+			s.log.Error(err, "failed to get system cert pool; using empty pool")
+			caCertPool = x509.NewCertPool()
 		}
 		caCertPool.AppendCertsFromPEM(caCert)
 	}

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,8 @@ func (s *AlertmanagerSink) Configure(c Client, config map[string][]byte) error {`
`84`	`84`	`if ok {`
`85`	`85`	`caCertPool, err = x509.SystemCertPool()`
`86`	`86`	`if err != nil {`
`87`		`- return errors.Wrap(err, "invalid Alertmanager config: failed to get system cert pool")`
	`87`	`+ s.log.Error(err, "failed to get system cert pool; using empty pool")`
	`88`	`+ caCertPool = x509.NewCertPool()`
`88`	`89`	`}`
`89`	`90`	`caCertPool.AppendCertsFromPEM(caCert)`
`90`	`91`	`}`