<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/1197/">https://reviewboard.asterisk.org/r/1197/</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.</div>
<div>By jrose.</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;">This is a patch submitted by Igor to resolve a URI encoding issue with pedantic mode. Encoding the fromuser string isn't particularly appropriate to the URI since it eliminates non-ASCII UTF-8 characters, which are allowed according to RFC 3261. Since ast_uri_encode is described as being "encoding agnostic", for the time being I have left this as is. I'm not sure if it's especially safe though since there might be times with the fed string isn't UTF-8... in which case the SIP message might end up screwy.
Right now I'm wondering if it would be prudent to add another function to utils.h to drop characters that don't fit in UTF-8 and perform that on l instead of the function that got dropped 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;">Patch off:
SIP general context set to pedantic mode.
Created a DAHDI group with caller ID = "Örn Örnyman"
Used phone connected to Örn Örnyman to dial a second asterisk box (with sip debug) through SIP.
Caller ID in from field was escaped at the Ös (Something like %C396) on SIP debug
Patch on:
Same as above, except from field came out as:
From: "Örn Örnyman" <sip:asterisk@10.24.20.132>;tag=as13dc4c39
Note though that on my crumby grandstream phone the final caller id output was:
Ãrn Ãrnyman (c383 by unicode standards, but it is probably just the phone being stupid.)</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=18298">18298</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>/branches/1.8/channels/chan_sip.c <span style="color: grey">(314904)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1197/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>