Summary: | [authz_core] IPv6 addresses are logged in a not recommended format | ||
---|---|---|---|
Product: | Apache httpd-2 | Reporter: | shadow <shadow> |
Component: | mod_authz_core | Assignee: | Apache HTTPD Bugs Mailing List <bugs> |
Status: | NEW --- | ||
Severity: | normal | ||
Priority: | P2 | ||
Version: | 2.4.51 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Linux |
Description
shadow
2021-11-22 09:25:20 UTC
@my peers / fellow developers: Any preference? Should we in case of an IPV6 address 1. Replace the ':' before the port with a '.'? 2. Replace the ':' before the port with a '#'? 3. Have the IPV6 address in [] and leave the ':'? While the recommended style [ipv6]:port is clear, this will mess up any naive log parser that chunks on "[]" pairs. The format reported seems to be our generic log_remote_address() from server/log.c I favour option 3, but in case we need to be careful with log parsers, I'd choose option 2. Hm, no parser would recognize 1. or 2., and 3. might break the default layout. Do nothing? By default the last colon is for the port, which is always there.. Maybe handle a new ->arg letter for the ErrorLogFormat, e.g. 'x' which would allow those who care to specify "%{x}A" or "%{x}a" or "%{cx}a"? |