Bug #6585
closedstatus_logs_filter.php ipv6 support (reverse lookup)
100%
Description
status_logs_filter.php reverse lookup lacks IPv6 support,
"i" click to resolve doesn't returs any info for IPv6 address.
Updated by Phillip Davis over 8 years ago
The IPv6 addresses there are displayed using the [1:2::3]:80 format with [IPv6-address]:port which is good for display. But the code seems to be trying to resolve the IPv6 address with the square brackets around it.
I don't have IPv6 resolution where I am right now, so can;t really test. But the change in this pull request at least makes it now come back with "Cannot resolve" on the GUI, where previously it just appeared to do nothing.
https://github.com/pfsense/pfsense/pull/3047
Can you test and confirm if it works or not?
Note: Once a fix is confirmed for this, the same issue needs to be checked in other places where IPv6 reverse lookups are done.
Updated by Luca Moncelli over 8 years ago
It works!
Now for ipv6 addresses ptr records are returned,
and in case of no prt "Cannot resolve".
Great work, thanks!
Updated by Phillip Davis over 8 years ago
Thanks for testing.
The Firewall Log Dynamic view does not have the reverse lookup buton, so it won't be broken!
The dashboard Firewall Log widget works a bit differently for reverse lookups, taking the user to the Diagnostics DNS Lookup page and filling in the address. That is doing the right thing with an IPv6 address - no square brackets around it.
So I think this place in status_logs_filter.php is the only place to fix.
Updated by Chris Buechler over 8 years ago
- Status changed from New to Feedback
- Target version set to 2.3.2
Merged, thanks Phil.
Updated by Phillip Davis over 8 years ago
- % Done changed from 0 to 100
Applied in changeset 30df6b722d577fe8c1b38476244bfb797ec89b5a.
Updated by Phillip Davis over 8 years ago
This issue could be changed from "feature" to "bug".
Updated by Renato Botelho over 8 years ago
- Status changed from Feedback to Resolved