[SR-Users] topos and PRACK problem

[Sergey Basov]

May be, but there is one more problem.

I just test call to CSIPSimple softphone, it does not returm 183, it
just return 180 Ringing.Call was not answered for some time, and I
decide to terminate it from the caller side.
BYE massege goes wrong, to recor-route value, as PRACK before, because
contact from CSIPSimple side was not get from 180, call was not
answered so 200 OK was not send.
And topos doea not have contact value to send by to CSIPSimple.

I will send dump to your private e-mail, it has real IP adreses.

Thank you.
--
Best regards,
Sergey Basov                     e-mail: sergey.v.basov at gmail.com


2017-04-27 15:12 GMT+03:00 Daniel-Constantin Mierla <miconda at gmail.com>:
> 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
>