<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/3816/">https://reviewboard.asterisk.org/r/3816/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 17th, 2014, 9 p.m. UTC, <b>Jonathan Rose</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://reviewboard.asterisk.org/r/3816/diff/2/?file=64708#file64708line1539" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/main/bridge_basic.c</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">1539</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="tb"> </span><span class="tb"> </span><span class="tb"> </span><span class="n">ast_log</span><span class="p">(</span><span class="n">LOG_NOTICE</span><span class="p">,</span> <span class="s">"Non-peered party %s</span><span class="se">\n</span><span class="s">"</span><span class="p">,</span> <span class="n">ast_channel_name</span><span class="p">(</span><span class="o">*</span><span class="n">party</span><span class="p">));</span></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Should this be a NOTICE log message? It appears to occur kind of deeply and might be better as debug.</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yep, all of these were extra debugging messages that I left in. As-is, they definitely shouldn't be NOTICEs, and they probably should be more detailed if they're going to be debug messages, too. I'll either make them more detailed or just remove them altogether.</pre>
<br />
<p>- Mark</p>
<br />
<p>On July 17th, 2014, 6:01 p.m. UTC, Mark Michelson 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 Mark Michelson.</div>
<p style="color: grey;"><i>Updated July 17, 2014, 6:01 p.m.</i></p>
<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;">Transfer publications in Stasis have, in the past given transferer channels and the bridges they belong to. The theory behind this is that if given that information, and you have kept track of who all has entered and left the associated bridges, you can derive who the transferee channels are and who the transfer target channels are. In practice, however, it's more useful to present this information directly in the publication.
This changeset adds transferee channel snapshots to blind and attended transfer publications and a transfer target channel snapshot to the blind transfer publication. We only provide such snapshots if there is a single transferee or transfer target. If transferring multiple parties or transferring to multiple parties, these snapshots are omitted.
One thing to check for is whether consumers of the transfer events have been properly updated to make use of the new information being added in this review (I'm looking at you, ARI!).</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;">Testing on this is pretty paltry at the moment. All I have done is to perform manual tests of nominal cases. I can confirm that AMI now prints transferee channel information for both blind and attended transfers. I can also confirm that AMI prints transfer target information for attended transfers.
Due to the looming deadline of the feature freeze, plus the stipulation that the code only needs to be up for review to be a candidate for inclusion, I have posted this without any testsuite tests. This will not be committed without either modifying existing tests or adding new tests, though.</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>/trunk/res/stasis/app.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/stasis_bridges.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/cel.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/bridge_basic.c <span style="color: grey">(418633)</span></li>
<li>/trunk/main/bridge.c <span style="color: grey">(418633)</span></li>
<li>/trunk/include/asterisk/stasis_bridges.h <span style="color: grey">(418633)</span></li>
<li>/trunk/apps/app_queue.c <span style="color: grey">(418633)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3816/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>