Review Board 1.7.16


Allow passing options and command to MixMonitor when recording in ConfBridge

Review Request #4023 - Created Sept. 25, 2014 and submitted

gareth
ASTERISK-24351
Reviewers
asterisk-dev
Asterisk
This patch adds the ability to pass options and a command to MixMontor when recording a conference using ConfBridge.

New options are -

record_options: Options to MixMontor, eg: m(), W() etc.
record_command: The command to execute when recording is over.

eg: Set(CONFBRIDGE(bridge,record_command)=/path/to/command ^{MIXMONITOR_FILENAME}))

Note:

The current behavior of set_rec_filename is to always append a timestamp to rec_file. This is desirable for dynamically generated rec_file, but doesn't make sense for a user-supplied rec_file and the documenation does not mention that it does this either.

So the patch changes set_rec_filename to use the user-supplied rec_file as-is. 
Set record_options to m(${MAILBOX}) and verified that a recording was delivered to ${MAILBOX}.

Set record_command to /bin/rm ^{MIXMONITOR_FILENAME} and checked that recording was deleted on ending the conference.
Total:
1
Open:
0
Resolved:
1
Dropped:
0
Status:
From:
Review request changed
Updated (Dec. 21, 2014, 9:23 p.m.)
  • changed from pending to submitted
Committed in revision 429951

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.