Add an option that makes logger.go fetch the ip in X-Real-IP about goji HOT 4 CLOSED

I think the right way to do this is with a middleware layer that parses either X-Forwarded-For or X-Real-IP and sets r.RemoteAddr based on that. Let me cook something up for you.

from goji.

Well, I updated my code to use goji.Use(middleware.RealIP), and tested.

The headers (both of them) exists and works, but Logger still shows me localhost as the calling ip adress. My full code is here: http://paste.cubox.me/R2J5H

and my nginx conf is here:

location / {
proxy_pass http://localhost:2345;
proxy_set_header  X-Real-IP  $remote_addr;
proxy_set_header X-Forwarded-For $remote_addr;
}

from goji.

Your code is 502'ing, so I can't take a look at it, but are you sure you're inserting middleware.RealIP above the request logger? Something like goji.Insert(middleware.RealIP, middleware.Logger) should do the trick.

from goji.

Yep, I was using .Use(). Now, with insert it's working!

My code is back up, if you want to take a look at it (if I did something horrible in the code).

Thanks!

from goji.