<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/3998/">https://reviewboard.asterisk.org/r/3998/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 17th, 2014, 4:44 p.m. MDT, <b>Mark Michelson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I'd like to point out that the count fix will also solve issue ASTERISK-24161. Just as a note, when you commit this, the testsuite test channels/pjsip/ami/show_endpoint will start failing in Asterisk 13 and trunk since that test is adjusted to deal with the incorrect ListItems header reported back in the EndpointDetailComplete event. If you look in channels/pjsip/ami/show_endpoint/test-config.yaml and scroll all the way to the bottom, you'll see a XXX comment there explaining the problem. Right now, the test expects ListItems: '6', but that should be changed to ListItems: '7' after this patch goes in.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch is only good for 12 since 13/trunk add the ContactStatusDetail event which will require a slight mod. I'll patch the testsuite when the 13/trunk patch goes up.
</pre>
<br />
<p>- George</p>
<br />
<p>On September 17th, 2014, 5:43 p.m. MDT, George Joseph wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers and Kevin Harwell.</div>
<div>By George Joseph.</div>
<p style="color: grey;"><i>Updated Sept. 17, 2014, 5:43 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/24161">24161</a>,
<a href="https://issues.asterisk.org/jira/browse/24331">24331</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">When no transport is associated to an endpoint, the AMI output for PJSIPShowEndpoint indicates an error instead of silently ignoring the missing transport.
This patch causes the error to appear only if a transport was specified on the endpoint and the transport doesn't exist. It also fixes an issue with counting the objects that were actually found.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Checked the output of PJSIPShowEndpoint to make sure the error appears (or doesn't) correctly and that ListItems is set correctly.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>branches/12/res/res_pjsip_endpoint_identifier_ip.c <span style="color: grey">(423274)</span></li>
<li>branches/12/res/res_pjsip/pjsip_configuration.c <span style="color: grey">(423274)</span></li>
<li>branches/12/res/res_pjsip/location.c <span style="color: grey">(423274)</span></li>
<li>branches/12/res/res_pjsip/config_transport.c <span style="color: grey">(423274)</span></li>
<li>branches/12/res/res_pjsip/config_auth.c <span style="color: grey">(423274)</span></li>
<li>branches/12/include/asterisk/res_pjsip.h <span style="color: grey">(423274)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3998/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>