transform-encoder icon indicating copy to clipboard operation
transform-encoder copied to clipboard
verified publisher icon caddyserver

Use client_ip instead of remote_ip in common formats?

Open fliespl opened this issue 1 year ago • 2 comments

Is there a reason for it to be remote_ip? In loadbalancer/proxy setup it states LB ip instead of end user. Client_ip is validated by caddy - i.e. cloudflare x-connecting-ip.

fliespl avatar Sep 28 '24 19:09 fliespl

No reason other than it's left-over from the early days when client_ip wasn't a thing. PR is welcome if you beat us to it.

mohammed90 avatar Sep 28 '24 20:09 mohammed90

@mohammed90 done, I wasn't sure about x-forwarded-for examples in README.md so left them intact.

fliespl avatar Sep 28 '24 20:09 fliespl