Hi Daniel. I does not see problem with parallel forking to 2 destinations.
I will send 2 dumps in private email. -- Best regards, Sergey Basov e-mail: sergey.v.ba...@gmail.com 2017-04-27 15:34 GMT+03:00 Sergey Basov <sergey.v.ba...@gmail.com>: > 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.ba...@gmail.com > > > 2017-04-27 15:12 GMT+03:00 Daniel-Constantin Mierla <mico...@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.ba...@gmail.com >>> >>> >>> 2017-04-27 13:04 GMT+03:00 Sergey Basov <sergey.v.ba...@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.ba...@gmail.com >>>> >>>> >>>> 2017-04-27 12:37 GMT+03:00 Daniel-Constantin Mierla <mico...@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.ba...@gmail.com >>>>>> >>>>>> >>>>>> 2017-04-27 10:30 GMT+03:00 Sergey Basov <sergey.v.ba...@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.ba...@gmail.com >>>>>>> >>>>>>> >>>>>>> 2017-04-26 17:08 GMT+03:00 Sergey Basov <sergey.v.ba...@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.ba...@gmail.com >>>>>>>> >>>>>>>> >>>>>>>> 2017-04-26 16:57 GMT+03:00 Daniel-Constantin Mierla >>>>>>>> <mico...@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@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 >> _______________________________________________ Kamailio (SER) - Users Mailing List sr-users@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users