fix: disable health check request logs

project0 · project0 · commit b831ff54facf · 2023-04-19T09:47:43.000+02:00
The excluded path is wrong
diff --git a/cmd/server/handler.go b/cmd/server/handler.go
@@ -201,7 +201,7 @@ func setup(ctx context.Context, d driver.Registry, cmd *cobra.Command) (admin *h
 		NewMiddlewareFromLogger(d.Logger(),
 			fmt.Sprintf("hydra/admin: %s", d.Config().IssuerURL(ctx).String()))
 	if d.Config().DisableHealthAccessLog(config.AdminInterface) {
-		adminLogger = adminLogger.ExcludePaths("/admin"+healthx.AliveCheckPath, "/admin"+healthx.ReadyCheckPath)
+		adminLogger = adminLogger.ExcludePaths(healthx.AliveCheckPath, healthx.ReadyCheckPath)
 	}
 
 	adminmw.Use(adminLogger)

Original file line number	Diff line number	Diff line change
`@@ -201,7 +201,7 @@ func setup(ctx context.Context, d driver.Registry, cmd cobra.Command) (admin h`
`201`	`201`	`NewMiddlewareFromLogger(d.Logger(),`
`202`	`202`	`fmt.Sprintf("hydra/admin: %s", d.Config().IssuerURL(ctx).String()))`
`203`	`203`	`if d.Config().DisableHealthAccessLog(config.AdminInterface) {`
`204`		`- adminLogger = adminLogger.ExcludePaths("/admin"+healthx.AliveCheckPath, "/admin"+healthx.ReadyCheckPath)`
	`204`	`+ adminLogger = adminLogger.ExcludePaths(healthx.AliveCheckPath, healthx.ReadyCheckPath)`
`205`	`205`	`}`
`206`	`206`
`207`	`207`	`adminmw.Use(adminLogger)`