Hello, were the two branches both requiring prack?
Cheers, Daniel On 28.04.17 08:47, Sergey Basov wrote: > 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 >>> -- 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