sr-dev February 2024

sr-dev@lists.kamailio.org

19 participants
247 discussions

[kamailio/kamailio] Module pipelimit add pl.reload RPC Command (Issue #3632)
by jerry 22 Feb '24

22 Feb '24

### Description Sometimes it is necessary to dynamically add current limiting rules without restarting kamailio. After adding pipe rules in the database, it is hoped that the new rules can be reloaded into memory through the rpc command. ### Expected behavior #### Actual observed behavior #### Debugging Data ``` (paste your debugging data here) ``` #### Log Messages  ``` (paste your log messages here) ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.7.0 (x86_64/linux) f1e91d flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, MEM_JOIN_FREE, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: f1e91d compiled on 14:56:02 May 17 2023 with gcc 4.8.5 ``` * **Operating System**: ``` Debian 9+, Ubuntu 16.04+, CentOS 7, ... ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3632 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3632(a)github.com>

2 4

xhttp_prom missing from kamailio-extra-modules
by Ovidiu Sas 21 Feb '24

21 Feb '24

Hello, Any particular reason for not shipping xhttp_prom in a deb package? -ovidiu

1 1

[kamailio/kamailio] tools/kemi: fix bug in mock (PR #3760)
by tsearle 21 Feb '24

21 Feb '24

remove stray debug print statement that results in generated code being invalid   #### 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) - [ ] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3760 -- Commit Summary -- * tools/kemi: fix bug in mock -- File Changes -- M misc/tools/kemi/python_mock/kemi_mock.py (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3760.patch https://github.com/kamailio/kamailio/pull/3760.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3760 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3760(a)github.com>

2 1

git:master:e54152e0: tools/kemi: fix bug in mock
by Victor Seva 21 Feb '24

21 Feb '24

Module: kamailio Branch: master Commit: e54152e06e4a097ac17522e282f26b277ecae4da URL: https://github.com/kamailio/kamailio/commit/e54152e06e4a097ac17522e282f26b2… Author: Torrey Searle <torrey.searle(a)wavecrest.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-02-21T08:28:20+01:00 tools/kemi: fix bug in mock remove stray debug print statement that results in generated code being invalid --- Modified: misc/tools/kemi/python_mock/kemi_mock.py --- Diff: https://github.com/kamailio/kamailio/commit/e54152e06e4a097ac17522e282f26b2… Patch: https://github.com/kamailio/kamailio/commit/e54152e06e4a097ac17522e282f26b2… --- diff --git a/misc/tools/kemi/python_mock/kemi_mock.py b/misc/tools/kemi/python_mock/kemi_mock.py index a755ae4a7c0..48a73efa232 100755 --- a/misc/tools/kemi/python_mock/kemi_mock.py +++ b/misc/tools/kemi/python_mock/kemi_mock.py @@ -82,7 +82,6 @@ def printFunction(module_name, func, indent): log_format_params = "%s" param_list = [] - print("func['params']=%s" % func['params']) if func['params'] is not None and func['params'] != "none": param_list = func['params'].split(", ") i = 0

1 0

rpm kamailio git master version
by Henning Westerholt 20 Feb '24

20 Feb '24

Hello, just a question about the https://rpm.kamailio.org/ repository, especially regarding git master builds. It seems that the git master build there was updated last autumn. It would be great if it could be updated, now as we want to test for the coming release. Thank you, Henning Westerholt -- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://gilawa.com<https://gilawa.com/>

1 0

git:master:e3e6fd75: core: tcp - skip checking when tcp_accept_iplimit is 0
by Daniel-Constantin Mierla 20 Feb '24

20 Feb '24

Module: kamailio Branch: master Commit: e3e6fd75bdd688acebf0ec8a1b7efa6fcada1de6 URL: https://github.com/kamailio/kamailio/commit/e3e6fd75bdd688acebf0ec8a1b7efa6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-02-19T22:24:21+01:00 core: tcp - skip checking when tcp_accept_iplimit is 0 --- Modified: src/core/tcp_main.c --- Diff: https://github.com/kamailio/kamailio/commit/e3e6fd75bdd688acebf0ec8a1b7efa6… Patch: https://github.com/kamailio/kamailio/commit/e3e6fd75bdd688acebf0ec8a1b7efa6… --- diff --git a/src/core/tcp_main.c b/src/core/tcp_main.c index 35034d1baa8..757329a8543 100644 --- a/src/core/tcp_main.c +++ b/src/core/tcp_main.c @@ -1169,6 +1169,10 @@ int tcp_connection_limit_srcip(union sockaddr_union *srcaddr, int limit) int n; int i; + if(limit <= 0) { + return 0; + } + n = 0; su2ip_addr(&src_ip, srcaddr); TCPCONN_LOCK;

1 0

git:master:dad7f3ff: file_out: small extensions to the module docs
by Henning Westerholt 19 Feb '24

19 Feb '24

Module: kamailio Branch: master Commit: dad7f3ff3af02d086c1cb3f915f1502bca127d58 URL: https://github.com/kamailio/kamailio/commit/dad7f3ff3af02d086c1cb3f915f1502… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-02-19T20:30:45Z file_out: small extensions to the module docs --- Modified: src/modules/file_out/doc/file_out_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/dad7f3ff3af02d086c1cb3f915f1502… Patch: https://github.com/kamailio/kamailio/commit/dad7f3ff3af02d086c1cb3f915f1502… --- diff --git a/src/modules/file_out/doc/file_out_admin.xml b/src/modules/file_out/doc/file_out_admin.xml index a10ae1974d0..ce02069ae83 100644 --- a/src/modules/file_out/doc/file_out_admin.xml +++ b/src/modules/file_out/doc/file_out_admin.xml @@ -87,7 +87,7 @@ <title>Set <varname>base_folder</varname> parameter</title> <programlisting format="linespecific"> ... -modparam("file_out", "base_folder", "/tmp/file_out") # Trailling slash will be added. +modparam("file_out", "base_folder", "/tmp/file_out") # trailing slash will be added. ... </programlisting> </example> @@ -102,7 +102,7 @@ modparam("file_out", "base_folder", "/tmp/file_out") # Trailling slash will be a <itemizedlist> <listitem> <para> - "name=accounting;extension=.out;interval=20;prefix=acc_" + "name=accounting;extension=.out;interval=20;prefix=accounting:" </para> </listitem> </itemizedlist>

1 0

[kamailio/kamailio] file_out: Add prefix parameter and docs and fix leaks. (PR #3758)
by Xenofon Karamanos 19 Feb '24

19 Feb '24

#### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  This PR adds a new parameter `prefix` to the function and its docs. It allows the message to be fixed with a prefix string. Also fixes some leak issues. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3758 -- Commit Summary -- * file_out: Free allocated memory * file_out: Add a prefix paramater -- File Changes -- M src/modules/file_out/doc/file_out_admin.xml (13) M src/modules/file_out/file_out.c (32) M src/modules/file_out/types.c (9) M src/modules/file_out/types.h (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3758.patch https://github.com/kamailio/kamailio/pull/3758.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3758 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3758(a)github.com>

2 2

git:master:a722d4cb: file_out: Add a prefix paramater
by Henning Westerholt 19 Feb '24

19 Feb '24

Module: kamailio Branch: master Commit: a722d4cba1ab18f481bfe8353cebc0c6fa1a7ebb URL: https://github.com/kamailio/kamailio/commit/a722d4cba1ab18f481bfe8353cebc0c… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-02-19T21:28:19+01:00 file_out: Add a prefix paramater --- Modified: src/modules/file_out/doc/file_out_admin.xml Modified: src/modules/file_out/file_out.c --- Diff: https://github.com/kamailio/kamailio/commit/a722d4cba1ab18f481bfe8353cebc0c… Patch: https://github.com/kamailio/kamailio/commit/a722d4cba1ab18f481bfe8353cebc0c… --- diff --git a/src/modules/file_out/doc/file_out_admin.xml b/src/modules/file_out/doc/file_out_admin.xml index 02c9d978e66..a10ae1974d0 100644 --- a/src/modules/file_out/doc/file_out_admin.xml +++ b/src/modules/file_out/doc/file_out_admin.xml @@ -102,7 +102,7 @@ modparam("file_out", "base_folder", "/tmp/file_out") # Trailling slash will be a <itemizedlist> <listitem> <para> - "name=accounting;extension=.out;interval=20" + "name=accounting;extension=.out;interval=20;prefix=acc_" </para> </listitem> </itemizedlist> @@ -145,6 +145,17 @@ modparam("file_out", "base_folder", "/tmp/file_out") # Trailling slash will be a </emphasis> </para> </listitem> + + <listitem> + <para> + prefix (Optional) - the prefix for log messages + </para> + <para> + <emphasis> + Default value is <quote>""</quote> (empty string). + </emphasis> + </para> + </listitem> </itemizedlist> <example> diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index 5a7643d0e4f..443eb118d20 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -40,6 +40,7 @@ MODULE_VERSION #define FO_MAX_FILES 10 /* Maximum number of files */ #define FO_DEFAULT_INTERVAL 10 * 60 #define FO_DEFAULT_EXTENSION ".out" +#define FO_DEFAULT_PREFIX "" static int mod_init(void); static int child_init(int rank); @@ -64,6 +65,7 @@ static int fo_parse_filename_params(str input); char *fo_base_folder = "/var/log/kamailio/file_out"; char *fo_base_filename[FO_MAX_FILES] = {""}; char *fo_extension[FO_MAX_FILES] = {".out"}; +char *fo_prefix[FO_MAX_FILES] = {""}; int fo_interval_seconds[FO_MAX_FILES] = {10 * 60}; int fo_worker_usleep = 10000; @@ -211,6 +213,13 @@ static void fo_log_writer_process(int rank) return; } + /* Get prefix for the file */ + if(fo_prefix[log_message.dest_file] != NULL) { + if(fprintf(out, "%s", fo_prefix[log_message.dest_file]) < 0) { + LM_ERR("Failed to write prefix to file with err {%s}\n", + strerror(errno)); + } + } if(fprintf(out, "%.*s\n", log_message.message->len, log_message.message->s) < 0) { @@ -348,6 +357,7 @@ static int fo_parse_filename_params(str in) char *name = NULL; char *extension = NULL; char *interval = NULL; + char *prefix = NULL; char *token = NULL; char *saveptr = NULL; char *input = in.s; @@ -359,6 +369,11 @@ static int fo_parse_filename_params(str in) extension = token + 10; } else if(strstr(token, "interval=") != NULL) { interval = token + 9; + } else if(strstr(token, "prefix=") != NULL) { + prefix = token + 7; + } else { + LM_ERR("Unknown parameter %s\n", token); + return -1; } token = strtok_r(NULL, ";", &saveptr); } @@ -389,6 +404,13 @@ static int fo_parse_filename_params(str in) fo_interval_seconds[*fo_number_of_files] = FO_DEFAULT_INTERVAL; } + if(prefix != NULL) { + LM_DBG("prefix = %s\n", prefix); + fo_prefix[*fo_number_of_files] = prefix; + } else { + LM_DBG("no prefix= provided. Using default %s\n", FO_DEFAULT_PREFIX); + fo_prefix[*fo_number_of_files] = FO_DEFAULT_PREFIX; + } return 1; }

1 0

git:master:22581f95: file_out: Free allocated memory
by Henning Westerholt 19 Feb '24

19 Feb '24

Module: kamailio Branch: master Commit: 22581f95690415681ea233c97923137e46fb55ae URL: https://github.com/kamailio/kamailio/commit/22581f95690415681ea233c97923137… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-02-19T21:28:19+01:00 file_out: Free allocated memory --- Modified: src/modules/file_out/file_out.c Modified: src/modules/file_out/types.c Modified: src/modules/file_out/types.h --- Diff: https://github.com/kamailio/kamailio/commit/22581f95690415681ea233c97923137… Patch: https://github.com/kamailio/kamailio/commit/22581f95690415681ea233c97923137… --- diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index fde63db5a0b..5a7643d0e4f 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -183,6 +183,16 @@ static void destroy(void) ERR("Failed to close output file"); } } + + /* Free allocated mem */ + if(fo_number_of_files != NULL) { + shm_free(fo_number_of_files); + fo_number_of_files = NULL; + } + + if(fo_queue != NULL) { + fo_free_queue(fo_queue); + } } static void fo_log_writer_process(int rank) diff --git a/src/modules/file_out/types.c b/src/modules/file_out/types.c index d2815333dd0..f5c43a41695 100644 --- a/src/modules/file_out/types.c +++ b/src/modules/file_out/types.c @@ -113,3 +113,12 @@ int fo_queue_size(fo_queue_t *q) lock_release(&(q->lock)); return count; } + +void fo_free_queue(fo_queue_t *q) +{ + fo_log_message_t data; + while(fo_dequeue(q, &data) > 0) { + shm_free(data.message); + } + shm_free(q); +} diff --git a/src/modules/file_out/types.h b/src/modules/file_out/types.h index c6fb72b28ef..b9c3b411165 100644 --- a/src/modules/file_out/types.h +++ b/src/modules/file_out/types.h @@ -43,3 +43,4 @@ int fo_enqueue(fo_queue_t *q, fo_log_message_t data); int fo_dequeue(fo_queue_t *q, fo_log_message_t *data); int fo_is_queue_empty(fo_queue_t *q); int fo_queue_size(fo_queue_t *q); +void fo_free_queue(fo_queue_t *q);

1 0

← Newer
1
...
9
10
11
12
13
14
15
...
25
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev February 2024