Planned change to Caddy's access logging

[francislavoie]

This is part question, part "notice".

In caddyserver/caddy#4403, we're planning on splitting up remote_addr into two fields, remote_ip and remote_port. The change is still pending, and the field names may change before merging.

I wanted to ask if it's possible/easy to support both remote_addr and remote_ip at the same time in GoAccess, so that it wouldn't be dependent on the version of Caddy?

[allinurl]

Thanks for checking on this. Just so I understand a bit better, there would be a remote_addr property and a remote_port property instead of having both on the same field? e.g.,

from

{"level":"info","ts":1609440464.1045835,"logger":"http.log.access.log0","request":{"remote_addr":"10.0.2.2:59722" ...}

to

{"level":"info","ts":1609440464.1045835,"logger":"http.log.access.log0","request":{"remote_addr":"10.0.2.2", "remote_port": 59722 ...}

[francislavoie]

Likely remote_addr will go away, and be replaced with remote_ip and remote_port. Still TBD on the field names. The port will be a string (quoted).

[allinurl]

I see. There shouldn't be a problem and actually as pointed on 4403, handling IPv6 without the port on it, would be much easier for goaccess too.

[francislavoie]

FYI these changes have been released in https://github.com/caddyserver/caddy/releases/tag/v2.5.0-beta.1

[allinurl]

Thanks for sharing this. Do you have an ETA for the production release? I'm thinking it may be best to do the switch on goaccess' format when Caddy reaches the stable release.

[francislavoie]

We don't really have a schedule, we usually just cut the release when it "feels good". If we don't get much feedback, we might just make the stable release.

Maybe you could set up a PR in the meantime so you just need to click merge when the time comes? 🤷‍♂️

But yeah, I'll let you know when stable is released. Or if you like, you can subscribe to releases on the Caddy repo in github (Watch button etc).

[allinurl]

Got it. I just subscribed to Caddy's releases. Yes, please feel free to let me know in case I miss it. PR is ready. Thanks again, Francis!

[mholt]

Thanks so much for your ongoing maintenance, @allinurl ! We appreciate it!

[allinurl]

@mholt No, thank you guys for letting me know about these changes.