fix: do not record ip in metric (#1016)

douyu · Oct 13, 2023 · 933ca67 · 933ca67
1 parent ceb40fa
commit 933ca67
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/pkg/server/xecho/middleware.go b/pkg/server/xecho/middleware.go
@@ -90,12 +90,8 @@ func metricServerInterceptor() echo.MiddlewareFunc {
 			beg := time.Now()
 			err = next(c)
 			method := c.Request().Method + "_" + c.Path()
-			peer := c.RealIP()
-			if aid := extractAID(c); aid != "" {
-				peer += "?aid=" + aid
-			}
-			metric.ServerHandleHistogram.Observe(time.Since(beg).Seconds(), metric.TypeHTTP, method, peer)
-			metric.ServerHandleCounter.Inc(metric.TypeHTTP, method, peer, http.StatusText(c.Response().Status))
+			metric.ServerHandleHistogram.Observe(time.Since(beg).Seconds(), metric.TypeHTTP, method, extractAID(c))
+			metric.ServerHandleCounter.Inc(metric.TypeHTTP, method, extractAID(c), http.StatusText(c.Response().Status))
 			return err
 		}
 	}