<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/3043/">https://reviewboard.asterisk.org/r/3043/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 20th, 2013, 7:35 p.m. UTC, <b>rnewton</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;">For "pjsip show endpoints", a formatting issue:
<Endpoint/CID................................................> <State.....> <Channels.>
I/OAuth: <AuthId/UserName...........................................................>
Aor: <Aor............................................> <MaxContact>
Contact: <Aor/ContactUri...............................> <Status....> <RTT(ms)..>
Identify: <MatchList.................................................................>
Channel: <ChannelId......................................> <State.....> <Time(sec)>
Codec: <Codec> Exten: <DialedExten...........> CLCID: <ConnectedLineCID.......>
=========================================================================================
6001 In use 1 of inf
InAuth: 6001/6001
Aor: 6001 2
Contact: 6001/sip:6001@10.24.18.16:5060;ob Avail 3.739
Channel: PJSIP/6001-00000000/Dial Up 497
Codec: (ulaw) Exten: 6002 CLCID: "" <>
For the "Channel:" line, the "State" and "Time(sec)" values are out of their columns and the State is pushed up against the Channel ID to it's left.</pre>
</blockquote>
<p>On December 20th, 2013, 7:39 p.m. UTC, <b>rnewton</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;">This alignment issue also appears to apply to "pjsip show channels"</pre>
</blockquote>
<p>On December 20th, 2013, 7:40 p.m. UTC, <b>rnewton</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;">and "pjsip list channels"</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;">I updated to latest for the pjsip-cli branch and these all appear fixed</pre>
<br />
<p>- rnewton</p>
<br />
<p>On December 20th, 2013, 4:24 a.m. UTC, 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.</div>
<div>By George Joseph.</div>
<p style="color: grey;"><i>Updated Dec. 20, 2013, 4:24 a.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/ASTERISK-22610">ASTERISK-22610</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;">Implements the following cli commands:
pjsip list aors
pjsip list auths
pjsip list channels
pjsip list contacts
pjsip list endpoints
pjsip show aor(s)
pjsip show auth(s)
pjsip show channels
pjsip show endpoint(s)
Also...
Minor modifications made to the AMI command implementations to facilitate reuse.
New function ast_variable_list_sort added to config.c and config.h to implement variable list sorting.
New api ast_sip_for_each_identify added to module res_pjsip_endpoint_identifier_ip. Required new files res_pjsip_endpoint_identifier_ip.h and res_pjsip_endpoint_identifier_ip.exports.in.
Implementation of the summary statistics is still pending.
I'm sure there will be lots of feedback here. :)</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;">I've gone through all the combinations of actions and objects and made sure at least the correct objects are returned, there are no segfaults, errors, etc.
I *think* the formatting is correct but I'll need some feedback.</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_registrar.c <span style="color: grey">(404396)</span></li>
<li>branches/12/res/res_pjsip_endpoint_identifier_ip.exports.in <span style="color: grey">(PRE-CREATION)</span></li>
<li>branches/12/res/res_pjsip_endpoint_identifier_ip.c <span style="color: grey">(404396)</span></li>
<li>branches/12/res/res_pjsip/pjsip_configuration.c <span style="color: grey">(404396)</span></li>
<li>branches/12/res/res_pjsip/pjsip_cli.c <span style="color: grey">(PRE-CREATION)</span></li>
<li>branches/12/res/res_pjsip/location.c <span style="color: grey">(404396)</span></li>
<li>branches/12/res/res_pjsip/include/res_pjsip_private.h <span style="color: grey">(404396)</span></li>
<li>branches/12/res/res_pjsip/config_auth.c <span style="color: grey">(404396)</span></li>
<li>branches/12/main/sorcery.c <span style="color: grey">(404396)</span></li>
<li>branches/12/main/config.c <span style="color: grey">(404396)</span></li>
<li>branches/12/include/asterisk/sorcery.h <span style="color: grey">(404396)</span></li>
<li>branches/12/include/asterisk/res_pjsip_endpoint_identifier_ip.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>branches/12/include/asterisk/res_pjsip_cli.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>branches/12/include/asterisk/res_pjsip.h <span style="color: grey">(404396)</span></li>
<li>branches/12/include/asterisk/config.h <span style="color: grey">(404396)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3043/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>