<p>Sean Bright <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/19300">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">app_queue.c: Don't escape commas in MONITOR_FILENAME.<br><br>The contents of the MONITOR_FILENAME variable is evaluated when a<br>queue member is connected to a caller, allowing information about the<br>member to be included in the filename.<br><br>This is made possible by using ^{MEMBERINTERFACE} syntax instead of<br>${MEMBERINTERFACE} when assigning a value to MONITOR_FILENAME so that<br>evaluation does not occur immediately. The MONITOR_EXEC variable is<br>handled similarly.<br><br>Because of the way that MONITOR_EXEC works under the hood, the code<br>that performs the variable replacement also escapes commas before<br>doing so. However, this replacement should be taking place after the<br>variable replacement is done so that unescaped commas are not<br>erroneously passed to the MixMonitor start function.<br><br>Finally, having a comma in a filename is not problematic so we now<br>only escape commas after variable replacement is performed on<br>MONITOR_EXEC.<br><br>The main consequence of this change is that you can now use dialplan<br>functions with multiple arguments in MONITOR_FILENAME without the<br>commas being escaped before variable replacement occurs.<br><br>ASTERISK-30236 #close<br><br>Change-Id: Ib18481288c5292e27321e95364d7e63e8917252f<br>---<br>M apps/app_queue.c<br>A doc/CHANGES-staging/app_queue_monitor_filename_commas.txt<br>2 files changed, 58 insertions(+), 6 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/00/19300/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19300">change 19300</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/19300"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ib18481288c5292e27321e95364d7e63e8917252f </div>
<div style="display:none"> Gerrit-Change-Number: 19300 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-CC: Friendly Automation </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>