Re: [sr-dev] [kamailio/kamailio] siptrace: added extra params to $siptrace() pseudo-variable (#2777)

2021-06-10 Thread sergey-safarov
I finished PR -- 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/2777#issuecomment-858899029___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] siptrace: added extra params to $siptrace() pseudo-variable (#2777)

2021-06-10 Thread sergey-safarov
@sergey-safarov pushed 1 commit. 21454a3698f865ed3e62d4f28cab1da8ab32eba1 siptrace: removed brackets from IPv6 address [skip ci] -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] siptrace: added extra params to $siptrace() pseudo-variable (#2777)

2021-06-10 Thread sergey-safarov
@miconda no I want to call `C` function to strip brackets. I will dev mail list -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] nathelper: fix_nated_sdp added ignoring RFC3605-param if omitted (#2737)

2021-06-10 Thread Dmitry Wagin
Hello All, I'll check it out 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/2737#issuecomment-858826109___ Kamailio (SER) -

Re: [sr-dev] [kamailio/kamailio] siptrace: added extra params to $siptrace() pseudo-variable (#2777)

2021-06-10 Thread Daniel-Constantin Mierla
> How I can remove brackets from IPv6 address? > Example > Now `$siptrace(src_host)` returns `[::1]` > I want get `::1` from `$siptrace(src_host)`. @sergey-safarov - use sr-users mailing list for questions about operations that can be done in kamailio.cfg, the comments on PR should be related to

Re: [sr-dev] [kamailio/kamailio] siptrace: added extra params to $siptrace() pseudo-variable (#2777)

2021-06-10 Thread sergey-safarov
How I can remove brackets from IPv6 address? Example Now `$siptrace(src_host)` returns `[::1]` I want get `::1` from `$siptrace(src_host)`. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] statsd: adding support for histograms (#2774)

2021-06-10 Thread Julien Chavanton
Merged #2774 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/2774#event-4872674658___ Kamailio (SER) - Development Mailing

[sr-dev] git:master:f2d1d644: stastd: adding histogram (#2774)

2021-06-10 Thread GitHub
Module: kamailio Branch: master Commit: f2d1d64408ce3cc6e24e4709933990c8c80cf131 URL: https://github.com/kamailio/kamailio/commit/f2d1d64408ce3cc6e24e4709933990c8c80cf131 Author: Julien Chavanton Committer: GitHub Date: 2021-06-10T09:49:47-07:00 stastd: adding histogram (#2774) ---

[sr-dev] [kamailio/kamailio] siptrace: added extra params to $siptrace() pseudo-variable (#2777)

2021-06-10 Thread sergey-safarov
Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for

Re: [sr-dev] [kamailio/kamailio] [pv] tr_init_buffers() does malloc but memory is never freed (#2776)

2021-06-10 Thread Andrey Volk
Hah. `json_tr_clear_buffers(void)` is never called as well -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] [pv] tr_init_buffers() does malloc but memory is never freed (#2776)

2021-06-10 Thread Daniel-Constantin Mierla
The buffers are allocated once, at startup and reused during runtime many times, till kamailio is stopped. They are per process and, while we have a destroy module callback on shutdown, that's executed only on main process to clean the shared resources. The process private resources as cleaned

Re: [sr-dev] [kamailio/kamailio] TLS: same TLSc is used for different outbound connection when it is switched with tls_set_connect_server_id() (#2760)

2021-06-10 Thread Tim Chubb
@miconda it was a very naïve modification so it just created a new connection and never reused which was a horrible solution Re-reading through tcp_main.c I had a spark of inspiration when I noticed that `tcpconn_rm` checked if `c->extra_data` had a value other than 0 to determine if

[sr-dev] [kamailio/kamailio] [pv] tr_init_buffers() does malloc but memory is never freed (#2776)

2021-06-10 Thread Andrey Volk
### Description tr_init_buffers() does malloc 512kb and this memory is never freed https://github.com/kamailio/kamailio/blob/master/src/modules/pv/pv_trans.c#L78 Compare with kazoo which does have `kz_tr_clear_buffers()`

Re: [sr-dev] strip [] from IPv6 address

2021-06-10 Thread Daniel-Constantin Mierla
On 10.06.21 19:45, Sergey Safarov wrote: > I want call C function to strip IPv6 brackets. > Which can be used? C code or config operation? In github PR I understood is about config operation. In c code is practically coping string from start+1 a number of len-1 characters. Cheers, Daniel --

[sr-dev] strip [] from IPv6 address

2021-06-10 Thread Sergey Safarov
I want call C function to strip IPv6 brackets. Which can be used? ___ Kamailio (SER) - Development Mailing List sr-dev@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:20b39858: modules: readme files regenerated - statsd ... [skip ci]

2021-06-10 Thread Kamailio Dev
Module: kamailio Branch: master Commit: 20b39858b6e2707b40dcf564eceef55241d800ca URL: https://github.com/kamailio/kamailio/commit/20b39858b6e2707b40dcf564eceef55241d800ca Author: Kamailio Dev Committer: Kamailio Dev Date: 2021-06-10T19:01:14+02:00 modules: readme files regenerated - statsd

Re: [sr-dev] [kamailio/kamailio] postrgresql: kamdbctl create issue (#2773)

2021-06-10 Thread Daniel-Constantin Mierla
Closed #2773. -- 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/2773#event-4870958774___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] postrgresql: kamdbctl create issue (#2773)

2021-06-10 Thread Daniel-Constantin Mierla
Thanks for testing and feedback! -- 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/2773#issuecomment-858564218___ Kamailio (SER) -

Re: [sr-dev] [kamailio/kamailio] rtpengine: ignore mos 0 when selecting min-mos (#2771)

2021-06-10 Thread Richard Fuchs
Merged #2771 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/2771#event-4870937904___ Kamailio (SER) - Development Mailing

Re: [sr-dev] [kamailio/kamailio] rtpengine: ignore mos 0 when selecting min-mos (#2771)

2021-06-10 Thread Richard Fuchs
Fine with me, merging -- 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/2771#issuecomment-858561628___ Kamailio (SER) - Development Mailing

[sr-dev] git:master:a378f97a: rtpengine: ignore mos 0 when selecting min-mos (#2771)

2021-06-10 Thread GitHub
Module: kamailio Branch: master Commit: a378f97adac89077de2fb162ec20295f6e4352fe URL: https://github.com/kamailio/kamailio/commit/a378f97adac89077de2fb162ec20295f6e4352fe Author: Julien Chavanton Committer: GitHub Date: 2021-06-10T08:03:13-04:00 rtpengine: ignore mos 0 when selecting min-mos

Re: [sr-dev] [kamailio/kamailio] postrgresql: kamdbctl create issue (#2773)

2021-06-10 Thread Ilya Demyanov
branch 5.5 from Git - now there is no error: ``` # kamdbctl create -e \E[37;33mINFO: creating database kamailio ... -e \E[37;33mINFO: Core Kamailio tables successfully created. Install presence related tables? (y/n): y -e \E[37;33mINFO: creating presence tables into kamailio ... -e

Re: [sr-dev] [kamailio/kamailio] ERROR: [core/resolve.c:941]: get_record(): unknown type 41 (#2758)

2021-06-10 Thread Daniel-Constantin Mierla
Closing, based on remarks in #2759. -- 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/2758#issuecomment-858519202___ Kamailio (SER) -

Re: [sr-dev] [kamailio/kamailio] ERROR: [core/resolve.c:941]: get_record(): unknown type 41 (#2758)

2021-06-10 Thread Daniel-Constantin Mierla
Closed #2758. -- 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/2758#event-4870585531___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] TLS: same TLSc is used for different outbound connection when it is switched with tls_set_connect_server_id() (#2760)

2021-06-10 Thread Daniel-Constantin Mierla
With your changes, does it create a new connection every time? Or connections are reused in some cases? You can make a pull request with your code and we can analyze/review/comment there if can be integrated. The new behaviour should be guarded by a core parameter, because the current one

Re: [sr-dev] [kamailio/kamailio] postrgresql: kamdbctl create issue (#2773)

2021-06-10 Thread Daniel-Constantin Mierla
Can you try with latest branch 5.5? -- 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/2773#issuecomment-858514678___ Kamailio (SER) -

Re: [sr-dev] [kamailio/kamailio] statsd: adding support for histograms (#2774)

2021-06-10 Thread Daniel-Constantin Mierla
I am fine to merge 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/2774#issuecomment-858496047___ Kamailio (SER) - Development Mailing

Re: [sr-dev] [kamailio/kamailio] rtpengine: ignore mos 0 when selecting min-mos (#2771)

2021-06-10 Thread Daniel-Constantin Mierla
Maybe @rfuchs wants to have a quick look. >From my point of view, it is fine to merge. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] siptrace: fixed IPv6 without brackets (#2775)

2021-06-10 Thread sergey-safarov
@miconda correct -- 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/2775#issuecomment-858492633___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] siptrace: fixed IPv6 without brackets (#2775)

2021-06-10 Thread Daniel-Constantin Mierla
Merged #2775 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/2775#event-4870369640___ Kamailio (SER) - Development Mailing

Re: [sr-dev] [kamailio/kamailio] siptrace: fixed IPv6 without brackets (#2775)

2021-06-10 Thread Daniel-Constantin Mierla
Thanks! I will merge it. The fix is because kamailio printed `udp:::1:2345` instead `udp:[::1]:2345` in some siptrace structure fields. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] siptrace: fixed IPv6 without brackets (#2775)

2021-06-10 Thread Henning Westerholt
Thanks for the pull request. :::1:2345 is not a valid IPv6 address, so why we should support it in Kamailio? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] [kamailio/kamailio] siptrace: fixed IPv6 without brackets (#2775)

2021-06-10 Thread sergey-safarov
Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for

Re: [sr-dev] [kamailio/kamailio] TLS: same TLSc is used for different outbound connection when it is switched with tls_set_connect_server_id() (#2760)

2021-06-10 Thread Tim Chubb
I would agree its a connection reuse issue, #1107 is probably a more relevant discussion to have. As an experiment I tried a modified version of tcp_main.c with the connection lookup modified to not return an existing connection and the expected behaviour was observed. Thinking the further

[sr-dev] git:5.5:66370c47: kamctl: update name to globalblocklist in EXTRA_TABLES

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: 66370c473967d567452a6cf26b8b3496cb1386be URL: https://github.com/kamailio/kamailio/commit/66370c473967d567452a6cf26b8b3496cb1386be Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:29:42+02:00 kamctl: update name to

[sr-dev] git:5.5:903e4146: siptrace: fixed IPv6 without brackets

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: 903e41468a00edeff52cbf1e3c0053c82860c92c URL: https://github.com/kamailio/kamailio/commit/903e41468a00edeff52cbf1e3c0053c82860c92c Author: Sergey Safarov Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:29:32+02:00 siptrace: fixed IPv6 without

[sr-dev] git:5.5:7b89c5e6: pv: added missing WS and WSS to $K(key) variable

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: 7b89c5e6abe5f4ae3ee55d9c938c865806f29dec URL: https://github.com/kamailio/kamailio/commit/7b89c5e6abe5f4ae3ee55d9c938c865806f29dec Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:29:07+02:00 pv: added missing WS and

[sr-dev] git:5.5:38459725: smsops: support for 7bit special chars

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: 3845972598098add3996b19c6cedfbfb4d4d2ee6 URL: https://github.com/kamailio/kamailio/commit/3845972598098add3996b19c6cedfbfb4d4d2ee6 Author: Aleksandar Yosifov Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:28:34+02:00 smsops: support for 7bit

[sr-dev] git:5.5:7192139d: app_perl change to ignore "die" signals from eval blocks

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: 7192139d77f844cd927c326ce16cadfc63378b35 URL: https://github.com/kamailio/kamailio/commit/7192139d77f844cd927c326ce16cadfc63378b35 Author: rodolfojcj <458749+rodolfo...@users.noreply.github.com> Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:5.5:d1c83128: dmq: warning about protocol made info

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: d1c831288746bac7850343da10ae02099cb7c80d URL: https://github.com/kamailio/kamailio/commit/d1c831288746bac7850343da10ae02099cb7c80d Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:27:42+02:00 dmq: warning about

[sr-dev] git:5.5:18e85c98: pv: test for HDR_ERROR_T after header name parsing

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: 18e85c98bb6eebf9afe83aaa0a832d1c4af46e62 URL: https://github.com/kamailio/kamailio/commit/18e85c98bb6eebf9afe83aaa0a832d1c4af46e62 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:27:18+02:00 pv: test for HDR_ERROR_T

[sr-dev] git:5.5:b6428ca8: dialog: safety checks for bind addr fields

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: b6428ca86c821ddf74fd4ea8a17aecdce39934f4 URL: https://github.com/kamailio/kamailio/commit/b6428ca86c821ddf74fd4ea8a17aecdce39934f4 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:27:02+02:00 dialog: safety checks

[sr-dev] git:5.5:b5041e2c: core: kemi - check hdr type HDR_ERROR_T after parsing

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.5 Commit: b5041e2c4a44843318fcd69d092342e2b29532f4 URL: https://github.com/kamailio/kamailio/commit/b5041e2c4a44843318fcd69d092342e2b29532f4 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:25:21+02:00 core: kemi - check hdr

[sr-dev] git:master:90fa160b: kamctl: update name to globalblocklist in EXTRA_TABLES

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 90fa160b5d4a7155fc3d82f39932ac0e9d74fae7 URL: https://github.com/kamailio/kamailio/commit/90fa160b5d4a7155fc3d82f39932ac0e9d74fae7 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:15:29+02:00 kamctl: update name

[sr-dev] git:master:417eb2a7: siptrace: fixed IPv6 without brackets

2021-06-10 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 417eb2a7052a9354712663cca9e3bc075a3f69c0 URL: https://github.com/kamailio/kamailio/commit/417eb2a7052a9354712663cca9e3bc075a3f69c0 Author: Sergey Safarov Committer: Daniel-Constantin Mierla Date: 2021-06-10T12:05:49+02:00 siptrace: fixed IPv6 without

Re: [sr-dev] [SR-Users] Releasing Kamailio v5.5.1

2021-06-10 Thread Sergey Safarov
Looks as commit not backported https://github.com/kamailio/kamailio/commits/5.5/src/modules/mohqueue On Thu, Jun 10, 2021 at 9:09 AM RDB Programmer wrote: > Daniel, > > Will this release include the latest changes to mohqueue >

Re: [sr-dev] [SR-Users] Releasing Kamailio v5.5.1

2021-06-10 Thread RDB Programmer
Daniel, Will this release include the latest changes to mohqueue that I pushed to the master on May 19? Bob > From: Daniel-Constantin Mierla > Date: Wed, Jun 9, 2021 at 6:00 AM > Subject: [SR-Users] Releasing Kamailio