Summary:	ASTERISK-13808: [patch] Added ability to specify different SIP and media addresses
Reporter:	Brent Thomson (_brent_)	Labels:
Date Opened:	2009-03-23 17:52:40	Date Closed:	2009-10-21 18:46:02
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Core/RTP
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) media_address.patch
Description:	Added rtp_bindaddr setting to chan_sip. Enabled by setting rtp_bindaddr=x.x.x.x in sip.conf. This should be particularly useful to anyone running Asterisk behind a SIP proxy, like OpenSER/OpenSIPS/Kamailio. This patch allows Asterisk to report its externip (again, set in sip.conf) as the proxy's IP address, but its media address as its own to bypass the proxy. ****** ADDITIONAL INFORMATION ****** Related to issue ASTERISK-7197252, but not implemented by actually binding the RTP to the specified IP address. This patch only manipulates the information in the SDP of a SIP message. This allows the user to specify an IP address for media that isn't necessarily bindable. This, in turn, allows Asterisk to run with only a private IP address behind a router that forwards RTP traffic to that private IP address.
Comments:	By: Brent Thomson (_brent_) 2009-03-23 18:53:33 Hmmm... I chose rtp_bindaddr as the name of the configuration parameter because that's what the other issue had named it. I'm having seconds thoughts about that name, though, since it's not actually binding the address, just inserting it in the SDP. Any thoughts on this? By: Brent Thomson (_brent_) 2009-03-25 10:06:38 I've changed the name of everything from rtp_bindaddr to media_address and uploaded a new patch. rtp_bindaddr.patch can be deleted, please. Now, the configuration to be added to sip.conf is: media_address=x.x.x.x By: Leif Madsen (lmadsen) 2009-09-18 09:21:19 Can you gather some folks from the mailing lists (asterisk-users,asterisk-dev) to do some testing and report back here? Thanks! By: John Todd (jtodd) 2009-10-05 12:25:50 OEJ, can you make a comment on this? By: John Todd (jtodd) 2009-10-05 12:26:59 Can you take a look at this one and make comments?  Seems reasonable, and might be relevant to some of the things you're doing. By: Digium Subversion (svnbot) 2009-10-21 08:39:13 Repository: asterisk Revision: 225003 U   trunk/CHANGES U   trunk/channels/chan_sip.c U   trunk/configs/sip.conf.sample ------------------------------------------------------------------------ r225003 | file | 2009-10-21 08:39:12 -0500 (Wed, 21 Oct 2009) | 7 lines Add support for specifying the IP address to use for media streams in sip.conf (closes issue ASTERISK-13808) Reported by: _brent_ Patches:      media_address.patch uploaded by brent (license 388) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225003 By: Digium Subversion (svnbot) 2009-10-21 08:39:38 Repository: asterisk Revision: 225004 _U  branches/1.6.2/ ------------------------------------------------------------------------ r225004 | file | 2009-10-21 08:39:37 -0500 (Wed, 21 Oct 2009) | 13 lines Blocked revisions 225003 via svnmerge ........  r225003 | file | 2009-10-21 10:34:49 -0300 (Wed, 21 Oct 2009) | 7 lines    Add support for specifying the IP address to use for media streams in sip.conf    (closes issue ASTERISK-13808)  Reported by: _brent_  Patches:        media_address.patch uploaded by brent (license 388) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225004 By: Digium Subversion (svnbot) 2009-10-21 11:09:21 Repository: asterisk Revision: 225089 U   trunk/CHANGES U   trunk/channels/chan_sip.c U   trunk/configs/sip.conf.sample ------------------------------------------------------------------------ r225089 | file | 2009-10-21 11:09:21 -0500 (Wed, 21 Oct 2009) | 11 lines Add support for specifying the IP address to use for media streams in sip.conf This is the second commit for this and documents the text stream using the configured IP address and fixes a bug in the original patch where the UDPTL stream would also use the different IP address. (closes issue ASTERISK-13808) Reported by: _brent_ Patches:      media_address.patch uploaded by brent (license 388) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225089