Re: [sr-dev] [kamailio/kamailio] Issue with new memory-only presence feature in 5.4.3 (#2642)

2021-02-17 Thread Joe Mordica
Commenting out `modparam("presence", "enable_dmq", 1)` allows the memory-only presence module to function properly and not crash/lock. I don't actually need the dmq to work for presence module so that's ok for me but if you need additional testing I can try to help out. -- You are receiving th

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
cool. it's working 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/2639#issuecomment-781009054___ Kamailio (SER) - Development Maili

Re: [sr-dev] [kamailio/kamailio] Issue with new memory-only presence feature in 5.4.3 (#2642)

2021-02-17 Thread Daniel-Constantin Mierla
It looks like a crash happening on shutdown, because the log message from pres_dmq_init_proc() is preventing kamailio to start. You can try to get a core dump, you need to start as a root and enable core dumping -- some guidelines at: * https://www.kamailio.org/wiki/tutorials/troubleshooting/c

Re: [sr-dev] [kamailio/kamailio] rtpengine: rebuild sockets if any skipped nodes (#2633)

2021-02-17 Thread Henning Westerholt
Thanks for the review. It seems like a smaller change, merged. -- 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/2633#issuecomment-780674031_

Re: [sr-dev] [kamailio/kamailio] rtpengine: rebuild sockets if any skipped nodes (#2633)

2021-02-17 Thread Henning Westerholt
Merged #2633 into master. -- 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/2633#event-4342295629___ Kamailio (SER) - Development Mailing Lis

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Daniel-Constantin Mierla
One more try with a fresh master and let's see if `local_rport=yes` works ... thanks for assistance! -- 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/2639#issuecomment-780550936

[sr-dev] git:master:220f9ca3: kex: display proper values for pkg mem stats

2021-02-17 Thread Ovidiu Sas
Module: kamailio Branch: master Commit: 220f9ca3c486b92797982a243fab463e9e5a URL: https://github.com/kamailio/kamailio/commit/220f9ca3c486b92797982a243fab463e9e5a Author: Ovidiu Sas Committer: Ovidiu Sas Date: 2021-02-17T12:08:10-05:00 kex: display proper values for pkg mem stats - la

Re: [sr-dev] [kamailio/kamailio] lost: resolve geolocation url using POST (#2641)

2021-02-17 Thread sergey-safarov
I observed that http_client drop the HTTP request after a timeout. So will be good to let LIS know Kamailio timeout. To do this may be used `responseTime` paramer according rfc5985#section-6.1 > The "responseTime" attribute MAY be included in a location request message. The "responseTime" att

[sr-dev] git:master:c88e65ed: rtpengine: rebuild sockets if any skipped nodes

2021-02-17 Thread Henning Westerholt
Module: kamailio Branch: master Commit: c88e65edf05f7e61826831a2f7528edbc184f68b URL: https://github.com/kamailio/kamailio/commit/c88e65edf05f7e61826831a2f7528edbc184f68b Author: Stefan Mititelu Committer: Henning Westerholt Date: 2021-02-17T17:21:22+01:00 rtpengine: rebuild sockets if any ski

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Daniel-Constantin Mierla
The Via header was first time created before execution of `event_route[tm:local-request]`, the condition there is to figure out if it needs updates. Starting from this, I just looked back into the code and discovered that the tm module does not create Via header first time using create_via_hf()

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
I added some debug log, the `create_via_hf` func is not been called when `FL_ADD_LOCAL_RPORT` is not set, so in the following line it's going to the `else` branch unless force send socket is set. https://github.com/kamailio/kamailio/blob/master/src/modules/tm/uac.c#L315 -- You are receiving

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Daniel-Constantin Mierla
Regarding `local_rport=yes`, it is somehow strange, because the code has the condition: ``` if((ksr_local_rport) || (msg && (msg->msg_flags&FL_ADD_LOCAL_RPORT))) { ``` `ksr_local_rport` corresponds to `local_rport` parameter and `FL_ADD_LOCAL_RPORT` is set by `add_local_rport()`. The cfg value

[sr-dev] git:master:6863cb8b: core: early detect of empty host for check_self()

2021-02-17 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 6863cb8b0253c641146633bd063575a53bc6a466 URL: https://github.com/kamailio/kamailio/commit/6863cb8b0253c641146633bd063575a53bc6a466 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-02-17T14:07:43+01:00 core: early detect of

[sr-dev] git:master:bfbf41d8: tm: consider local_rport parameter when building local via

2021-02-17 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: bfbf41d8ba2ba2b6eed8eb8fff2e554476c0a5b8 URL: https://github.com/kamailio/kamailio/commit/bfbf41d8ba2ba2b6eed8eb8fff2e554476c0a5b8 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-02-17T13:31:34+01:00 tm: consider local_rpo

[sr-dev] git:master:81cd9c5d: rr: catch uri without host part when matching local addresses

2021-02-17 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 81cd9c5d6934392cafd4b73541a19dbfa4b553f9 URL: https://github.com/kamailio/kamailio/commit/81cd9c5d6934392cafd4b73541a19dbfa4b553f9 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-02-17T12:15:55+01:00 rr: catch uri without

Re: [sr-dev] [kamailio/kamailio] lreproxy module (#2208)

2021-02-17 Thread Julien Chavanton
> @jchavanton, Would you please let me know more details related to first > comment that you mentioned? > The second comment is accepted and i will do it. > Let me know how do you test it? What unexpected performance degradation did > you see? @mojtabaesfandiari Considering that my test report

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
thanks a lot, it's working. gloal `local_rport=yes` has no effect. and the event callback works for me. -- 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/2639#issuecomment-780372599_

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Seven Du
Closed #2639. -- 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/2639#event-4339932691___ Kamailio (SER) - Development Mailing List sr-dev@l

Re: [sr-dev] [kamailio/kamailio] lreproxy module (#2208)

2021-02-17 Thread mojtabaesfandiari
Actually, I am planning for having a workshop of using lreproxy( LRKProxy ) as soon. -- 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/2208#issuecomment-780369541_

Re: [sr-dev] [kamailio/kamailio] lreproxy module (#2208)

2021-02-17 Thread mojtabaesfandiari
@jchavanton, Would you please let me know more details related to first comment that you mentioned? The second comment is accepted and i will do it. Let me know how do you test it?What unexpected performance degradation did you see? -- You are receiving this because you are subscribed to this

Re: [sr-dev] [kamailio/kamailio] lreproxy module (#2208)

2021-02-17 Thread mojtabaesfandiari
@jchavanton,there was a quite bit issue when the lreproxy kernel engine received duplicate ports. You could check it after update. Thanks -- 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/p

Re: [sr-dev] [kamailio/kamailio] add rport to uac register msg (#2639)

2021-02-17 Thread Daniel-Constantin Mierla
Try now with a fresh master, I pushed some commits trying to fix what I thought can be the issue based on log messages, otherwise I was not able to test myself so far. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https:/