envoyproxy/envoy

Upstream request statistics by path

Open

#13,673 建立於 2020年10月21日

在 GitHub 查看
 (11 留言) (12 反應) (0 負責人)C++ (5,373 fork)batch import
area/httphelp wanted

倉庫指標

Star
 (27,997 star)
PR 合併指標
 (平均合併 8天) (30 天內合併 378 個 PR)

描述

Description: Continuing the discussion on https://github.com/envoyproxy/envoy/issues/1585, it would be nice to have path tags on the envoy.upstream_rq for example.

In my opinion it should not be the actual path of the current request, as this could expose sensitive information and also make a lot of unnecessary dimensions on the metric.

But rather, having it based on the RouteMatcher, specifically on prefix, path and safe_regex.

In the end, it does not really matter the full path being used at the moment, but rather which Envoy route was used, and the RouteMatcher would provide us a static and safe tag to be used.

Another approach:

Maybe another thing that could be done is by attaching a tag statically to a route and make that as a tag? Maybe this exists somehow and I'm not aware? Like adding a tag from the route metadata for example...

貢獻者指南