<p> Attention is currently required from: Joshua Colp. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/18063">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">Patchset:</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/+/18063?tab=comments">Patch Set #1:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I believe this should exist as a separate module, externally, and if it sees enough use then it coul […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This seems like such a basic core thing I couldn't believe there wasn't already a function for it. How else do you suggest doing this? It's not like there's a "workaround" currently.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Dial will bail out if it fails, but at that point it's already two late. A user needs to be able to know if a tech is available before the Dial actually runs. I should not need to do this:</p><p style="white-space: pre-wrap; word-wrap: break-word;">same => n,Dial(PJSIP/something)<br>same => n,ExecIf($["${HANGUPCAUSE}"="20"]?Dial(SIP/something)) ; channel not registered</p><p style="white-space: pre-wrap; word-wrap: break-word;">Besides being clunky, it also emits errors.</p><p style="white-space: pre-wrap; word-wrap: break-word;">A separate module just seems silly considering the function itself is literally 4 lines. And if it's not in the tree (and even if it is), how do you propose "measuring use"? There's just no way to know at that point, those stats don't exist.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm fine with keeping things out of tree at your discretion generally, but I really do think this specifically is such a basic, core thing that it's silly not to be there.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18063">change 18063</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/+/18063"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Idf110464ed69a757b3e0c9d0e8d95d4d24559b46 </div>
<div style="display:none"> Gerrit-Change-Number: 18063 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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-CC: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 22 Feb 2022 17:03:18 +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: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>