Wireshark-bugs: [Wireshark-bugs] [Bug 839] Inconsistent inclusion of CRLF/0d0a in SIP fields

Date: Sat, 15 Mar 2008 18:41:16 +0000 (GMT)
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=839


Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED




--- Comment #5 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>  2008-03-15 18:41:09 GMT ---
I would argue that the CRLF should be part of each field--like it is with HTTP.
 One good reason for this is so we can Copy (as bytes, text, etc.) and get the
full line including the CRLF so we can copy/paste multiple lines without having
to hit enter each time..

That being said, it appears the SIP dissector is now consistent in including
the CRLF in the fields.  I checked all the lines in the attached capture
(including the Status line) and they all had it.  Resolving this was Fixed (who
knows when it was fixed, probably a long time ago).

ps. this bug also mentioned inconsistency in RTSP regarding whether the CRLF is
included.  I checked out one of the sample captures on the Wiki and each field
included the CRLF so it appears that problem, too, has been fixed.


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