Summary: | ASTERISK-22674: [patch]add a channel variable for recording status in mixmonitor application | ||
Reporter: | Deepak Lohani (deepaklohani) | Labels: | patch |
Date Opened: | 2013-10-10 02:01:17 | Date Closed: | |
Priority: | Minor | Regression? | |
Status: | Open/New | Components: | Applications/app_mixmonitor |
Versions: | 1.8.23.1 13.18.4 | Frequency of Occurrence | |
Related Issues: | |||
Environment: | Attachments: | ( 0) diff ( 1) diff ( 2) diff.txt | |
Description: | A channel variable can be added in mixmonitor application to give the recording status. In certain cases where recording is critical for calls, this variable can be used in dialplan to stop dialing or disconnect incoming calls. Status value is set at start of the calls or while writing to the recording file. | ||
Comments: | By: Deepak Lohani (deepaklohani) 2013-10-10 07:20:16.012-0500 This patch provides MIXMONITOR_STATUS channel variable for recording status. This has been used in production for last 5 month and working fine. By: Rusty Newton (rnewton) 2013-10-17 19:27:51.896-0500 1. New features need to be patched on and tested with Trunk of Asterisk and not a release version. (With some small exceptions to Asterisk 12) 2. Your patch won't show up until your license agreement is completed, then you'll need to re-attach. By: Praveen Rustagi (praveen@teckinfo.in) 2013-10-28 23:41:45.066-0500 we have filled up the license agreement on 10/10/2013 but not received any response. can you guide us on where to follow-up. is there any other forum, where we can submit the patch? By: Rusty Newton (rnewton) 2013-11-14 09:54:14.849-0600 I see your patch and license now. I'm opening the issue up. By: Rusty Newton (rnewton) 2013-11-14 09:54:58.322-0600 Have you tested with your patch on trunk? By: Praveen Rustagi (praveen@teckinfo.in) 2013-11-20 04:55:08.784-0600 diff file for branch of 1.8 #svn checkout http://svn.asterisk.org/svn/asterisk/branches/1.8/ #svn diff By: Matt Jordan (mjordan) 2013-11-20 10:59:49.112-0600 As this is a new feature, your patch needs to be written against trunk. By: Deepak Lohani (deepaklohani) 2013-11-27 04:45:05.865-0600 svn checkout http://svn.asterisk.org/svn/asterisk/trunk asterisk Patch tested against trunk. Added manager event "MixMonitorStatus" for recording error By: Deepak Lohani (deepaklohani) 2013-11-30 01:50:54.118-0600 Patch uploaded after testing on trunk. By: Rusty Newton (rnewton) 2013-12-12 17:27:45.993-0600 Thanks Deepak! |