[SR-Users] topos and PRACK problem

[Daniel-Constantin Mierla]

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 at gmail.com
>
>
> 2017-04-27 13:04 GMT+03:00 Sergey Basov <sergey.v.basov at gmail.com>:
>> 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 at gmail.com
>>
>>
>> 2017-04-27 12:37 GMT+03:00 Daniel-Constantin Mierla <miconda at gmail.com>:
>>> 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 at gmail.com
>>>>
>>>>
>>>> 2017-04-27 10:30 GMT+03:00 Sergey Basov <sergey.v.basov at gmail.com>:
>>>>> 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 at gmail.com
>>>>>
>>>>>
>>>>> 2017-04-26 17:08 GMT+03:00 Sergey Basov <sergey.v.basov at gmail.com>:
>>>>>> 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 at gmail.com
>>>>>>
>>>>>>
>>>>>> 2017-04-26 16:57 GMT+03:00 Daniel-Constantin Mierla <miconda at gmail.com>:
>>>>>>> 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 at 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
>>>

-- 
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