freckle / blammo

Batteries-included structured logging library
29 stars 2 forks source link

requestLogger setting localhost as clientIp #18

Closed pbrisbin closed 2 years ago

pbrisbin commented 2 years ago

We're seeing clientIp as a 127.0.0.1 address in request logs.

It looks like bugsnag-wai has an approach that looks at the first value of X-Forwarded-For, then X-Real-Ip, then Wai.remoteHost. I think we should copy that.

We currently just use Wai.remoteHost, which is apparently wrong in an AWS/ALB context.