<div dir="ltr"><div><br></div><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 7, 2014 at 5:03 PM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank">mjordan@digium.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Tue, Jan 7, 2014 at 9:27 AM, Joshua Colp <<a href="mailto:jcolp@digium.com">jcolp@digium.com</a>> wrote:<br>

> Olle E. Johansson wrote:<br>
>><br>
>> On 07 Jan 2014, at 15:55, SVN commits to the Digium<br>
>> repositories<<a href="mailto:svn-commits@lists.digium.com">svn-commits@lists.digium.com</a>>  wrote:<br>
>><br>
>>> if (endpoint->nat.rewrite_contact&&  (contact =<br>
>>> pjsip_msg_find_hdr(rdata->msg_info.msg, PJSIP_H_CONTACT, NULL))&& -<br>
>>><br>
>>> (PJSIP_URI_SCHEME_IS_SIP(contact->uri) ||<br>
>>> PJSIP_URI_SCHEME_IS_SIPS(contact->uri))) { +            !contact->star&&<br>
>>> (PJSIP_URI_SCHEME_IS_SIP(contact->uri) ||<br>
>>> PJSIP_URI_SCHEME_IS_SIPS(contact->uri))) { pjsip_sip_uri *uri =<br>
>>> pjsip_uri_get_uri(contact->uri);<br>
>>><br>
>>> pj_cstr(&uri->host, rdata->pkt_info.src_name);<br>
>>><br>
>><br>
>> Seems like this code assumes that SIP only have SIP: and SIPS: as<br>
>> URI's. We should actually be quite transparent in the schemas<br>
>> supported - especially proxys but also b2bua's like Asterisk. Tel:<br>
>> uri's are not unknown. In the security area we are discussing<br>
>> improved end-2-end security which may end up using a new SIP uri.<br>
><br>
><br>
> Hrm, I've been trying to think of the best place to express this information<br>
> but I'm coming up empty. I'm not sure it's something for the config<br>
> documentation, maybe more the wiki.<br>
><br>
><br>
>> Instead of testing with two functions for two classes of schemas a<br>
>> table could be used? That would be more extensible. And please<br>
>> implement tel: support :-)<br>
><br>
><br>
> The code above stems from PJSIP itself and is used to determine what<br>
> structure PJSIP has used to store the parsed URI for manipulation. I'm not<br>
> sure a table would buy us anything in this area. As a developer you would<br>
> still need to know what structure was used and what it looks like.<br>
><br>
<br>
</div></div>We probably should have an issue made to (finally) implement TEL<br>
support. I know PJSIP supports it, but there'd obviously be some<br>
subtle things that would have to change in how we make use of PJSIP.<br>
<br>
I'll make an issue for it later today.<br>
<br>
--<br>
Matthew Jordan<br>
Digium, Inc. | Engineering Manager<br>
<div class="im">445 Jan Davis Drive NW - Huntsville, AL 35806 - USA<br>
</div>Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> & <a href="http://asterisk.org" target="_blank">http://asterisk.org</a><br>
<div class=""><div class="h5"><br>
--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a></div></div></blockquote><div><br></div><div><br></div><div><br></div>I love this fix, as the issue it fixes is causing my Asterisk 12 install to die...<div>
<br></div><div><a href="https://gist.github.com/danjenkins/e604c978b2a803140dce">https://gist.github.com/danjenkins/e604c978b2a803140dce</a></div></div><br></div><div class="gmail_extra">Any idea when the next point release will be? Is it so far off I should just recompile against the 12 branch later on?</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Dan</div></div>