<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7808">View Change</a></p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p style="white-space: pre-wrap; word-wrap: break-word;">A mention in the Asterisk 15 to 16 CHANGES file section is needed.</p><p>(6 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip.c">File res/res_pjsip.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip.c@2313">Patch Set #1, Line 2313:</a> <code style="font-family:monospace,monospace">       <managerEvent language="en_US" name="AuthList"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This event is redundant with the AuthDetail event.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip.c@2530">Patch Set #1, Line 2530:</a> <code style="font-family:monospace,monospace">        <manager name="PJSIPShowAuths" language="en_US"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The AuthList event is the same as the AuthDetail event.  The EndpointName header is optional to that event.  See res/res_pjsip/config_auth.c:format_ami_auth_handler()</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip.c@2536">Patch Set #1, Line 2536:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">                      <para><br>                  Provides a listing of all Auths. For each Auth an <literal>AuthList</literal> event<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">There shouldn't be a line break immediately after the <para> because the CLI help inserts a space for line wrapping.  See the CLI "manager show command PJSIPShowAuths" output for this paragraph.</p><p style="white-space: pre-wrap; word-wrap: break-word;"><para>Provides a listing..</p></li></ul></li><li><p><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip/config_auth.c">File res/res_pjsip/config_auth.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip/config_auth.c@208">Patch Set #1, Line 208:</a> <code style="font-family:monospace,monospace">static int format_ami_authlist_handler(void *obj, void *arg, int flags)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The AuthList event is the same as the AuthDetail event.  The EndpointName header is optional to that event.</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Thus this function's contents should be:<br>{<br>  return format_ami_auth_handler(obj, arg, flags) ? CMP_STOP : 0;<br>}</pre></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip/config_auth.c@216">Patch Set #1, Line 216:</a> <code style="font-family:monospace,monospace">                return -1;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is an ao2_callback traversal matching function.  This should be CMP_STOP to abort the container traversal.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7808/1/res/res_pjsip/config_auth.c@246">Patch Set #1, Line 246:</a> <code style="font-family:monospace,monospace">        astman_send_listack(s, m, "A listing of Auths follows, presented as AuthList events",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">s/AuthList/AuthDetail/</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7808">change 7808</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/7808"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I1a88a75b38a2b1dd9d1de6c0307b20a3f584c817 </div>
<div style="display:none"> Gerrit-Change-Number: 7808 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 07 Jan 2018 04:14:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>