| Summary: | ASTERISK-04417: Uniden "workarounds" (nat=route) seems to be partly broken in CVS-HEAD | ||
| Reporter: | nbougues (nbougues) | Labels: | |
| Date Opened: | 2005-06-15 08:45:16 | Date Closed: | 2008-01-15 15:38:29.000-0600 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Channels/chan_sip/General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) patch_uniden_sip_route | |
| Description: | After having been running for a while on 1.0, I upgraded to CVS-HEAD, and discovered that the Uniden nat=route workaround didn't work as expected. Namely, messages with the ";rport" suffix in Via were sent to phones (and unfortunatly, UIP200 still ignores such messages) when INVITEing them. So here is a one-line patch that corrects it. | ||
| Comments: | By: Brian West (bkw918) 2005-06-16 10:08:52 this is what nat=never was invented for. Go try that. /b By: nbougues (nbougues) 2005-06-16 14:36:25 Err, yes, of course, unless the phone happens to be located behind a NAT... By: Kevin P. Fleming (kpfleming) 2005-06-17 11:03:01 You are correct, this was an unintended behavior change in an earlier patch. Committed to CVS HEAD, thanks! By: Digium Subversion (svnbot) 2008-01-15 15:38:29.000-0600 Repository: asterisk Revision: 5931 U trunk/channels/chan_sip.c ------------------------------------------------------------------------ r5931 | kpfleming | 2008-01-15 15:38:29 -0600 (Tue, 15 Jan 2008) | 2 lines don't generate 'rport' unless RFC3581 support is enabled (bug ASTERISK-4417) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5931 | ||