[chore] include attemptno in httpclient logs (#2887)

* include request attempt number in httpclient logs

* slightly nicer attempt number formatting
This commit is contained in:
kim 2024-04-30 15:15:50 +01:00 committed by GitHub
parent 39b3a27c82
commit ec334ece20
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -50,10 +50,16 @@ type Request struct {
func WrapRequest(r *http.Request) Request {
var rr Request
rr.Request = r
rr.Entry = log.WithContext(r.Context()).
WithField("method", r.Method).
WithField("url", r.URL.String()).
WithField("contentType", r.Header.Get("Content-Type"))
entry := log.WithContext(r.Context())
entry = entry.WithField("method", r.Method)
entry = entry.WithField("url", r.URL.String())
if r.Body != nil {
// Only add content-type header if a request body exists.
entry = entry.WithField("contentType", r.Header.Get("Content-Type"))
}
// note our formatting library follows ptr values
entry = entry.WithField("attempt", &rr.attempts)
rr.Entry = entry
return rr
}