Re: [sr-dev] [kamailio/kamailio] topos: adding param contact_hostname (#1596)

2018-07-18 Thread Julien Chavanton
Hi, reading the code I found it was taken from the RR header, I missed `record_route_preset()` This param was not absolutely necessary but I guess it is making things more clear for someone wanting to use TOPOS. And maybe simpler as well. I have tested the code "back ported" only for now, but I

Re: [sr-dev] [kamailio/kamailio] RPC command stats.get_statistics randomly reporting 9223372036854776000 for current active/early dialogs (#1591)

2018-07-18 Thread Joel Serrano
Thanks! Let me know if you need any more info -- 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/1591#issuecomment-406000670___ Kamailio (SE

Re: [sr-dev] [kamailio/kamailio] coredumps possibly generated by DMQ in kamailio v5.1.4 (#1594)

2018-07-18 Thread Joel Serrano
@miconda: ``` (gdb) frame 0 #0 0x7f4e12ce6c68 in get_dlg (callid=0x7ffdc5993850, ftag=0x7ffdc5993860, ttag=0x7ffdc5993870, dir=0x7ffdc5993840) at dlg_hash.c:797 797 in dlg_hash.c (gdb) p *callid $1 = {s = 0x7f4dec572b2e "72a1671957944647-777@138.99.136.21\r\nContent-Length: 70\r\nMax-F

Re: [sr-dev] [kamailio/kamailio] core/parser:parse_headers error. (#1582)

2018-07-18 Thread gaby weeki
I've got the same problem, i'll sngrep the traffic to send a pcap file as soon as possible. Warmly FULCHIC Gaby -- 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/1582#issuecomment-40

Re: [sr-dev] [kamailio/kamailio] coredumps possibly generated by DMQ in kamailio v5.1.4 (#1594)

2018-07-18 Thread Daniel-Constantin Mierla
@joelsdc - if you still have the core file used to get the content for core2_* files, can you grab the output for next gdb command in frame 0: ``` p *callid p *ftag p *ttag p *dir p d_table p *d_table ``` -- You are receiving this because you are subscribed to this thread. Reply to this email

Re: [sr-dev] [kamailio/kamailio] Module RTPPROXY param Timeout_Socket not Working (#1597)

2018-07-18 Thread Daniel-Constantin Mierla
Kamailio should not start listening on the socket if timeout_socket param is set. As explained in the documentation, the value is passed to rtpproxy application, that's all done by rtpproxy module in kamailio. At that socket can be kamailio if you provide an xmlrpc url (see the docs) and use xm

Re: [sr-dev] [kamailio/kamailio] Module RTPPROXY param Timeout_Socket not Working (#1597)

2018-07-18 Thread Daniel-Constantin Mierla
Closed #1597. -- 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/1597#event-1740329401___ Kamailio (SER) - Development Mailing List sr-dev@l

Re: [sr-dev] [kamailio/kamailio] RPC command stats.get_statistics randomly reporting 9223372036854776000 for current active/early dialogs (#1591)

2018-07-18 Thread Daniel-Constantin Mierla
The values are closed to ULONG_MAX for 64bit, so likely some dialog end events are counted many times (likely due to dmq replication) and the value goes below 0, turning it to max value. For reference, ranges for number types: * https://en.wikibooks.org/wiki/C_Programming/limits.h In the past

Re: [sr-dev] [kamailio/kamailio] kamailio always retries deleted RTPEngine Nodes which generates PDD (#1593)

2018-07-18 Thread Daniel-Constantin Mierla
Merged #1593. -- 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/pull/1593#event-1739733669___ Kamailio (SER) - Development Mailing List sr-dev@lis

Re: [sr-dev] [kamailio/kamailio] kamailio always retries deleted RTPEngine Nodes which generates PDD (#1593)

2018-07-18 Thread Daniel-Constantin Mierla
@rfuchs - thanks for the review. I am merging it. -- 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/pull/1593#issuecomment-405879920___ Kamailio

[sr-dev] git:master:6c94217d: Merge pull request #1593 from adil-mafzool/master

2018-07-18 Thread GitHub
Module: kamailio Branch: master Commit: 6c94217d68bdde980bd4b77bdc6d22062bd650ee URL: https://github.com/kamailio/kamailio/commit/6c94217d68bdde980bd4b77bdc6d22062bd650ee Author: Daniel-Constantin Mierla Committer: GitHub Date: 2018-07-18T12:03:59+02:00 Merge pull request #1593 from adil-mafzo

Re: [sr-dev] [kamailio/kamailio] topos: adding param contact_hostname (#1596)

2018-07-18 Thread Daniel-Constantin Mierla
I am fine to add this parameter, but the patch lacks checking if the size is exceeding the available space in the buffer -- it checks only sv.len, which is the uri from where to extract ip, port and transport, not it should also check for param len. If this and the docs are added, then can be me