<p> Attention is currently required from: Mark Petersen. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/16344">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 res/res_pjsip_rfc3326.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/+/16344/comment/8ecc1e5c_35bf1033">Patch Set #1, Line 108:</a> <code style="font-family:monospace,monospace">            if (ast_channel_hangupcause(session->channel) == AST_CAUSE_NOTDEFINED) {</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">initially I just want CANCEL send a valid Reason cause<br>but as Q.850 do not have "0" as a valid value it always need to be replaced</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yeah but with what?  I understand that for CANCEL NORMAL_CLEARING is expected but for other cases where the cause is NOTDEFINED,  NORMAL_CLEARING may not be appropriate.  I'm concerned that if we send NORMAL_CLEARING for any method, we may break something else.  So for now, I'd limit the replacement just for CANCEL methods and if you'd like to follow up with another change to set the cause for other methods, that'd be fine.  I'd also like to know why the hangupcause is not set correctly in the first place.  Should we be fixing this somewhere else?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/16344">change 16344</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/+/16344"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 18 </div>
<div style="display:none"> Gerrit-Change-Id: Ib653aec2282f55b59d87484391cc07c8e6612b89 </div>
<div style="display:none"> Gerrit-Change-Number: 16344 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Mark Petersen <bugs.digium.com@zombie.dk> </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-Attention: Mark Petersen <bugs.digium.com@zombie.dk> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 31 Aug 2021 20:47:08 +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: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Comment-In-Reply-To: Mark Petersen <bugs.digium.com@zombie.dk> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>