switch to client_ip instead of remote_ip

[fliespl]

Handles cases with load balancers when caddy already verified source (i.e. cloudflare)

Resolves: #45

[francislavoie]

I think if we make that change, it should mention that trusted_proxies should be enabled for it to actually have a useful effect.

[pothi]

On Caddy server 2.10.0, both client_ip and real_ip are same if trusted_proxies isn't used. To be safe, I'd recommend keeping the configuration something like...

commonLogFormat = `{request>client_ip:request>real_ip} ` + commonLogEmptyValue + ` {user_id} [{ts}] "{request>method} {request>uri} {request>proto}" {status} {size}`

In this way, if client_ip is not present, real_ip is used. Yet, when trusted_proxies is used, then client_ip is given the precedence in the log.

[mohammed90]

In this way, if client_ip is not present, real_ip is used. Yet, when trusted_proxies is used, then client_ip is given the precedence in the log.

The format of assuming a default value isn't supported by the transform-encoder module.

[mohammed90]

In this way, if client_ip is not present, real_ip is used. Yet, when trusted_proxies is used, then client_ip is given the precedence in the log.

The format of assuming a default value isn't supported by the transform-encoder module.

I don't know why I said this. It's not true, per the README. It shows how long it has been since I touched or looked at this repo. I'm sorry.

Now trying to recover