diff --git a/tsweb/tsweb.go b/tsweb/tsweb.go index a45e6ebc7..8d3cdd8b9 100644 --- a/tsweb/tsweb.go +++ b/tsweb/tsweb.go @@ -26,6 +26,7 @@ import ( "go4.org/mem" "tailscale.com/envknob" + "tailscale.com/metrics" "tailscale.com/net/tsaddr" "tailscale.com/tsweb/varz" "tailscale.com/types/logger" @@ -185,18 +186,18 @@ type BucketedStatsOptions struct { // If non-nil, Started maintains a counter of all requests which // have begun processing. - Started *expvar.Map + Started *metrics.LabelMap // If non-nil, Finished maintains a counter of all requests which // have finished processing (that is, the HTTP handler has returned). - Finished *expvar.Map + Finished *metrics.LabelMap } // normalizePathRegex matches components in a HTTP request path // that should be replaced. // // See: https://regex101.com/r/WIfpaR/1 for the explainer and test cases. -var normalizePathRegex = regexp.MustCompile("([a-fA-F0-9]{9,}|([^\\/])+\\.([^\\/]){2,})") +var normalizePathRegex = regexp.MustCompile("([a-fA-F0-9]{9,}|([^\\/])+\\.([^\\/]){2,}|((n|k|u|L|t|S)[a-zA-Z0-9]{5,}(CNTRL|Djz1H|LV5CY|mxgaY|jNy1b)))") // NormalizedPath returns the given path with the following modifications: // @@ -205,6 +206,7 @@ var normalizePathRegex = regexp.MustCompile("([a-fA-F0-9]{9,}|([^\\/])+\\.([^\\/ // replaced by an ellipsis // - any path component containing a period with at least two characters // after the period (i.e. an email or domain) +// - any path component consisting of a common Tailscale Stable ID. func NormalizedPath(p string) string { // Fastpath: No hex sequences in there we might have to trim. // Avoids allocating. diff --git a/tsweb/tsweb_test.go b/tsweb/tsweb_test.go index 0ace46236..154769ddc 100644 --- a/tsweb/tsweb_test.go +++ b/tsweb/tsweb_test.go @@ -684,6 +684,8 @@ func TestBucket(t *testing.T) { {"/api/v2/tailnet/jeremiah@squish.com/devices", "/api/v2/tailnet/…/devices"}, {"/machine/ssh/wait/5227109621243650/to/7111899293970143/a/a9e4e04cc01b", "/machine/ssh/wait/…/to/…/a/…"}, {"/a/831a4bf39856?refreshed=true", "/a/…"}, + {"/c2n/nxaaa1CNTRL", "/c2n/…"}, + {"/api/v2/tailnet/blueberries.com/keys/kxaDK21CNTRL", "/api/v2/tailnet/…/keys/…"}, } for _, tc := range tcs {