<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7392">View Change</a></p><p>Patch set 1:</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7392/1/main/bridge_basic.c">File main/bridge_basic.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7392/1/main/bridge_basic.c@3352">Patch Set #1, Line 3352:</a> <code style="font-family:monospace,monospace">            ast_log(LOG_WARNING, "Unable to acquire target extension for attended transfer: '%s@%s'\n", exten, props->context);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I get: </blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">grab_transfer() does add numbers to the exten buffer.  Its purpose is to collect digits to a valid exten for transfer after all.  You do not know why grab_transfer() returned failure.  However, the most likely reason there would be something in the buffer on failure is if the caller hungup after dialing a few digits that didn't match an exten in the context.  It is better to promote the debug messages in grab_transfer() to ast_verb().  For the hangup/error case returned by ast_app_dtget() in grab_transfer() you can add another appropriate ast_verb() message.</p><p style="white-space: pre-wrap; word-wrap: break-word;">grab_transfer() can return error with an empty exten buffer if the user ran out of the configured attempts to enter a number.  That does not mean the user failed to enter digits.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Making the change in grab_transfer() allows blind transfers to benefit as well.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7392">change 7392</a>. To unsubscribe, 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/7392"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Idac66b7628c99379cc9269158377fd87dc97a880 </div>
<div style="display:none"> Gerrit-Change-Number: 7392 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Niklas Larsson <niklas@tese.se> </div>
<div style="display:none"> Gerrit-Reviewer: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Niklas Larsson <niklas@tese.se> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 29 Nov 2017 17:40:14 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>