Wireshark-bugs: [Wireshark-bugs] [Bug 4343] [PATCH] SIP dissector: remove [ ] brackets from IPv6

Date: Tue, 26 Jan 2010 08:54:43 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4343

--- Comment #7 from Jaap Keuter <jaap.keuter@xxxxxxxxx> 2010-01-26 17:54:40 CET ---
(In reply to comment #6)
> from the RFC point of view it's the same in case of IPV6:
> 
> via: ipv6 host = IPv6reference =  "[" IPv6address "]"
> r-uri: ipv6 host = IP-literal = "[" ( IPv6address / IPvFuture  ) "]"

Not quite. As the screenshot shows:
In the details of the Request URI the 'host part' is presented. According to
the BNF the hostpart with an IPv6 address contains the square brackets and the
IPv6 address.
In the details of the Via header the 'Address' is presented. According to the
BNF the address in case of IPv6 is the plain IPv6 address. No square brackets,
these are part of the IPv6reference, as described in the BNF.

> but it's different displayed by Wireshark today, what do you 
> think would be correct display?

I would consider the best option to be:
1.) The header lines shown in full, including all grouping and separation
characters, as present in the message captured.
2.) The details as user friendly as possible, stripping all non-relevant
grouping and separation characters.

Indeed this patch would contribute to that, but I would like to see a more
overall approach in which proper names are used in the header fields.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.