Skip to content

Commit

Permalink
internal/teeproxy: do not include https:// in event key
Browse files Browse the repository at this point in the history
Change-Id: I9e3e201138584b3c5cd050c8bd4666ffa22929b1
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/249960
Run-TryBot: Julie Qiu <julie@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
  • Loading branch information
julieqiu committed Aug 28, 2020
1 parent dee9bdb commit ddade37
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/teeproxy/teeproxy.go
Original file line number Diff line number Diff line change
Expand Up @@ -283,7 +283,7 @@ func (s *Server) doRequest(r *http.Request) (results map[string]*RequestEvent, s
if event.Error != nil {
log.Errorf(r.Context(), "teeproxy.Server.doRequest(%q): %s", host, event.Error)
}
results[host] = event
results[strings.TrimPrefix(host, "https://")] = event
recordTeeProxyMetric(r.Context(), host, gddoEvent.Path, gddoEvent.Status, event.Status, gddoEvent.Latency, event.Latency)
}
}
Expand Down

0 comments on commit ddade37

Please sign in to comment.