Summary:	ASTERISK-15289: [patch] IP and port is not transferred for t.38
Reporter:	stanusr (stanusr)	Labels:
Date Opened:	2009-12-08 07:32:54.000-0600	Date Closed:	2010-01-12 13:13:57.000-0600
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Channels/chan_sip/T.38
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) __20091210-sip.conf.sample-documentation.txt ( 1) debug.log
Description:	A SIP audio call is set up between two parties with an Asterisk server in between. The media  stream goes directly from the caller to the callee. When Asterisk receives a re-invite on channel leg2 (callee) to change to T.38 it does not transfer the new media IP address and Port to the invite for channel leg1 (caller), as expected. It sends the ”old” media IP address and presumably a PORT generated by Asterisk self in the udptlstart-udptlend range specified in udptl.conf.
Comments:	By: stanusr (stanusr) 2009-12-08 07:49:27.000-0600 I have attached a log file from Asterisk. The media IP address and port in the last invite is not as expected. I expect the IP address to be 10.0.0.190 and the port to be 38102. By: Leif Madsen (lmadsen) 2009-12-08 10:11:50.000-0600 Per jcolp, this is considered a feature request as we don't support direct media on UDPTL streams. By: stanusr (stanusr) 2009-12-09 03:56:23.000-0600 I think that there should be an indication that it is not supported. I would expect Asterisk to give a warning or send an error back in response to the INVITE and not just setup a wrong media path. Another acceptable handling would be that Asterisk automatically change back from direct media stream as I have not set directrtpsetup=yes. By: Leif Madsen (lmadsen) 2010-01-12 12:41:40.000-0600 Closed by commit 239520 to trunk, and then merged down. By: Digium Subversion (svnbot) 2010-01-12 13:13:47.000-0600 Repository: asterisk Revision: 239520 U   trunk/configs/sip.conf.sample ------------------------------------------------------------------------ r239520 | lmadsen | 2010-01-12 13:13:47 -0600 (Tue, 12 Jan 2010) | 6 lines Note that direct T.38 is not supported. (closes issue ASTERISK-15289) Reported by: stanusr Patches:      __20091210-sip.conf.sample-documentation.txt uploaded by lmadsen (license 10) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239520 By: Digium Subversion (svnbot) 2010-01-12 13:13:50.000-0600 Repository: asterisk Revision: 239521 _U  branches/1.6.0/ U   branches/1.6.0/configs/sip.conf.sample ------------------------------------------------------------------------ r239521 | lmadsen | 2010-01-12 13:13:50 -0600 (Tue, 12 Jan 2010) | 14 lines Merged revisions 239520 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r239520 | lmadsen | 2010-01-12 12:22:45 -0600 (Tue, 12 Jan 2010) | 6 lines    Note that direct T.38 is not supported.    (closes issue ASTERISK-15289)  Reported by: stanusr  Patches:        __20091210-sip.conf.sample-documentation.txt uploaded by lmadsen (license 10) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239521 By: Digium Subversion (svnbot) 2010-01-12 13:13:53.000-0600 Repository: asterisk Revision: 239522 _U  branches/1.6.1/ U   branches/1.6.1/configs/sip.conf.sample ------------------------------------------------------------------------ r239522 | lmadsen | 2010-01-12 13:13:53 -0600 (Tue, 12 Jan 2010) | 14 lines Merged revisions 239520 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r239520 | lmadsen | 2010-01-12 12:22:45 -0600 (Tue, 12 Jan 2010) | 6 lines    Note that direct T.38 is not supported.    (closes issue ASTERISK-15289)  Reported by: stanusr  Patches:        __20091210-sip.conf.sample-documentation.txt uploaded by lmadsen (license 10) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239522 By: Digium Subversion (svnbot) 2010-01-12 13:13:56.000-0600 Repository: asterisk Revision: 239523 _U  branches/1.6.2/ U   branches/1.6.2/configs/sip.conf.sample ------------------------------------------------------------------------ r239523 | lmadsen | 2010-01-12 13:13:56 -0600 (Tue, 12 Jan 2010) | 14 lines Merged revisions 239520 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r239520 | lmadsen | 2010-01-12 12:22:45 -0600 (Tue, 12 Jan 2010) | 6 lines    Note that direct T.38 is not supported.    (closes issue ASTERISK-15289)  Reported by: stanusr  Patches:        __20091210-sip.conf.sample-documentation.txt uploaded by lmadsen (license 10) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239523