<div dir="ltr"><div><div><div>I see a similar problem in $uac_req(furi):<br><br></div>In 4.0 the following configuration $uac_req(furi) = "\"My Name\" <sip:MyNumber@" + $Ri + ":" + $Rp + ">"; produced:<br>
From: My Name <<a href="http://sip:MyNumber@0.0.0.0:5062">sip:MyNumber@0.0.0.0:5062</a>><br><br></div>Nowadays, in dev code it is puts '<' char at the beginning, changing case and so on, example:<br><br>
$uac_req(furi) = "\"My Name\" <sip:MyNumber@" + $Ri + ":" + $Rp + ">"; produced:<br>
From: <"my name" <<a href="http://sip:MyNumber@0.0.0.0:5062">sip:MyNumber@0.0.0.0:5062</a>><br><br>$uac_req(furi) = "\"My Name\" sip:MyNumber@" + $Ri + ":" + $Rp + ""; produced:<br>
From: <"my name" <a href="http://sip:MyNumber@0.0.0.0:5062">sip:MyNumber@0.0.0.0:5062</a>><br><br></div>However, it is works correct when I do not specify a name: <br>$uac_req(furi) = "sip:MyNumber@" + $Ri + ":" + $Rp; produced:<br>
From: <<a href="http://sip:MyNumber@0.0.0.0:5062">sip:MyNumber@0.0.0.0:5062</a>><br><div><br><br><br><br><br><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/5/15 Daniel-Constantin Mierla <span dir="ltr"><<a href="mailto:miconda@gmail.com" target="_blank">miconda@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
Hello,<br>
<br>
it is ok from my point of view, it looks like a bug otherwise. But
please add notes in the docbook/readme documentation about this
change for each of these functions.<br>
<br>
Cheers,<br>
Daniel<div><div class="h5"><br>
<br>
<br>
<div>On 5/15/13 2:36 PM, Vicente Hernando
wrote:<br>
</div>
</div></div><blockquote type="cite"><div><div class="h5">Hello,
<br>
<br>
uac_replace_to and uac_replace_from functions can receive two
arguments, display and uri.
<br>
<br>
In current implementation two double quotes are automatically
added to display during fixup.
<br>
<br>
<br>
Would it be a better behavior not to add any quotes? If the user
wants some quotes he should add them by himself.
<br>
<br>
E.g:
<br>
<br>
right now, if I get a display "Vicente" from an invite and store
it in $fn variable, if I call
<br>
<br>
uac_replace_from($fn, uri_foo) it produces ""Vicente"" in the
display which kamailio complaints about.
<br>
<br>
<br>
If we would avoid quotes, uac_replace_from function could accept:
<br>
<br>
uac_replace_from("Bob", uri) display = Bob
<br>
<br>
uac_replace_from("\"Bob\"", uri) display = "Bob"
<br>
<br>
<br>
standard supports quoted strings, not quoted tokens, spaces before
and after quotes, etc.
<br>
<br>
<br>
Attached patch should avoid quotes in uac_replace functons.
<br>
<br>
<br>
What are your opinions about this?
<br>
<br>
Kind regards,
<br>
Vicente.
<br>
<br>
<br>
<fieldset></fieldset>
<br>
</div></div><pre>_______________________________________________
sr-dev mailing list
<a href="mailto:sr-dev@lists.sip-router.org" target="_blank">sr-dev@lists.sip-router.org</a>
<a href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev" target="_blank">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a><span class="HOEnZb"><font color="#888888">
</font></span></pre><span class="HOEnZb"><font color="#888888">
</font></span></blockquote><span class="HOEnZb"><font color="#888888">
<br>
<pre cols="72">--
Daniel-Constantin Mierla - <a href="http://www.asipto.com" target="_blank">http://www.asipto.com</a>
<a href="http://twitter.com/#%21/miconda" target="_blank">http://twitter.com/#!/miconda</a> - <a href="http://www.linkedin.com/in/miconda" target="_blank">http://www.linkedin.com/in/miconda</a>
Kamailio Advanced Training, San Francisco, USA - June 24-27, 2013
* <a href="http://asipto.com/u/katu" target="_blank">http://asipto.com/u/katu</a> *</pre>
</font></span></div>
<br>_______________________________________________<br>
sr-dev mailing list<br>
<a href="mailto:sr-dev@lists.sip-router.org">sr-dev@lists.sip-router.org</a><br>
<a href="http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev" target="_blank">http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev</a><br>
<br></blockquote></div><br></div>