- sr-dev - kamailio.org

[kamailio/kamailio] file_out: Add prefix support with PVs and refactor properties in structure. (PR #3763)
by Xenofon Karamanos 22 Feb '24

22 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, ...) - [x] 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) - [ ] 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 the support of using PVs in the `prefix` parameter. Also refactors the properties into a structure for each file. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3763 -- Commit Summary -- * file_out: Prefix can use PVs * file_out: Refactor -- File Changes -- M src/modules/file_out/file_out.c (140) M src/modules/file_out/types.c (38) M src/modules/file_out/types.h (17) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3763.patch https://github.com/kamailio/kamailio/pull/3763.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3763 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3763(a)github.com>

1 1

git:master:998a7905: file_out: Refactor
by Xenofon Karamanos 22 Feb '24

22 Feb '24

Module: kamailio Branch: master Commit: 998a790563fcba9d6263c18f83f59e9080c78af0 URL: https://github.com/kamailio/kamailio/commit/998a790563fcba9d6263c18f83f59e9… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Committer: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2024-02-22T20:53:57+02:00 file_out: Refactor --- 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/998a790563fcba9d6263c18f83f59e9… Patch: https://github.com/kamailio/kamailio/commit/998a790563fcba9d6263c18f83f59e9…

1 0

git:master:06d0596e: file_out: Prefix can use PVs
by Xenofon Karamanos 22 Feb '24

22 Feb '24

Module: kamailio Branch: master Commit: 06d0596ef34f1c213cc28442594bb6f1ebae46b4 URL: https://github.com/kamailio/kamailio/commit/06d0596ef34f1c213cc28442594bb6f… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Committer: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2024-02-22T20:53:57+02:00 file_out: Prefix can use PVs --- 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/06d0596ef34f1c213cc28442594bb6f… Patch: https://github.com/kamailio/kamailio/commit/06d0596ef34f1c213cc28442594bb6f…

1 0

[kamailio/kamailio] microhttpd: fixed typo in the documentation (PR #3761)
by falshunov 22 Feb '24

22 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, ...) - [x] 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) - [ ] 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/3761 -- Commit Summary -- * microhttpd: fixed typo in the documentation -- File Changes -- M src/modules/microhttpd/doc/microhttpd_admin.xml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3761.patch https://github.com/kamailio/kamailio/pull/3761.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3761 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3761(a)github.com>

2 2

git:master:ff8229b5: microhttpd: fixed typo in the documentation [skip ci]
by Victor Seva 22 Feb '24

22 Feb '24

Module: kamailio Branch: master Commit: ff8229b58d6f91bae527bc2f5912c7d291c55268 URL: https://github.com/kamailio/kamailio/commit/ff8229b58d6f91bae527bc2f5912c7d… Author: falshunov <37283637+falshunov(a)users.noreply.github.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-02-22T10:28:11+01:00 microhttpd: fixed typo in the documentation [skip ci] --- Modified: src/modules/microhttpd/doc/microhttpd_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/ff8229b58d6f91bae527bc2f5912c7d… Patch: https://github.com/kamailio/kamailio/commit/ff8229b58d6f91bae527bc2f5912c7d… --- diff --git a/src/modules/microhttpd/doc/microhttpd_admin.xml b/src/modules/microhttpd/doc/microhttpd_admin.xml index a50edb79818..1967c845b82 100644 --- a/src/modules/microhttpd/doc/microhttpd_admin.xml +++ b/src/modules/microhttpd/doc/microhttpd_admin.xml @@ -43,7 +43,7 @@ <itemizedlist> <listitem> <para> - <emphasis>libmicrohhtpd</emphasis> - libmicrohttpd library + <emphasis>libmicrohttpd</emphasis> - libmicrohttpd library </para> </listitem> </itemizedlist>

1 0

[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

Latest point releases seem to break early dialog UPDATEs from B-side when using topos (again)
by George Diamantopoulos 16 Feb '24

16 Feb '24

Hello all, I've noticed that there seems to be a regression with the topos module, more specifically the redis flavour, but I'm assuming the storage backend shouldn't make a difference. I have confirmed this affects both 5.6.5 and 5.7-nightly, so I'm assuming some backported commit is to blame. Kamailio 5.6.4 used previously, to the best of my memory, was not affected. Early dialog UPDATEs sent from the callee seem to be somehow malformed, since the 200 OK the UAS produces goes directly to the UAC despite having executed record_route() for the original INVITE. I can't imagine how this is possible, I thought responses always honour the Via header and record route shouldn't play a role in this case, right? Simply turning topos off restores the intended behaviour, however, so I can't help but think this is somehow related. Here's two call flows demonstrating the behaviour: * Topos on, 200 OK to UPDATE missing: https://pastebin.com/raw/J0zQeM5g * Topos off, 200 OK to UPDATE routed correctly: https://pastebin.com/raw/49yErezb I was wondering if anyone is aware of any commits that might be responsible for this. Additionally, is there an archive of debian packages with previous point releases so that I can confirm this regression with the latest versions? Thank you! Best regards, George

2 3

[kamailio/kamailio] file_out: Fix a bug on fixup function, update docs, logging information. (PR #3756)
by Xenofon Karamanos 16 Feb '24

16 Feb '24

#### Pre-Submission Checklist    - [ ] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] 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 - [x] 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 - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  This PR fixes small bugs, and improves docs. #### Main bug fix: - Fixup now should be able to handle the string from `.cfg` with multiple PVs, strings, and more. - Add fixup_free functions #### Smaller fixes: - `base_folder` should not contain a trailing slash. Trailing slash is allowed, but resulting paths can contain `//log_file.extenion`, which in more OS is fine and equal to `/`. - Add initial Readme and fix some docs. - Switch to `str` type in internal types for better string handling. - Free message when done You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3756 -- Commit Summary -- * file_out: Change default base folder format * file_out: Add initial Readme and improve docs * file_out: Switch to str type for string handling * file_out: Free message when done * file_out: Improve logging * file_out: Fix fixup functions to handle all string. -- File Changes -- A src/modules/file_out/README (171) M src/modules/file_out/doc/file_out_admin.xml (4) M src/modules/file_out/file_out.c (52) M src/modules/file_out/types.c (19) M src/modules/file_out/types.h (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3756.patch https://github.com/kamailio/kamailio/pull/3756.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3756 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3756(a)github.com>

1 1

git:master:d2459f4a: file_out: Fix fixup functions to handle all string.
by Xenofon Karamanos 16 Feb '24

16 Feb '24

Module: kamailio Branch: master Commit: d2459f4a9978e6b42ba99212b77dd856005df0d7 URL: https://github.com/kamailio/kamailio/commit/d2459f4a9978e6b42ba99212b77dd85… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xenakiskaramanos(a)gmail.com> Date: 2024-02-16T12:20:44+02:00 file_out: Fix fixup functions to handle all string. --- Modified: src/modules/file_out/file_out.c --- Diff: https://github.com/kamailio/kamailio/commit/d2459f4a9978e6b42ba99212b77dd85… Patch: https://github.com/kamailio/kamailio/commit/d2459f4a9978e6b42ba99212b77dd85… --- diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index c985c04935a..fde63db5a0b 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -54,6 +54,7 @@ static int fo_close_file(const int index); static int fo_check_interval(int index); static int fo_fixup_int_pvar(void **param, int param_no); static int fo_fixup_str_index(void **param, int param_no); +static int fo_fixup_free_int_pvar(void **param, int param_no); static int fo_count_assigned_files(); static void fo_log_writer_process(int rank); static int fo_add_filename(modparam_t type, void *val); @@ -74,8 +75,9 @@ time_t fo_stored_timestamp[FO_MAX_FILES] = {0}; time_t fo_current_timestamp = 0; FILE *fo_file_output[FO_MAX_FILES]; -static cmd_export_t cmds[] = {{"file_out", (cmd_function)fo_write_to_file, 2, - fo_fixup_int_pvar, 0, ANY_ROUTE}, +static cmd_export_t cmds[] = { + {"file_out", (cmd_function)fo_write_to_file, 2, fo_fixup_int_pvar, + fo_fixup_free_int_pvar, ANY_ROUTE}, {0, 0, 0, 0, 0, 0}}; static param_export_t params[] = { @@ -266,11 +268,22 @@ static int fo_fixup_int_pvar(void **param, int param_no) if(param_no == 1) { return fo_fixup_str_index(param, param_no); } else if(param_no == 2) { - return fixup_var_pve_str_12(param, param_no); + return fixup_spve_all(param, param_no); } return 0; } +static int fo_fixup_free_int_pvar(void **param, int param_no) +{ + if(param_no == 1) { + return fixup_free_igp_null(param, param_no); + } else if(param_no == 2) { + return fixup_free_spve_all(param, param_no); + } + + return 0; +} + static int fo_add_filename(modparam_t type, void *val) { str in;

1 0

git:master:d3fbee58: file_out: Improve logging
by Xenofon Karamanos 16 Feb '24

16 Feb '24

Module: kamailio Branch: master Commit: d3fbee58cb5505130ea73cfa0143adfd470d2c27 URL: https://github.com/kamailio/kamailio/commit/d3fbee58cb5505130ea73cfa0143adf… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xenakiskaramanos(a)gmail.com> Date: 2024-02-16T12:20:44+02:00 file_out: Improve logging --- Modified: src/modules/file_out/file_out.c --- Diff: https://github.com/kamailio/kamailio/commit/d3fbee58cb5505130ea73cfa0143adf… Patch: https://github.com/kamailio/kamailio/commit/d3fbee58cb5505130ea73cfa0143adf… --- diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index d6fdd51ce9d..c985c04935a 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -195,7 +195,7 @@ static void fo_log_writer_process(int rank) } FILE *out = fo_get_file_handle(log_message.dest_file); if(out == NULL) { - LM_ERR("out is NULL\n"); + LM_ERR("file handle is NULL\n"); return; } @@ -340,7 +340,7 @@ static int fo_parse_filename_params(str in) token = strtok_r(NULL, ";", &saveptr); } if(name != NULL) { - LM_ERR("name = %s\n", name); + LM_DBG("name = %s\n", name); fo_base_filename[*fo_number_of_files] = name; } else { LM_ERR("name is required. Make sure you provided name= in modparam " @@ -414,7 +414,6 @@ static int fo_check_interval(int index) // Calculate the difference between the current timestamp and the stored timestamp int difference = difftime(fo_current_timestamp, fo_stored_timestamp[index]); if(difference >= fo_interval_seconds[index]) { - LM_ERR("interval has passed\n"); return 1; } return 0; @@ -480,7 +479,7 @@ static int fo_write_to_file(sip_msg_t *msg, char *index, char *log_message) str value = str_init(""); result = get_str_fparam(&value, msg, (fparam_t *)log_message); if(result < 0) { - LM_ERR("Failed to string from param 1: %d\n", result); + LM_ERR("Failed to get string from param 1: %d\n", result); return -1; }

1 0

git:master:6ac4148d: file_out: Free message when done
by Xenofon Karamanos 16 Feb '24

16 Feb '24

Module: kamailio Branch: master Commit: 6ac4148d3d7e6ea772438b7c5b42807175e5888a URL: https://github.com/kamailio/kamailio/commit/6ac4148d3d7e6ea772438b7c5b42807… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xenakiskaramanos(a)gmail.com> Date: 2024-02-16T12:20:44+02:00 file_out: Free message when done --- Modified: src/modules/file_out/file_out.c --- Diff: https://github.com/kamailio/kamailio/commit/6ac4148d3d7e6ea772438b7c5b42807… Patch: https://github.com/kamailio/kamailio/commit/6ac4148d3d7e6ea772438b7c5b42807… --- diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index 5815d61cb63..d6fdd51ce9d 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -207,6 +207,14 @@ static void fo_log_writer_process(int rank) if(fflush(out) < 0) { LM_ERR("Failed to flush file with err {%s}\n", strerror(errno)); } + + if(log_message.message != NULL) { + if(log_message.message->s != NULL) { + shm_free(log_message.message->s); + } + shm_free(log_message.message); + log_message.message = NULL; + } } }

1 0

git:master:e4163d28: file_out: Switch to str type for string handling
by Xenofon Karamanos 16 Feb '24

16 Feb '24

Module: kamailio Branch: master Commit: e4163d287e1c8c617ef9f21452a88f09c040bd47 URL: https://github.com/kamailio/kamailio/commit/e4163d287e1c8c617ef9f21452a88f0… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xenakiskaramanos(a)gmail.com> Date: 2024-02-16T12:20:44+02:00 file_out: Switch to str type for string handling --- 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/e4163d287e1c8c617ef9f21452a88f0… Patch: https://github.com/kamailio/kamailio/commit/e4163d287e1c8c617ef9f21452a88f0… --- diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index b52dad270d5..5815d61cb63 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -199,8 +199,14 @@ static void fo_log_writer_process(int rank) return; } - fprintf(out, "%s\n", log_message.message); - fflush(out); + if(fprintf(out, "%.*s\n", log_message.message->len, + log_message.message->s) + < 0) { + LM_ERR("Failed to write to file with err {%s}\n", strerror(errno)); + } + if(fflush(out) < 0) { + LM_ERR("Failed to flush file with err {%s}\n", strerror(errno)); + } } } @@ -463,7 +469,7 @@ static int fo_write_to_file(sip_msg_t *msg, char *index, char *log_message) return -1; } - str value; + str value = str_init(""); result = get_str_fparam(&value, msg, (fparam_t *)log_message); if(result < 0) { LM_ERR("Failed to string from param 1: %d\n", result); @@ -471,8 +477,8 @@ static int fo_write_to_file(sip_msg_t *msg, char *index, char *log_message) } /* Add the logging string to the global gueue */ - fo_log_message_t logMessage; - logMessage.message = value.s; + fo_log_message_t logMessage = {0, 0}; + logMessage.message = &value; logMessage.dest_file = file_index; fo_enqueue(fo_queue, logMessage); diff --git a/src/modules/file_out/types.c b/src/modules/file_out/types.c index a80566dfc8f..d2815333dd0 100644 --- a/src/modules/file_out/types.c +++ b/src/modules/file_out/types.c @@ -22,6 +22,7 @@ #include <stdlib.h> #include "types.h" +#include "../../core/ut.h" static fo_node_t *fo_new_node(fo_log_message_t data) { @@ -37,8 +38,18 @@ int fo_enqueue(fo_queue_t *q, fo_log_message_t data) /* Copy the contents of data.message */ - char *message_copy = (char *)shm_malloc(strlen(data.message) + 1); - strcpy(message_copy, data.message); + str *message_copy = 0; + message_copy = (str *)shm_malloc(sizeof(str)); + if(message_copy == 0) { + SHM_MEM_ERROR; + return -1; + } + + if(shm_str_dup(message_copy, data.message) < 0) { + LM_ERR("Failed to duplicate message\n"); + return -1; + } + data.message = message_copy; fo_node_t *temp = fo_new_node(data); @@ -74,10 +85,6 @@ int fo_dequeue(fo_queue_t *q, fo_log_message_t *data) if(temp != NULL) { - if(temp->data.message != NULL) { - shm_free(temp->data.message); - temp->data.message = NULL; - } shm_free(temp); temp = NULL; } diff --git a/src/modules/file_out/types.h b/src/modules/file_out/types.h index 0c81cb20a7b..c6fb72b28ef 100644 --- a/src/modules/file_out/types.h +++ b/src/modules/file_out/types.h @@ -22,7 +22,7 @@ typedef struct log_message { - char *message; + str *message; int dest_file; } fo_log_message_t;

1 0

git:master:e15a15cf: file_out: Add initial Readme and improve docs
by Xenofon Karamanos 16 Feb '24

16 Feb '24

Module: kamailio Branch: master Commit: e15a15cf8ce0c09aa6b26200dd2bce0e3933b6cd URL: https://github.com/kamailio/kamailio/commit/e15a15cf8ce0c09aa6b26200dd2bce0… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xenakiskaramanos(a)gmail.com> Date: 2024-02-16T12:20:44+02:00 file_out: Add initial Readme and improve docs --- Added: src/modules/file_out/README Modified: src/modules/file_out/doc/file_out_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/e15a15cf8ce0c09aa6b26200dd2bce0… Patch: https://github.com/kamailio/kamailio/commit/e15a15cf8ce0c09aa6b26200dd2bce0…

1 0

git:master:107d8c9a: file_out: Change default base folder format
by Xenofon Karamanos 16 Feb '24

16 Feb '24

Module: kamailio Branch: master Commit: 107d8c9ae4cb1191d9e0132a42acad8af4cc66c9 URL: https://github.com/kamailio/kamailio/commit/107d8c9ae4cb1191d9e0132a42acad8… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xenakiskaramanos(a)gmail.com> Date: 2024-02-16T12:20:44+02:00 file_out: Change default base folder format --- Modified: src/modules/file_out/file_out.c --- Diff: https://github.com/kamailio/kamailio/commit/107d8c9ae4cb1191d9e0132a42acad8… Patch: https://github.com/kamailio/kamailio/commit/107d8c9ae4cb1191d9e0132a42acad8… --- diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index d64ffdec74e..b52dad270d5 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -60,7 +60,7 @@ static int fo_add_filename(modparam_t type, void *val); static int fo_parse_filename_params(str input); /* Default parameters */ -char *fo_base_folder = "/var/log/kamailio/file_out/"; +char *fo_base_folder = "/var/log/kamailio/file_out"; char *fo_base_filename[FO_MAX_FILES] = {""}; char *fo_extension[FO_MAX_FILES] = {".out"}; int fo_interval_seconds[FO_MAX_FILES] = {10 * 60};

1 0

[kamailio/kamailio] Undocumented TCP connection limit introduced in 5.7.3 must be configurable (Issue #3755)
by Sebastian Damm 15 Feb '24

15 Feb '24

### Description Scenario: Kamailio servers running behind a loadbalancer (AWS network loadbalancer in this case). This causes all packets to come from the same IP address in Kamailio's POV. The change https://github.com/kamailio/kamailio/commit/a902e4a032a85a7755de32eeadac800… introduced a connection limit per source IP address. This obviously conflicts with the setup mentioned above. We need up to 10000 client connections, which would seem to all come from the same IP address. Apart from this limit being introduced without a changelog entry (at least I didn't find one), I would expect it to be configurable. But from the commit it looks like it is a compile time option only.  #### Log Messages ``` Feb 15 09:24:00 sipproxy /usr/sbin/kamailio[174]: CRITICAL: <core> [core/tcp_main.c:4447]: handle_new_connect(): hit the limit of connections per source IP (100.68.15.172:33334) - rejecting Feb 15 09:24:01 sipproxy /usr/sbin/kamailio[174]: CRITICAL: <core> [core/tcp_main.c:4447]: handle_new_connect(): hit the limit of connections per source IP (100.68.15.172:1153) - rejecting Feb 15 09:24:01 sipproxy /usr/sbin/kamailio[174]: CRITICAL: <core> [core/tcp_main.c:4447]: handle_new_connect(): hit the limit of connections per source IP (100.68.15.172:59946) - rejecting ``` ### Additional Information Using the packages from the official Kamailio repository. * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.7.4 (x86_64/linux) 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: unknown compiled with gcc 11.4.0 ``` * **Operating System**:  ``` Ubuntu Jammy ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3755 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3755(a)github.com>

2 3

Missing values in xhttp_prom output
by Christian Berger 15 Feb '24

15 Feb '24

Hi, we at sipgate would like to know if there is a reason why the xhttp_prom module doesn't export statistics like stats.fetch all. We are missing the tls.info as well as pkg.stats values. Is there a reason for this? Are there any pointers on how we could implement it ourselves and provide patches? Best regards Christian Berger -- Christian Berger - berger(a)sipgate.de Telefon: +49 (0)211-63 55 55-0 Telefax: +49 (0)211-63 55 55-22 sipgate GmbH - Gladbacher Str. 74 - 40219 Düsseldorf HRB Düsseldorf 39841 - Geschäftsführer: Thilo Salmon, Tim Mois Steuernummer: 106/5724/7147, Umsatzsteuer-ID: DE219349391 www.sipgate.de - www.sipgate.co.uk

2 2

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev