Merge pull request #92 from lcgash/lcg-ignore-health-log

fix: skip log health api

internal/server/endpoint_test.go

@@ -33,6 +33,7 @@ func TestEndpointParams(t *testing.T) {
 	cancel := appPointer.server(&app.Config{
 		ServerPort:            port,
 		PluginEndpointEnabled: parser.ToPtr(true),
+		HealthApiLogEnabled:   parser.ToPtr(true),
 	})
 	defer cancel()
 

internal/server/http_server.go

@@ -18,7 +18,15 @@ import (
 
 // server starts a http server and returns a function to stop it
 func (app *App) server(config *app.Config) func() {
-	engine := gin.Default()
+	engine := gin.New()
+	if *config.HealthApiLogEnabled {
+		engine.Use(gin.Logger())
+	} else {
+		engine.Use(gin.LoggerWithConfig(gin.LoggerConfig{
+			SkipPaths: []string{"/health/check"},
+		}))
+	}
+	engine.Use(gin.Recovery())
 	engine.GET("/health/check", controllers.HealthCheck(config))
 
 	endpointGroup := engine.Group("/e")

internal/types/app/config.go

@@ -110,6 +110,9 @@ type Config struct {
 	// proxy settings
 	HttpProxy  string `envconfig:"HTTP_PROXY"`
 	HttpsProxy string `envconfig:"HTTPS_PROXY"`
+
+	// log settings
+	HealthApiLogEnabled *bool `envconfig:"HEALTH_API_LOG_ENABLED"`
 }
 
 func (c *Config) Validate() error {

internal/types/app/default.go

@@ -34,6 +34,7 @@ func (config *Config) SetDefault() {
 	setDefaultBoolPtr(&config.PipPreferBinary, true)
 	setDefaultBoolPtr(&config.PipVerbose, true)
 	setDefaultString(&config.DBDefaultDatabase, "postgres")
+	setDefaultBoolPtr(&config.HealthApiLogEnabled, true)
 }
 
 func setDefaultInt[T constraints.Integer](value *T, defaultValue T) {