Hello,
although I just looked briefly at the patch, I think that works in case
of a single branch sent out, but if there is going to be a parallel
forking to two or more destinations, this is not going to work.
Cheers,
Daniel
On 27.04.17 13:33, Sergey Basov wrote:
Hi, Daniel.
Seems I found how to fix PRACK handling.
It works for me.
please lock at
https://github.com/kamailio/kamailio/pull/1097
Thank you.
--
Best regards,
Sergey Basov e-mail: sergey.v.basov(a)gmail.com
2017-04-27 13:04 GMT+03:00 Sergey Basov <sergey.v.basov(a)gmail.com>om>:
> Yes, you are right.
>
> But now before 200 OK there empty field b_contact.
>
> May be you does not populate it from contact in 183 Progress?
>
> I see that this field is not empty only after 200 OK with a Contact field.
> --
> Best regards,
> Sergey Basov e-mail: sergey.v.basov(a)gmail.com
>
>
> 2017-04-27 12:37 GMT+03:00 Daniel-Constantin Mierla <miconda(a)gmail.com>om>:
>> Hello,
>>
>> thanks for troubleshooting further. I haven't got the time to look at
>> the source code, but I expect that the b-leg attributes (contact, record
>> routes) to be set on 200ok for dialog (topos_d). I think for PRACK, the
>> routing information should be stored and taken from transaction (topos_t).
>>
>> Cheers,
>> Daniel
>>
>> On 27.04.17 11:29, Sergey Basov wrote:
>>> Hi, Daniel.
>>>
>>> I just done one more test topos with re-invite which comes from caller
>>> to callee, same direction as PRACK.
>>>
>>> And I found that, in case of prack, b_contact field is empty...
>>>
>>> Please find attached debug=3 part of re-invite message.
>>>
>>> Hope it helps.
>>> --
>>> Best regards,
>>> Sergey Basov e-mail: sergey.v.basov(a)gmail.com
>>>
>>>
>>> 2017-04-27 10:30 GMT+03:00 Sergey Basov <sergey.v.basov(a)gmail.com>om>:
>>>> Hi, Daniel
>>>>
>>>> Please look at attached part of debug=3 while receiving and parsing
PRACK.
>>>> At line 208 seems rr module does not find correct part of record_route
>>>> which in DB consists from 2 parts.
>>>>
>>>> So at line 219 and later uac module cannot restore uris.
>>>>
>>>> Thank you.
>>>> --
>>>> Best regards,
>>>> Sergey Basov e-mail: sergey.v.basov(a)gmail.com
>>>>
>>>>
>>>> 2017-04-26 17:08 GMT+03:00 Sergey Basov
<sergey.v.basov(a)gmail.com>om>:
>>>>> Thanks for workaround.
>>>>>
>>>>> But I will wait for you solution )
>>>>>
>>>>> I ready for testing )
>>>>>
>>>>> Thank you Daniel for your work!
>>>>>
>>>>> --
>>>>> Best regards,
>>>>> Sergey Basov e-mail: sergey.v.basov(a)gmail.com
>>>>>
>>>>>
>>>>> 2017-04-26 16:57 GMT+03:00 Daniel-Constantin Mierla
<miconda(a)gmail.com>om>:
>>>>>> Hello,
>>>>>>
>>>>>>
>>>>>> On 26.04.17 14:53, Sergey Basov wrote:
>>>>>>> Hi All.
>>>>>>>
>>>>>>> I have just try to test topos with GW which requires PRACK.
>>>>>>>
>>>>>>> As you can see UA at packet 21 send PRACK to topos contact,
but after
>>>>>>> topos, on other kamailio side in PRACK request line present
not
>>>>>>> kontact but record-route header.
>>>>>>>
>>>>>>> Can you fix it?
>>>>>>>
>>>>>>>
>>>>>> probably needs to look into the code. If you need a quick
workaround,
>>>>>> try to remove Supported header from INVITE so the callee should
no
>>>>>> longer Require 100rel.
>>>>>>
>>>>>> Cheers,
>>>>>> Daniel
>>>>>>
>>>>>> --
>>>>>> Daniel-Constantin Mierla
>>>>>>
www.twitter.com/miconda --
www.linkedin.com/in/miconda
>>>>>> Kamailio Advanced Training - May 22-24 (USA) -
www.asipto.com
>>>>>> Kamailio World Conference - May 8-10, 2017 -
www.kamailioworld.com
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> Kamailio (SER) - Users Mailing List
>>>>>> sr-users(a)lists.kamailio.org
>>>>>>
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
>> --
>> Daniel-Constantin Mierla
>>
www.twitter.com/miconda --
www.linkedin.com/in/miconda
>> Kamailio Advanced Training - May 22-24 (USA) -
www.asipto.com
>> Kamailio World Conference - May 8-10, 2017 -
www.kamailioworld.com
>>