Summary:	ASTERISK-13898: Mixmonitor/Monitor stops when automated dial out call files used with local channel
Reporter:	Alisher (licedey)	Labels:
Date Opened:	2009-04-04 21:43:00	Date Closed:	2009-05-28 18:59:10
Priority:	Trivial	Regression?	No
Status:	Closed/Complete	Components:	Channels/chan_local
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) dialplan.zip
Description:	I use local channel to record automated dial out calls. However after receiver answer the phone, Asterisk hangs up local channel and bridge both parties at different call, making it impossible to record the calls. I have tried to use AUDIOHOOK_INHERIT function in dial plan and set it from auto dial out call file but it didn't help. ****** ADDITIONAL INFORMATION ****** auto dial out : -------------------- Channel: SIP/3005 Callerid: 3005 <3005> Set: AUDIOHOOK_INHERIT(MixMonitor)=yes MaxRetries: 1 RetryTime: 30 WaitTime: 30 Application: Dial Data: Local/822222233332@dialout
Comments:	By: Mark Michelson (mmichelson) 2009-04-06 08:27:51 I'm assuming that the call to the MixMonitor application is happening in extension 822222233332@dialout. The problem is that you are placing the MixMonitor audiohook on the Local channel which is optimized away once the call is bridged. A temporary workaround for this problem would be to replace the "Data" line in the call file with: Data:Local/822222233332@dialout/n By: Mark Michelson (mmichelson) 2009-04-06 09:11:36 Actually, I looked at the code in 1.4, and there are preventative actions taken to be sure that an audiohook is properly moved to where it should be during a Local channel masquerade. I am unable to reproduce your problem, even without using AUDIOHOOK_INHERIT. Could you upload the dialplan that is executed when you attempt using this call file? By: Alisher (licedey) 2009-04-06 11:59:13 I have uploaded the dialplan. The /n option solved the problem. Thank you very much. By: Mark Michelson (mmichelson) 2009-04-06 12:39:30 Thanks for the dialplan. Based on tests I am running, it appears that the 'b' option to MixMonitor may be what is causing the recording to stop. I'll need to run a few more tests to verify, but that is my early analysis. By: Mark Michelson (mmichelson) 2009-04-06 12:42:41 Changing status to 'acknowledged' since feedback is no longer necessary and I have reproduced the issue. By: Leif Madsen (lmadsen) 2009-04-20 10:31:33 The resolution for this issue is going to be something to do with documentation, and letting the user know that you will need to use /n in order to keep the Local channel up so that it can be monitored. Need to decide where this will go through (AUDIOHOOOK_INHERIT? MixMonitor? Both?) By: Alisher (licedey) 2009-04-20 18:08:22 The /n option works even when AUDIOHOOK_INHERIT function is not used. So, I think it should go with MixMonitor. By: Leif Madsen (lmadsen) 2009-05-27 14:44:38 NOTE: If you utilize this option inside a Local channel, you must    make sure the Local channel is not optimized away. To do this, be sure    to call your Local channel with the '/n' option. For example: Dial(Lo    cal/start@mycontext/n) That's the note I came up with. What does everyone think? This is a note associated with the 'b' option. By: Digium Subversion (svnbot) 2009-05-28 17:04:01 Repository: asterisk Revision: 197828 U   trunk/apps/app_mixmonitor.c ------------------------------------------------------------------------ r197828 | lmadsen | 2009-05-28 17:04:00 -0500 (Thu, 28 May 2009) | 8 lines Update documentation in MixMonitor. Updated the MixMonitor documentation for the 'b' option so that it is more obvious that you must not optimize away the Local channel when using this option. (closes issue ASTERISK-13898) Reported by: licedey Tested by: mmichelson, licedey, lmadsen ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=197828 By: Digium Subversion (svnbot) 2009-05-28 18:54:17 Repository: asterisk Revision: 197894 _U  branches/1.6.2/ U   branches/1.6.2/apps/app_mixmonitor.c ------------------------------------------------------------------------ r197894 | lmadsen | 2009-05-28 18:54:17 -0500 (Thu, 28 May 2009) | 16 lines Merged revisions 197828 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r197828 | lmadsen | 2009-05-28 18:04:00 -0400 (Thu, 28 May 2009) | 8 lines    Update documentation in MixMonitor.  Updated the MixMonitor documentation for the 'b' option so that  it is more obvious that you must not optimize away the Local  channel when using this option.    (closes issue ASTERISK-13898)  Reported by: licedey  Tested by: mmichelson, licedey, lmadsen ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=197894 By: Digium Subversion (svnbot) 2009-05-28 18:57:00 Repository: asterisk Revision: 197895 U   branches/1.4/apps/app_mixmonitor.c ------------------------------------------------------------------------ r197895 | lmadsen | 2009-05-28 18:57:00 -0500 (Thu, 28 May 2009) | 7 lines Update MixMonitor documentation. Updated the MixMonitor documentation for the 'b' option so that it is more obvious that you must not optimize awat the Local channel when using this option. (issue ASTERISK-13898) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=197895 By: Digium Subversion (svnbot) 2009-05-28 18:58:37 Repository: asterisk Revision: 197896 U   branches/1.6.0/apps/app_mixmonitor.c ------------------------------------------------------------------------ r197896 | lmadsen | 2009-05-28 18:58:36 -0500 (Thu, 28 May 2009) | 7 lines Update MixMonitor documentation. Updated the MixMonitor documentation for the 'b' option so that it is more obvious that you must not optimize awat the Local channel when using this option. (issue ASTERISK-13898) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=197896 By: Digium Subversion (svnbot) 2009-05-28 18:59:10 Repository: asterisk Revision: 197897 U   branches/1.6.1/apps/app_mixmonitor.c ------------------------------------------------------------------------ r197897 | lmadsen | 2009-05-28 18:59:10 -0500 (Thu, 28 May 2009) | 7 lines Update MixMonitor documentation. Updated the MixMonitor documentation for the 'b' option so that it is more obvious that you must not optimize awat the Local channel when using this option. (issue ASTERISK-13898) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=197897