fix: explicitly recover panics on all servers

GitOrigin-RevId: 4f914909d2e5aabd2b0803273d9d4377c26b5d09

Author: zepatrik

cmd/courier/watch.go

@@ -55,13 +55,17 @@ func StartCourier(ctx context.Context, r driver.Registry) error {
 func ServeMetrics(ctx context.Context, r driver.Registry, port int) error {
 	cfg := r.Config().ServeAdmin(ctx)
 	l := r.Logger()
-	n := negroni.New()
 
-	router := http.NewServeMux()
+	recovery := negroni.NewRecovery()
+	recovery.Logger = l
 
-	router.Handle(prometheusx.MetricsPrometheusPath, promhttp.Handler())
-	n.Use(reqlog.NewMiddlewareFromLogger(l, "admin#"+cfg.BaseURL.String()))
+	n := negroni.New(
+		recovery,
+		reqlog.NewMiddlewareFromLogger(l, "admin#"+cfg.BaseURL.String()),
+	)
 
+	router := http.NewServeMux()
+	router.Handle(prometheusx.MetricsPrometheusPath, promhttp.Handler())
 	n.UseHandler(router)
 
 	var handler http.Handler = n

cmd/daemon/serve.go

@@ -55,8 +55,13 @@ var httpMetrics = prometheusx.NewHTTPMetrics("kratos", prometheusx.HTTPPrefix, c
 func servePublic(ctx context.Context, r *driver.RegistryDefault, cmd *cobra.Command) (func() error, error) {
 	cfg := r.Config().ServePublic(ctx)
 	l := r.Logger()
+
+	rec := negroni.NewRecovery()
+	rec.Logger = l
+
 	router := httprouterx.NewRouterPublic()
 	n := negroni.New(
+		rec,
 		httprouterx.PopulatePatternNegroni(router),
 		httpMetrics,
 	)
@@ -151,8 +156,13 @@ func servePublic(ctx context.Context, r *driver.RegistryDefault, cmd *cobra.Comm
 func serveAdmin(ctx context.Context, r *driver.RegistryDefault, cmd *cobra.Command) (func() error, error) {
 	cfg := r.Config().ServeAdmin(ctx)
 	l := r.Logger()
+
+	rec := negroni.NewRecovery()
+	rec.Logger = l
+
 	router := httprouterx.NewRouterAdminWithPrefix()
 	n := negroni.New(
+		rec,
 		httprouterx.PopulatePatternNegroni(router),
 		httpMetrics,
 	)