sr-dev February 2025

sr-dev@lists.kamailio.org

9 participants
163 discussions

[kamailio/kamailio] kamcmd command doesn't allow the same arguments from one-liner call or from kamcmd console (Issue #4137)
by jflem1eux 18 Feb '25

18 Feb '25

Hello, I discovered that kamcmd command doesn't allow the same argument for a rpc command if I call a one-liner command or from the kamcmd console. it's related to ul.add command If I call this command from bash: kamcmd ul.add location 222(a)domain.com sip:222@tbd.domain.com:5088;transport=udp 0 -1.0 . 0 2240 5023 . udp:8.12.6.7:5060 the command will fail with 2 errors: error: 500 - Not enough parameters or wrong format -bash: 0: command not found I discovered if I remove ;transport=udp I get rid of the errors and I get new one: Unknown option `-1'. And if I set my Q value from -1.0 to 0.0 the command works BUT If I go into the kamcmd console and I ask the exact original command, it works and I can see in usrloc database that it accepted my Q value to -1.0 and the transport=udp string I tried to use s: to force some argument as string but it doesn't works , may be I don't use correctly I tried unsuccessfully: kamcmd ul.add location 298(a)domain.com s:sip:298@tbd.domain.com:5088;transport=udp 0 s:-1.0 . 0 2240 5023 . udp:8.12.6.7:5060 --> I use kamailio version kamailio 5.7.1 (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 10.2.1 kamcmd has been built from the same source code with the rest of the program Built on debian 11 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4137 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4137(a)github.com>

2 2

Info: maintenance work for kamailio.org server on Tuesday, Feb 18, 2025
by Daniel-Constantin Mierla 18 Feb '25

18 Feb '25

Hello, there is a planned mainenance work on the infrastructure for the server hosting kamailio.org services (among them: the main website, wiki, mailing lists) on next week, Tuesday, February 18, 2025, to start about 10:00am Berlin time (CET). Expect some intervals of unavailability, we hope they will be short and most of them not noticeable for the majority accessing these services during the day. Once the work is finished, an announcement will be sent out. The apt repository, github project and the matrix chat channels should not be affected, they can be accessed as usual and can be used a backup communication channels if anyone observes something that has to be addressed. Cheers, Daniel -- Daniel-Constantin Mierla (@ asipto.com) twitter.com/miconda -- linkedin.com/in/miconda Kamailio Consultancy, Training and Development Services -- asipto.com Kamailio Advanced Training, March 24-28, 2025 -- asipto.com Kamailio World Conference, May 12-13, 2025, Berlin -- kamailioworld.com

1 2

git:master:3f139624: modules: readme files regenerated - tm ... [skip ci]
by Kamailio Dev 18 Feb '25

18 Feb '25

Module: kamailio Branch: master Commit: 3f13962412ea0aa266f289d2fa88cdd716be6306 URL: https://github.com/kamailio/kamailio/commit/3f13962412ea0aa266f289d2fa88cdd… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-02-18T13:31:10+01:00 modules: readme files regenerated - tm ... [skip ci] --- Modified: src/modules/tm/README --- Diff: https://github.com/kamailio/kamailio/commit/3f13962412ea0aa266f289d2fa88cdd… Patch: https://github.com/kamailio/kamailio/commit/3f13962412ea0aa266f289d2fa88cdd…

1 0

git:master:6a7c8c23: etc/kamailio.cfg: use send_reply_error() instead of sl_reply_error()
by Daniel-Constantin Mierla 18 Feb '25

18 Feb '25

Module: kamailio Branch: master Commit: 6a7c8c232132078b97bd3023efaed30f369cfe63 URL: https://github.com/kamailio/kamailio/commit/6a7c8c232132078b97bd3023efaed30… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-02-18T13:21:25+01:00 etc/kamailio.cfg: use send_reply_error() instead of sl_reply_error() --- Modified: etc/kamailio.cfg --- Diff: https://github.com/kamailio/kamailio/commit/6a7c8c232132078b97bd3023efaed30… Patch: https://github.com/kamailio/kamailio/commit/6a7c8c232132078b97bd3023efaed30… --- diff --git a/etc/kamailio.cfg b/etc/kamailio.cfg index a06e09fdc77..a46f5734bfc 100644 --- a/etc/kamailio.cfg +++ b/etc/kamailio.cfg @@ -634,7 +634,7 @@ route[RELAY] { } if (!t_relay()) { - sl_reply_error(); + send_reply_error(); } exit; } @@ -742,7 +742,7 @@ route[REGISTRAR] { #!endif } if (!save("location")) { - sl_reply_error(); + send_reply_error(); } exit; } @@ -808,7 +808,7 @@ route[PRESENCE] { msg_apply_changes(); #!endif if (!t_newtran()) { - sl_reply_error(); + send_reply_error(); exit; }

1 0

git:master:357766ba: tm: docs for delayed_reply param
by Daniel-Constantin Mierla 18 Feb '25

18 Feb '25

Module: kamailio Branch: master Commit: 357766ba470401845b3ec2222e6fc11031823ee3 URL: https://github.com/kamailio/kamailio/commit/357766ba470401845b3ec2222e6fc11… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-02-18T13:20:48+01:00 tm: docs for delayed_reply param --- Modified: src/modules/tm/doc/params.xml --- Diff: https://github.com/kamailio/kamailio/commit/357766ba470401845b3ec2222e6fc11… Patch: https://github.com/kamailio/kamailio/commit/357766ba470401845b3ec2222e6fc11… --- diff --git a/src/modules/tm/doc/params.xml b/src/modules/tm/doc/params.xml index ad1f55e2061..1dd77f1df3f 100644 --- a/src/modules/tm/doc/params.xml +++ b/src/modules/tm/doc/params.xml @@ -1740,5 +1740,26 @@ modparam("tm", "reply_408_reason", "Gone") </example> </section> + <section id="tm.p.rdelayed_reply"> + <title><varname>delayed_reply</varname> (int)</title> + <para> + If set to 1, the reply generation in case of request + processing/forwarding error is delayed till the config script + execution ends. This allows the script writer to send its + own error reply. If set to 0, the transaction is replied and killed + immediately. + </para> + <emphasis> + Default value is <quote>1</quote>. + </emphasis> + <example> + <title>delayed_reply example</title> + <programlisting> +... +modparam("tm", "delayed_reply", 0) +.... + </programlisting> + </example> + </section> </section>

1 0

git:master:a705e601: tm: made TM_DELAYED_REPLY modparam delayed_reply
by Daniel-Constantin Mierla 18 Feb '25

18 Feb '25

Module: kamailio Branch: master Commit: a705e601d9924118ed6d79b21e92fd2af0267a05 URL: https://github.com/kamailio/kamailio/commit/a705e601d9924118ed6d79b21e92fd2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-02-18T13:14:16+01:00 tm: made TM_DELAYED_REPLY modparam delayed_reply - GH #4140 --- Modified: src/modules/tm/t_funcs.c Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/a705e601d9924118ed6d79b21e92fd2… Patch: https://github.com/kamailio/kamailio/commit/a705e601d9924118ed6d79b21e92fd2… --- diff --git a/src/modules/tm/t_funcs.c b/src/modules/tm/t_funcs.c index 55510742b39..dd14f83a15c 100644 --- a/src/modules/tm/t_funcs.c +++ b/src/modules/tm/t_funcs.c @@ -43,7 +43,7 @@ /* if defined t_relay* error reply generation will be delayed till script * end (this allows the script writer to send its own error reply) */ -#define TM_DELAYED_REPLY +extern int _tm_delayed_reply; #ifdef USE_DNS_FAILOVER extern int **failover_reply_codes; @@ -232,9 +232,7 @@ int t_relay_to( unsigned short port; str host; short comp; -#ifndef TM_DELAYED_REPLY int reply_ret; -#endif ret = 0; @@ -354,26 +352,25 @@ int t_relay_to( ret = -4; goto done; } -#ifdef TM_DELAYED_REPLY - /* current error in tm_error */ - tm_error = ser_error; - set_kr(REQ_ERR_DELAYED); - LM_DBG("%d error reply generation delayed \n", ser_error); -#else - - reply_ret = kill_transaction(t, ser_error); - if(reply_ret > 0) { - /* we have taken care of all -- do nothing in - script */ - LM_DBG("generation of a stateful reply " - "on error succeeded\n"); - /*ret=0; -- we don't want to stop the script */ + if(_tm_delayed_reply != 0) { + /* current error in tm_error */ + tm_error = ser_error; + set_kr(REQ_ERR_DELAYED); + LM_DBG("%d error reply generation delayed \n", ser_error); } else { - LM_DBG("generation of a stateful reply " - "on error failed\n"); - t_release_transaction(t); + reply_ret = kill_transaction(t, ser_error); + if(reply_ret > 0) { + /* we have taken care of all -- do nothing in + script */ + LM_DBG("generation of a stateful reply " + "on error succeeded\n"); + /*ret=0; -- we don't want to stop the script */ + } else { + LM_DBG("generation of a stateful reply " + "on error failed\n"); + t_release_transaction(t); + } } -#endif /* TM_DELAYED_REPLY */ } else { t_release_transaction(t); /* kill it silently */ } diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c index b10200dab4e..d6f7423b3c1 100644 --- a/src/modules/tm/tm.c +++ b/src/modules/tm/tm.c @@ -229,6 +229,7 @@ int tm_remap_503_500 = 1; str _tm_event_callback_lres_sent = {NULL, 0}; int _tm_reply_408_code = 408; str _tm_reply_408_reason = str_init("Request Timeout"); +int _tm_delayed_reply = 1; #ifdef USE_DNS_FAILOVER str failover_reply_codes_str = {NULL, 0}; @@ -528,6 +529,7 @@ static param_export_t params[] = { #endif {"reply_408_code", PARAM_INT, &_tm_reply_408_code}, {"reply_408_reason", PARAM_STR, &_tm_reply_408_reason}, + {"delayed_reply", PARAM_INT, &_tm_delayed_reply}, {0, 0, 0} };

1 0

[kamailio/kamailio] Enhance TOPOH and TOPOS Modules to Mask IPs in Request-URI and From/To Headers (Issue #4148)
by Farzaneh Soltanzadeh 18 Feb '25

18 Feb '25

Fr-Soltanzadeh created an issue (kamailio/kamailio#4148)  ### Description  Kamailio's topoh and topos modules currently hide/strip certain SIP headers but do not mask IP addresses in the From, To, and Request-URI headers. This feature request proposes extending these modules to hide or replace IP addresses for improved privacy and security. ### Expected behavior When enabled, Kamailio should replace IPs in: From header To header Request-URI This should work without breaking SIP message integrity or routing. #### Actual observed behavior Currently, IP addresses in these headers remain visible, exposing topology information(172.16.100.88 in this example). ``` Request-Line: INVITE sip:9121001111@172.16.100.88 SIP/2.0 Message Header Record-Route: <sip:10.14.17.83:5060;r2=on;lr;did=24f.c2e1> Record-Route: <sip:127.0.0.8;line=sr-erOb.UMbIUMBIUMZIUpA.Z1TwVXL.rlT.rHP8mBTyxnLnAjOaG**> Record-Route: <sip:127.0.0.8;line=sr-erOb.UMbIUMBIUMZIUpB.UNbyUuEeU1X7r5E741E8xODwd1B8Y5TyUN*> Record-Route: <sip:127.0.0.8;line=sr-erOb.UMbIUMBIUMZIUpB.Z1TwVXL.rlT.rHP8mBTyxnLaUn3> Record-Route: <sip:127.0.0.8;line=sr-erOb.UMbIUMBIUMZIUpT.UNbyUuEeU1X7r5E741E8xODwd1B8YfUaAul> Record-Route: <sip:127.0.0.8;line=sr-erOb.UMbIUMBIUMZIUpT.UNbyUMEo4jt7Cyb7ZjBw-HceA9TaU3s7U9ceU9DhVGXaUHRIRaAamM*> Record-Route: <sip:127.0.0.8;line=sr-erOb.UMbIUMBIUMZIUp5.UNbyUMEo4jt7Cyb7ZjBw-HceA9TaU3s7U9ceU9DhVGXaUHRIUyU8UM*> Record-Route: <sip:127.0.0.8;line=sr-erOb.UMZaY5lyY5lamuL.mpEeU1X7r5E741E8xODwd1B8Y5Anrnl> Via: SIP/2.0/UDP 10.14.17.83:5060;branch=z9hG4bKa514.f0b2dd496995e05c455b7c66f29e298a.0 Via: SIP/2.0/UDP 127.0.0.8;branch=z9hG4bKsr-NBOGIA1LaFXkHKupaduLadGLadeL.mGvyduram9TexXTomB3amnb.3H1wVHPoU9YeRKLnrpXzUO2HAHYSrM3adGLamyDaADZaUKt8m8UyAKUyUN5amGB8UKOarKDaAnBamnLau** Via: SIP/2.0/UDP 127.0.0.8;branch=z9hG4bKsr-NBOGIA1LaFXkHKupaduLadGLadeL.m1vyduram9TexXTomB3amnb.rjTnVfUhm3v.Vt4yxjIndNlyFfY8mpfnUnT8daf8VjYaU6D.xkO8m6R8der8dyUaAnZaS5b.rDXab** Via: SIP/2.0/UDP 127.0.0.8;branch=z9hG4bKsr-NBOGIA1LaFXNdKapaduLadGLadeL.mpvydurad9TexXTomBram1Z.m9YeRKLnrpXzUO2HAHYSrM3adGL8x8OyUeb.mD58mMZ8mjtnVNZax8R8dyU8dpByRNryVNLau** Via: SIP/2.0/UDP 127.0.0.8;branch=z9hG4bKsr-NBOGIA1LaFXkHKupadDTIUMr.F5TaF55ym2raUMB.d9T8VyOh-8O8mBl.d1Ladn5IU1bIUpB.Zjb7ZjBwdnTadGf.rjTnVfUhm3v.Vt4yxjINxPUad6OydaTnrNryrNBadyD.mabyrHtameZaVNbnAjYab** Max-Forwards: 66 From: <sip:9122001003@172.16.100.88>;tag=973f3044c163413fb5e7075113f96a7b To: <sip:9121001111@172.16.100.88> Contact: <sip:127.0.0.8;line=sr-erOb.UDlaU1bamMbamyuadDTIUMr.F5TaF55ym2raUMB.d9snp**> Call-ID: !!:yrnlaAtYadHOnRyRymjOydpT.d1faVKUymkRamalnAu* CSeq: 9661 INVITE Allow: PRACK, INVITE, ACK, BYE, CANCEL, UPDATE, INFO, SUBSCRIBE, NOTIFY, REFER, MESSAGE, OPTIONS Supported: replaces, 100rel, timer, norefersub Session-Expires: 1800 Min-SE: 90 User-Agent: MicroSIP/3.21.5 Content-Type: application/sdp Content-Length: 338 Message Body ``` #### Debugging Data ``` (paste your debugging data here) ``` #### Log Messages  ``` (paste your log messages here) ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions  * Introduce a new configuration parameter in topoh and topos that enables masking or encoding of IP addresses in From, To, and Request-URI. * Ensure proper handling of subsequent SIP messages like ACK, BYE, and re-INVITE to maintain end-to-end communication. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` kamailio 6.0.0 ``` * **Operating System**:  ``` uname ubuntu:22.04 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4148 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4148(a)github.com>

2 2

[kamailio/kamailio] tcp_timer_check_connections() logs on CRITICAL and is not caught by event_route[tcp:timeout] (Issue #3995)
by Fritjof Höst 18 Feb '25

18 Feb '25

### Description Hi! We are in the progress of upgrading from Kamailio 5.5 to 5.8. During our testing we have noticed a new error being reported from Kamailio. We don’t see any other errors following it. ```jsx /usr/sbin/kamailio[201]: CRITICAL: <core> [core/tcp_main.c:5544]: tcp_timer_check_connections(): message processing timeout on connection id: 67896 (state: 3) - closing ``` It does seem to be [new code](https://github.com/kamailio/kamailio/blob/master/src/core/tcp_main.c#… in Kamailio reporting this issue. Given that this is a fairly expected thing, cleaning up a connection which receives no traffic within the given time, is there a need for it to be reported on CRITICAL? I’d also expect it to be caught by ``` event_route[tcp:timeout] { xlog("L_INFO","connection $conid timeouts (unanswered keepalives)"); } ``` given that [the description](https://www.kamailio.org/docs/modules/stable/modules/tcpops.ht… of this one is `Called for connection timeouts (unanswered keepalives).`. ### Troubleshooting We don't have any way to reproduce it, we are still investigating it to figure out the cause. It happens around every 2 hours, so we think there might some some scheduled test or job running in our test system causing this. ### Additional Information * **Kamailio Version** - 5.8.3 * **Operating System**: Debian 12 AMI from the AWS Marketplace. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3995 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3995(a)github.com>

5 6

[kamailio/kamailio] kafka: Fix mod_destroy() crash (PR #4147)
by Stefan Mititelu 17 Feb '25

17 Feb '25

A crash happened on mod_destroy() due to librdkafka thrd_join(), in case of multiple restarts, in case of ssl kafka connection. Init kafka only for timer and routing processes. Add checks for rk pointer.   #### 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 - [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  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4147 -- Commit Summary -- * kafka: Fix mod_destroy() crash -- File Changes -- M src/modules/kafka/kafka_mod.c (9) M src/modules/kafka/kfk.c (95) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4147.patch https://github.com/kamailio/kamailio/pull/4147.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4147 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4147(a)github.com>

2 4

git:master:10012eeb: kafka: Fix mod_destroy() crash
by Victor Seva 17 Feb '25

17 Feb '25

Module: kamailio Branch: master Commit: 10012eeb17210ea7eb9549dc71be5af7e92b4a97 URL: https://github.com/kamailio/kamailio/commit/10012eeb17210ea7eb9549dc71be5af… Author: Stefan Mititelu <stefan.mititelu(a)net2phone.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-02-17T16:20:41+01:00 kafka: Fix mod_destroy() crash A crash happened on mod_destroy() due to librdkafka thrd_join(), in case of multiple restarts, in case of ssl kafka connection. Init kafka only for timer and routing processes. Add checks for rk pointer. --- Modified: src/modules/kafka/kafka_mod.c Modified: src/modules/kafka/kfk.c --- Diff: https://github.com/kamailio/kamailio/commit/10012eeb17210ea7eb9549dc71be5af… Patch: https://github.com/kamailio/kamailio/commit/10012eeb17210ea7eb9549dc71be5af…

1 0

← Newer
1
...
5
6
7
8
9
10
11
...
17
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 2025