Merge pull request #300 from clannadxr/master

Default EnableSkipHealthLog
gotomicro · Jun 22, 2022 · 3230688 · 3230688
2 parents 8381c17 + f410efe
commit 3230688
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/server/egrpc/config.go b/server/egrpc/config.go
@@ -20,7 +20,7 @@ type Config struct {
 	EnableMetricInterceptor    bool          // 是否开启监控，默认开启
 	EnableTraceInterceptor     bool          // 是否开启链路追踪，默认开启
 	EnableOfficialGrpcLog      bool          // 是否开启官方grpc日志，默认关闭
-	EnableSkipHealthLog        bool          // 是否屏蔽探活日志，默认关闭
+	EnableSkipHealthLog        bool          // 是否屏蔽探活日志，默认开启
 	SlowLogThreshold           time.Duration // 服务慢日志，默认500ms
 	EnableAccessInterceptor    bool          // 是否开启，记录请求数据
 	EnableAccessInterceptorReq bool          // 是否开启记录请求参数，默认不开启
@@ -40,7 +40,7 @@ func DefaultConfig() *Config {
 		Port:                       9002,
 		Deployment:                 "",
 		EnableMetricInterceptor:    true,
-		EnableSkipHealthLog:        false,
+		EnableSkipHealthLog:        true,
 		EnableTraceInterceptor:     true,
 		SlowLogThreshold:           xtime.Duration("500ms"),
 		EnableAccessInterceptor:    true,

diff --git a/server/egrpc/config_test.go b/server/egrpc/config_test.go
@@ -11,4 +11,5 @@ func TestDefaultConfig(t *testing.T) {
 	assert.Equal(t, "tcp4", cfg.Network)
 	assert.Equal(t, 9002, cfg.Port)
 	assert.Equal(t, ":9002", cfg.Address())
+	assert.Equal(t, true, cfg.EnableSkipHealthLog)
 }