<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/1159/">https://reviewboard.asterisk.org/r/1159/</a>
</td>
</tr>
</table>
<br />
<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, Russell Bryant and Olle E Johansson.</div>
<div>By kkm.</div>
<p style="color: grey;"><i>Updated 2011-04-03 21:36:44.878084</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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 a CHANGES stanza on breaking nature of the change.</pre>
</td>
</tr>
</table>
<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;">There is currently no way to augment a REFER message from Transfer with extra headers. The attached patch implements the feature.
The feature is enabled by default, with a new sip.conf setting to disable it. The rationale for it to be on by default is that it is in fact very controllable from the dialplan: It takes one application call to SIPRemoveHeader with no argument to remove all previously accumulated additional SIP headers. Since Transfer normally terminates the channel, there is no need in practice to keep any SIP headers beyond it in the channel, so that removing these does not impose any dialplan programming complexity.
A hunk near chan_sip.c line 11697 also fixes an issue with Refer-To header gaining an extra set of <> around the address only when retransmitted due to an authentication request.</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;">Confirmed working per spec with sip set debug on packet dump.
Deployed and used on a production server under 1.8.3 and working for 2 weeks already.</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=19059">19059</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/CHANGES <span style="color: grey">(312554)</span></li>
<li>/trunk/channels/chan_sip.c <span style="color: grey">(312554)</span></li>
<li>/trunk/channels/sip/include/sip.h <span style="color: grey">(312554)</span></li>
<li>/trunk/configs/sip.conf.sample <span style="color: grey">(312554)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1159/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>