search

xmidt-org / sallust

Logging for microservices, bringing together structured logging with file management
Apache License 2.0
2 stars 0 forks source link

Include headers in sallusthttp #19

Open johnabass opened 3 years ago

johnabass commented 3 years ago

sallusthttp already has a decorator for enriching loggers with http.Request information. However, it doesn't include HTTP headers.

We should add a way for sallusthttp.Builder to include HTTP headers, which would also involve filtering sensitive headers such as Authorization.

johnabass commented 3 years ago

This is the specific code to be augmented:

https://github.com/xmidt-org/sallust/blob/main/sallusthttp/builder.go