[Home]

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:17Date Closed:
Priority:MinorRegression?
Status:Open/NewComponents: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!