Skip to content

Commit

Permalink
fix: remove the req guid from the metric to avoid the high number of …
Browse files Browse the repository at this point in the history
…data points in the metric db (#55)
  • Loading branch information
AkbaraliShaikh authored Dec 7, 2022
1 parent d0c37da commit e504267
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion services/rest/websocket/handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ func (h *Handler) HandlerWSEvents(w http.ResponseWriter, r *http.Request) {
if config.Server.DedupEnabled {
// avoiding processing the same active connection's duplicate events.
if h.upgrader.Table.HasBatch(conn.Identifier, payload.ReqGuid) {
metrics.Increment("events_duplicate_total", fmt.Sprintf("request_guid=%s,reason=duplicate,conn_group=%s", payload.ReqGuid, conn.Identifier.Group))
metrics.Increment("events_duplicate_total", fmt.Sprintf("reason=duplicate,conn_group=%s", conn.Identifier.Group))
writeSuccessResponse(conn, s, messageType, payload.ReqGuid)
continue
}
Expand Down

0 comments on commit e504267

Please sign in to comment.