[Home]

Summary:ASTERISK-03711: [patch] Don't send tag in 100 Trying responses
Reporter:khb (khb)Labels:
Date Opened:2005-03-19 13:16:20.000-0600Date Closed:2008-01-15 15:29:04.000-0600
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Channels/chan_sip/Interoperability
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) patch_fix_100_trying.txt
( 1) patch_tag_parsing.txt
Description:Sending a to-tag in 100 Trying violates standard,
since at that point in the dialog, the dialog is not
established and should not have a to-tag assigned yet.
Comments:By: khb (khb) 2005-03-19 13:18:19.000-0600

Also, the tag= label should be parsed caseinsensituve.

disclaimed despite trivial, non IP, nature
By: Kevin P. Fleming (kpfleming) 2005-03-19 13:24:12.000-0600

This appears to be correct, the RFC is very explicit that only 101-199 progress responses establish the dialog.
By: khb (khb) 2005-03-19 13:28:10.000-0600

Now go and tell others, like Polycom (see parallel issue) to fix theirs.
By: khb (khb) 2005-03-19 13:49:22.000-0600

added additional patch to upgrade more tag= caseinsensitive parsers
By: Olle Johansson (oej) 2005-03-19 15:32:38.000-0600

khb: Can you produce one combined patch to make life easier for Markster? Thank you.
By: Mark Spencer (markster) 2005-03-19 16:52:42.000-0600

Added to CVS, thanks!
By: Russell Bryant (russell) 2005-03-28 00:41:41.000-0600

fixed in 1.0 with some small mods to compile on non-linux systems

(use ast_strcasestr instead of strcasestr)
By: Digium Subversion (svnbot) 2008-01-15 15:29:04.000-0600

Repository: asterisk
Revision: 5287

U   branches/v1-0/CHANGES
U   branches/v1-0/channels/chan_sip.c

------------------------------------------------------------------------
r5287 | russell | 2008-01-15 15:29:04 -0600 (Tue, 15 Jan 2008) | 2 lines

don't send a "to" on a "100 Trying", also be case insensitive when parsing the "to" tag (bug ASTERISK-3711)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=5287