<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/1163/">https://reviewboard.asterisk.org/r/1163/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 7th, 2011, 1:38 a.m., <b>Olle E Johansson</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 think this is a great addition. A few comments:
- This should apply to ALL channels, not only SIP. I need descriptions also for PRI channels.
- You forgot to add it to the manager "sip show peer/Peers"
- People will quickly ask for "sip show peer described as sexy" CLI commands.
</pre>
</blockquote>
<p>On April 7th, 2011, 6:39 a.m., <b>lathama</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 feature already exists in users.conf. It may be good to keep the naming of "fullname = New User" unless I am missing something. It does update the callerID.
*CLI> sip show peer 1234
...
Callerid : "Andrew Latham" <1234>
...</pre>
</blockquote>
<p>On April 7th, 2011, 12:59 p.m., <b>Leif Madsen</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;">If fullname in users.conf updates the callerid, then I would say they aren't really the same thing. I like 'description' a lot more because it's a more general option name rather than 'fullname' which seems to imply a callerID type name.</pre>
</blockquote>
<p>On April 7th, 2011, 1:16 p.m., <b>lathama</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 do agree with you. I just wanted to make sure that duplication did not happen. Is this description field also parsed from the users.conf for DAHDI, SIP, IAX and H323?</pre>
</blockquote>
<p>On April 7th, 2011, 1:18 p.m., <b>Leif Madsen</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'm not exactly sure how users.conf fits in with all of this, but I didn't specifically add any code for users.conf.</pre>
</blockquote>
<p>On April 7th, 2011, 1:31 p.m., <b>lathama</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;">Looks like it should work fine with users.conf I am applying and testing. If so you may want to add the note there too.</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;">:) Works from users.conf too. Maybe copy your sip.conf comments to users.conf.</pre>
<br />
<p>- lathama</p>
<br />
<p>On April 7th, 2011, 1:13 p.m., Leif Madsen wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers and Olle E Johansson.</div>
<div>By Leif Madsen.</div>
<p style="color: grey;"><i>Updated 2011-04-07 13:13:34</i></p>
<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;">Using some of the best practices names for SIP peers provides the quandary of knowing exactly what peer is where, or who it is assigned to.
If you name all your peers as the MAC address of the device, then it's difficult to determine what device is assigned to whom, or at least knowing where the device physically resides.
I've added a description field that can help. When you do 'sip show peers' then a new column called Description outputs 32 chars of a description to the Asterisk console.</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;">Added "description=My Awesome Peer" to [0004f2040001] definition in sip.conf, then did 'sip show peers' and 'sip show peer 0004f2040001'. Output was as expected. Compiled without error or warning.</pre>
</td>
</tr>
</table>
<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/view.php?id=19076">19076</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>trunk/CHANGES <span style="color: grey">(313091)</span></li>
<li>trunk/channels/chan_dahdi.c <span style="color: grey">(313091)</span></li>
<li>trunk/channels/chan_iax2.c <span style="color: grey">(313091)</span></li>
<li>trunk/channels/chan_sip.c <span style="color: grey">(313091)</span></li>
<li>trunk/channels/sip/include/sip.h <span style="color: grey">(313091)</span></li>
<li>trunk/configs/chan_dahdi.conf.sample <span style="color: grey">(313091)</span></li>
<li>trunk/configs/iax.conf.sample <span style="color: grey">(313091)</span></li>
<li>trunk/configs/sip.conf.sample <span style="color: grey">(313091)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1163/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>