<p>sungtae kim <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7808">View Change</a></p><p>Patch set 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thank you for kind review. :)</p><p style="white-space: pre-wrap; word-wrap: break-word;">In my understand, it would be better, if the PJSIP response separated Event name for each Action request(PJSIPShowEndpoint, PJSIPShowAuth).</p><p style="white-space: pre-wrap; word-wrap: break-word;">So, the user can recognize it easily what was the request and purpose was.</p><p>(5 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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This event is redundant with the AuthDetail event.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, agree.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But No EndpointName info in the AuthList event.<br>It represent configured auth info only.</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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The AuthList event is the same as the AuthDetail event.  The EndpointName h</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Hm.. got it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But I still thinking, it has to be separated with AuthDetail event. Because it's part of Action: PJSIPShowEndpoint's response message.</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>Provides a listing of all Auths. For each Auth an <literal>AuthList</literal> event<br>                   is raised that contains relevant attributes and status information.  Once all<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">There shouldn't be a line break immediately after the <para> because the CL</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Got it. :)</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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The AuthList event is the same as the AuthDetail event.  The EndpointName h</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Looks better. :)</p><p style="white-space: pre-wrap; word-wrap: break-word;">But, I think it would be worth if the Asterisk sends separated event for each purposes.</p><p style="white-space: pre-wrap; word-wrap: break-word;">AuthDetail is part of another action request's response compare than AuthList.</p></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 CMP_STOP;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">This is an ao2_callback traversal matching function.  This should be CMP_ST</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Good. Fixed it. :)</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: 2 </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-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 07 Jan 2018 15:44:39 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>