[asterisk-dev] [Code Review] Stop trying to uri_encode the display name for the caller ID

wdoekes reviewboard at asterisk.org
Mon May 30 01:34:19 CDT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1235/#review3653
-----------------------------------------------------------



/branches/1.8/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/1235/#comment7337>

    Space before If.



/branches/1.8/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/1235/#comment7338>

    Here the double-quote should be backslash-escaped in the cli-name. Simply removing the ast_uri_encode is not good enough.


- wdoekes


On 2011-05-27 12:35:43, jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1235/
> -----------------------------------------------------------
> 
> (Updated 2011-05-27 12:35:43)
> 
> 
> Review request for Asterisk Developers, Russell Bryant and David Vossel.
> 
> 
> Summary
> -------
> 
> Under SIP pedantic mode, this function was encoding strings meant for the display name into into a URI safe format when this was not specified in the SIP RFCs.  I also took the liberty of adding a little commentary.
> 
> pedantic mode was the default in Asterisk 1.8+
> 
> 
> This addresses bug 18298.
>     https://issues.asterisk.org/view.php?id=18298
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/channels/chan_sip.c 321272 
> 
> Diff: https://reviewboard.asterisk.org/r/1235/diff
> 
> 
> Testing
> -------
> 
> A number of different calls with user names including more interesting UTF-8 characters from a variety of characters from a variety of sources including trema (like German umlaut characters), Kanji (Japanese pictographic characters), Sanskrit (phonetic characters for one of India's languages), and Cyrillic (Russian).  Whether or not they'll display properly depends on the receiving phone... My Grandstream phone doesn't like the higher end UTF-8 characters, but all of my soft phones read them fine.
> 
> Testing involved a manually set DAHDI channel to have odd caller ID and an incoming call from another Asterisk Box sending SIP from a phone set to also have a display name with high UTF-8 chars.
> 
> 
> Thanks,
> 
> jrose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110530/07080e89/attachment.htm>


More information about the asterisk-dev mailing list