RE: [Patch] to add more auth parametersto SIP dissector
"Anders Broman (AL/EAB)" <[email protected]>
| Newsgroups | gmane.network.ethereal.devel |
|---|---|
| Message-ID | <C449F18A1C0F294CAF4C0515A18D6EE603741130@esealmw107.eemea.ericsson.se> |
Checked in. Please note that we are currently having some problems, so the patch may not be in the public svn repo for a few days... Brg Anders -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Martin Mathieson Sent: den 25 maj 2006 14:35 To: Ethereal development Subject: Re: [Ethereal-dev] [Patch] to add more auth parametersto SIP dissector Hi Lars, This version of the patch won't look for the authentication scheme (it just skips that part for Authentication-Info headers). I tested it using the enclosed file (pasted from the RFC and fed through od/text2pcap, then messed around with so I could test the other new parameters, even if they don't really belong in that header...). Thanks and regards, Martin > Lars Roland wrote: > >> Hello Martin, >> >> Martin Mathieson schrieb: >> >> >>> Hi, >>> >>> This patch adds a 3 more auth parameters, and now parses the auth >>> parameters in an Authentication-Info header. >>> >>> Regards, >>> Martin >>> >>> >> >> >> Two remarks/questions : >> >> - The authentication-info header has a slightly different syntax as >> the authenticate and authorization headers. It doesn't contain a >> field for the authentication scheme at the beginning. >> We should have an own routine for dissecting the authentication-info >> header. >> >> > Oops, yes. Hadn't tested this part yet... > I'll create a capture and try to post an updated patch later on. > >> - There is no "auts" parameter in RFC 3261. Is it defined in another >> SIP-Extension? >> >> >> > It is defined in RFC 3310, and can appear in the Authorization: header. > >> Regards, >> Lars >> >> > > Thanks, > Martin > _______________________________________________ > Ethereal-dev mailing list > [email protected] > http://www.ethereal.com/mailman/listinfo/ethereal-dev >