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-0600 | Date Closed: | 2008-01-15 15:29:04.000-0600 |
Priority: | Trivial | Regression? | No |
Status: | Closed/Complete | Components: | 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 |