Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-11-08 Thread Sergey Lavrov
ok thanks for info guys! will wait 5.1 release then. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-342748359___ Kamai

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-11-07 Thread Daniel-Constantin Mierla
The topos module got major refactoring in the master branch (to have support for other backends, redis (see topos_redis) being the one added along with the old support for db). I am not sure these patches can be backported without a consistent work. If someone does it and makes a pull request fo

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-11-07 Thread sergey-vb
Hi All. Looks like Daniel forgot to merge some commits to 5.0 branch. Latest commit to topos module in 5.0 branch is on May 8, 2017 Fixes for correct PRACK processing was committed after that date... Mostly in August. @slavrov can you compile master branch and perform tests? For me it works.

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-11-07 Thread Sergey Lavrov
Dear Daniel! Just compiled kamailio 5.0.4 and still have problem with PRACK packets. I sent SIP dump to your email. Hope this helps. Thanks in advance! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamai

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-14 Thread Daniel-Constantin Mierla
@sergey-vb no worries, thanks for testing troubleshooting! I am closing it and if there are issues, reopen or open a new issue, as you consider better. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kama

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-14 Thread Daniel-Constantin Mierla
Closed #1167. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#event-1205122263___ Kamailio (SER) - Development Mailing List sr-dev@l

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-14 Thread Sergey Lavrov
I will make a try and will report in a few days. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-322202217___ Kamailio

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-14 Thread sergey-vb
Sorry, Daniel It was my fault... After commit fbf8f68ed1f8df9af7ed60cb14d379e571550a03 PRACK is handled properly. @slavrov can you make tests with latest master? Thank you. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: ht

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-14 Thread Daniel-Constantin Mierla
I see log messages that are not in the default source code: ``` Aug 14 14:29:30 csbc-uat /usr/sbin/kamailio[4315]: ERROR: topos [tps_msg.c:766]: tps_request_received(): sbasov load_branch done ``` Can you test with fresh master branch? Otherwise I cannot correlate logs with source code. The R

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-14 Thread sergey-vb
Still same, as before. PRACK is construct incorrect. uri in PRACK after topos is first record-route part, not contact. record route restores incorrect to. Thank you. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://g

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-14 Thread Daniel-Constantin Mierla
The Contact field was not loaded for transaction record and R-URI was not changed, resulting in doing strict routing instead of loose routing. Can you try again? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://githu

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-11 Thread sergey-vb
Send result in email. Better but route headers are restored in wrong order. and uri for send prack seems got from record-route header... -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/iss

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-11 Thread Daniel-Constantin Mierla
Test one more with latest branch. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-321745779___ Kamailio (SER) - Develop

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-10 Thread sergey-vb
Send it via email -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-321559298___ Kamailio (SER) - Development Mailing Lis

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-10 Thread Daniel-Constantin Mierla
Can you add following xlog at thet top of request_route, redo the last test and send again all the troubleshooting info? ``` xlog("received from $si: [[$mb]]\n"); ``` I want to see the requests as constructed by topos after doing the request received callback. The pcap doesn't capture that. --

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-08 Thread sergey-vb
Hi Daniel. Pcap, db content and log with level 3 you can find in attach. PRACK is now generated but to the wrong uri... Details send to you via email. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamai

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-08 Thread Daniel-Constantin Mierla
So I'm getting close and closer ... another patch pushed, try again ... -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-320990905__

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-08 Thread sergey-vb
All same exept in logs Aug 8 16:47:08 csbc-uat /usr/sbin/kamailio[21308]: DEBUG: db_mysql [km_res.c:168]: db_mysql_convert_rows(): no rows returned from the query Aug 8 16:47:08 csbc-uat /usr/sbin/kamailio[21308]: DEBUG: topos [tps_storage.c:898]: tps_db_load_branch(): no stored record for **I

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-08 Thread Daniel-Constantin Mierla
Another related patch landed in master, give it another try and on issue, send a report as before. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-320940625__

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-07 Thread sergey-vb
Hi, Daniel. I just send you requested info with email. There no record about PRACK in topos_t table now... -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-3206684

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-08-07 Thread Daniel-Constantin Mierla
Finally pushed a new patch related to this. Can you test with latest master branch? If there is something not working properly, to get all needed data and have privacy, send me directly via email the pcap of such call along with the records from both topos database tables. Just make a comment h

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-07-31 Thread Daniel-Constantin Mierla
Hope to get to it in the next few days ... with the summer holidays around, I didn't get to work that long in a raw at the desk to put a testing scenario in place. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://git

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-07-31 Thread sergey-vb
Daniel, do you have some updates regarding this issue? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-319022204___ Kam

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-07-20 Thread sergey-vb
Hi Daniel This is the output from topos_t *** 29. row *** id: 1145715 rectime: 2017-07-21 08:14:54 s_method: INVITE s_cseq: 9022 a_callid: uACs6xvwpwRYZyfNDsRi2Z.3Ge809DJK a_uuid: atpsh-5967680b-2b6d-5 b_uuid: btpsh-596768

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-07-20 Thread Daniel-Constantin Mierla
Hello, is more useful to get the record from database as text instead of a screenshot. You can use: ``` select * from topos_t\G ``` That will make records with many columns to be printed in more compact form suitable for pasting here. Can you also tell the IP addresses for each node in the pa

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-07-18 Thread sergey-vb
Hi, Daniel. Do you have any updates regarding this issue? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1167#issuecomment-316274457___

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-07-13 Thread sergey-vb
Hi Daniel. I have just tested. Please look at screenshot from topos_t table. http://imgur.com/w4nnyQI Now PRACK is sending to contact from 183 Progress, but it does not recovers as it must be. I think it is record route restore issue. -- You are receiving this because you are subscribed to thi

Re: [sr-dev] [kamailio/kamailio] Incorrect routing of PRACK packets with topos module (#1167)

2017-07-12 Thread Daniel-Constantin Mierla
Although with rather long delay, I started to work on it. First step was to add support for storing the record-route and contact values in the branch record (topos_t table). Can you test with latest master branch and see if those fields are set after 180/183? I didn't have time to do any tests y