[sr-dev] Dispatcher broken by recent change?

[Daniel-Constantin Mierla]

Hello,

it is effect of that commit, because it expected pure sip uri, without 
angle brackets. I pushed a fix to shift the start by 1 and decrease 
length by 2 more. Can you try it?

But I'm wondering if it would be safer to parse the value of t->to as To 
header - more processing but safer for further changes in tm local 
request generating.

If anyone wonders, the change was only in master, so no stable branches 
are affected.

Cheers,
Daniel

On 4/26/13 5:54 PM, Peter Dunkley wrote:
> Hello,
>
> I've been running a test system keeping it fairly close to git master.
>
> I did a new build today and installed it and discovered dispatcher was 
> giving me this error:
> Apr 26 15:50:10 ip-10-244-155-159 /usr/sbin/kamailio[1280]: ERROR: 
> dispatcher [dispatch.c:2373]: Setting the state failed 
> (<sip:...hostname...=tc, group 1)
>
> This was working with master as built on 24/04/2013.
>
> Could it be related to this commit (note the missing "p>" from the end 
> of the URI in the log message)?
>
>     Module: sip-router
>     Branch: master
>     Commit: 4e4b1339bfd3a832f5feeb1d2a2380c7455ec82b
>     URL:http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=4e4b1339bfd3a832f5feeb1d2a2380c7455ec82b
>
>     Author: Daniel-Constantin Mierla<miconda at gmail.com>
>     Committer: Daniel-Constantin Mierla<miconda at gmail.com>
>     Date:   Thu Apr 25 14:50:40 2013 +0200
>
>     tm: add angle brackets around From/To URI for local generated requests
>
>
> Regards,
>
> Peter
>
>
>
> _______________________________________________
> sr-dev mailing list
> sr-dev at lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

-- 
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20130426/38ff2312/attachment.html>