<p> Attention is currently required from: Sean Bright, George Joseph. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16083">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File main/pbx_builtins.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/16083/comment/71af0d1a_97a1d21f">Patch Set #2, Line 1273:</a> <code style="font-family:monospace,monospace">       interrupt_string = pbx_builtin_getvar_helper(chan, "SAY_DTMF_INTERRUPT");</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">> The issue happens if the hangup is during ast_say_number, which is line 1300, after this point, so […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Yeah, the goal here is not the channel is not hanging up, it's to not output a confusing warning for no reason just because the channel was hung up while the digits are streamed, so to accomplish that it would need to be located after the ast_say_number, to determine if we should emit an error or not. Before ast_say_number, I'm thinking all that executes so quickly that if it got there, the channel was at least up then. The place where it's likely to not be up is at the end.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16083">change 16083</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/+/16083"/><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: Ieea4a67301c6ea83bbc7690c1d4808d79a704594 </div>
<div style="display:none"> Gerrit-Change-Number: 16083 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 10 Jun 2021 17:53:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>