sr-dev

sr-dev@lists.kamailio.org

16 participants
45378 discussions

git:master:8e4c52e3: modules: readme files regenerated - tm ... [skip ci]
by Kamailio Dev 10 Jul '25

10 Jul '25

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

1 0

git:master:2131af64: tm: docs for rpc command tm.t_uac_attrs
by Daniel-Constantin Mierla 10 Jul '25

10 Jul '25

Module: kamailio Branch: master Commit: 2131af6464623a6bab1b23489cc1891faf7378b8 URL: https://github.com/kamailio/kamailio/commit/2131af6464623a6bab1b23489cc1891… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-07-10T10:51:05+02:00 tm: docs for rpc command tm.t_uac_attrs --- Modified: src/modules/tm/doc/rpc.xml --- Diff: https://github.com/kamailio/kamailio/commit/2131af6464623a6bab1b23489cc1891… Patch: https://github.com/kamailio/kamailio/commit/2131af6464623a6bab1b23489cc1891… --- diff --git a/src/modules/tm/doc/rpc.xml b/src/modules/tm/doc/rpc.xml index 00164bc5793..21b77bab2ec 100644 --- a/src/modules/tm/doc/rpc.xml +++ b/src/modules/tm/doc/rpc.xml @@ -173,6 +173,46 @@ </para> </section> + <section id="tm.rpc.t_uac_attrs"> + <title> + <function moreinfo="none">tm.t_uac_attrs</function> + </title> + <para> + Combine the tm.t_uac_* variants, by providing the first parameter with + attributes, then the other parameters are like for tm.t_uac_start. + </para> + <para> + The attributes are in the format "name1=value1;...;nameX=valueX". The + name of the attribute can be: + </para> + <itemizedlist> + <listitem><para> + <emphasis>reply</emphasis> - control how the reply is waited. If + not provided, acts like tm.t_uac_start. The values can be: + "wait" - act like tm.t_uac_wait; "block" - act like + tm.t_uac_wait_block. + </para></listitem> + <listitem><para> + <emphasis>cbflags</emphasis> - the transaction callback flags. The + values can be: "noack" - to not automatically send ACK for local + INVITE requests + </para></listitem> + <listitem><para> + <emphasis>rpflags</emphasis> - RPC parameters flags. The values + can be: 1 - the body is hexa-encoded value. + </para></listitem> + </itemizedlist> + <example> + <title>Using <quote>tm.t_uac_attrs</quote></title> + <programlisting format="linespecific"> +... +kamcli rpc tm.t_uac_attrs 'reply=wait;cbflags=noack;rpflags=1' 'INVITE' ... +... +</programlisting> + </example> + </section> + + <section id="tm.rpc.cancel"> <title> <function moreinfo="none">tm.cancel</function>

1 0

git:master:3ee78cbd: tm: added rpc command tm.t_uac_attrs
by Daniel-Constantin Mierla 10 Jul '25

10 Jul '25

Module: kamailio Branch: master Commit: 3ee78cbdf6c01aaf1c3b1dd32f12396d77519a3b URL: https://github.com/kamailio/kamailio/commit/3ee78cbdf6c01aaf1c3b1dd32f12396… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-07-10T10:51:05+02:00 tm: added rpc command tm.t_uac_attrs - combine the variants of tm.t_uac_* by providing first parameter with attributes in the format 'name=value;...' --- Modified: src/modules/tm/rpc_uac.c Modified: src/modules/tm/rpc_uac.h Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/3ee78cbdf6c01aaf1c3b1dd32f12396… Patch: https://github.com/kamailio/kamailio/commit/3ee78cbdf6c01aaf1c3b1dd32f12396… --- diff --git a/src/modules/tm/rpc_uac.c b/src/modules/tm/rpc_uac.c index dfd038a3f2c..19b9b528a5c 100644 --- a/src/modules/tm/rpc_uac.c +++ b/src/modules/tm/rpc_uac.c @@ -1004,6 +1004,68 @@ void rpc_t_uac_wait_block_noack_hex(rpc_t *rpc, void *c) rpc_t_uac(rpc, c, 2, TMCB_DONT_ACK, 1); } +/** t_uac with attributes. + * @see rpc_t_uac. + */ +void rpc_t_uac_attrs(rpc_t *rpc, void *c) +{ + tm_rpc_uac_attrs_t tattrs; + str sattrs = STR_NULL; + param_t *params_list = NULL; + param_hooks_t phooks; + param_t *pit = NULL; + int ival = 0; + int ret; + + memset(&tattrs, 0, sizeof(tm_rpc_uac_attrs_t)); + + ret = rpc->scan(c, "SSSSSS*S", &sattrs, &tattrs.method, &tattrs.ruri, + &tattrs.nexthop, &tattrs.send_socket, &tattrs.headers, + &tattrs.body); + if(ret < 6 && !(-ret == 6)) { + rpc->fault(c, 400, "too few parameters (%d/5)", ret ? ret : -ret); + return; + } + if(sattrs.len > 0) { + if(sattrs.s[sattrs.len - 1] == ';') { + sattrs.len--; + } + if(parse_params(&sattrs, CLASS_ANY, &phooks, &params_list) < 0) { + return; + } + for(pit = params_list; pit; pit = pit->next) { + if(pit->name.len == 5 + && strncasecmp(pit->name.s, "reply", 5) == 0) { + if(pit->body.len == 4 + && strncasecmp(pit->name.s, "wait", 4) == 0) { + tattrs.reply_wait = 1; + } else if(pit->body.len == 5 + && strncasecmp(pit->name.s, "block", 5) == 0) { + tattrs.reply_wait = 2; + } else { + LM_ERR("unknown reply attribute value\n"); + } + } else if(pit->name.len == 7 + && strncasecmp(pit->name.s, "cbflags", 7) == 0) { + if(pit->body.len == 5 + && strncasecmp(pit->name.s, "noack", 5) == 0) { + tattrs.cbflags |= TMCB_DONT_ACK; + } else { + LM_ERR("unknown cbflags attribute value\n"); + } + } else if(pit->name.len == 7 + && strncasecmp(pit->name.s, "rpflags", 7) == 0) { + str2sint(&pit->body, &ival); + tattrs.rpflags |= ival; + } else { + LM_ERR("unknown attribute provided\n"); + } + } + } + + rpc_t_uac_attrs_helper(rpc, c, &tattrs); +} + static int t_uac_check_msg(struct sip_msg *msg, str *method, str *body, str *fromtag, int *cseq_is, int *cseq, str *callid) { diff --git a/src/modules/tm/rpc_uac.h b/src/modules/tm/rpc_uac.h index 7826c9ef544..95d01649178 100644 --- a/src/modules/tm/rpc_uac.h +++ b/src/modules/tm/rpc_uac.h @@ -38,6 +38,8 @@ void rpc_t_uac_wait_noack_hex(rpc_t *rpc, void *c); void rpc_t_uac_wait_block_noack(rpc_t *rpc, void *c); void rpc_t_uac_wait_block_noack_hex(rpc_t *rpc, void *c); +void rpc_t_uac_attrs(rpc_t *rpc, void *c); + int t_uac_send(str *method, str *ruri, str *nexthop, str *send_socket, str *headers, str *body); diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c index 54d06950199..8997abe983b 100644 --- a/src/modules/tm/tm.c +++ b/src/modules/tm/tm.c @@ -2975,6 +2975,12 @@ static const char *rpc_t_uac_wait_block_noack_hex_doc[2] = { 0 }; +static const char *rpc_t_uac_attrs_doc[2] = { + "starts a tm uac using a list of string parameters: attributes, method," + " ruri, dst_uri, send_sock, headers (CRLF separated) and body (optional)", + 0 +}; + static const char *tm_rpc_list_doc[2] = { "List transactions.", 0 @@ -3020,6 +3026,8 @@ static rpc_export_t tm_rpc[] = { rpc_t_uac_wait_block_noack_doc, 0}, {"tm.t_uac_wait_block_noack_hex", rpc_t_uac_wait_block_noack_hex, rpc_t_uac_wait_block_noack_hex_doc, 0}, + {"tm.t_uac_attrs", rpc_t_uac_attrs, + rpc_t_uac_attrs_doc, RET_ARRAY}, {"tm.list", tm_rpc_list, tm_rpc_list_doc, RET_ARRAY}, {"tm.clean", tm_rpc_clean, tm_rpc_clean_doc, 0}, {0, 0, 0, 0}

1 0

git:master:099292b5: tm: use struct for rpc uac helper function
by Daniel-Constantin Mierla 10 Jul '25

10 Jul '25

Module: kamailio Branch: master Commit: 099292b5c19847536bc89a2d04bac9749f07415e URL: https://github.com/kamailio/kamailio/commit/099292b5c19847536bc89a2d04bac97… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-07-10T10:51:05+02:00 tm: use struct for rpc uac helper function --- Modified: src/modules/tm/rpc_uac.c --- Diff: https://github.com/kamailio/kamailio/commit/099292b5c19847536bc89a2d04bac97… Patch: https://github.com/kamailio/kamailio/commit/099292b5c19847536bc89a2d04bac97…

1 0

git:master:558e6458: carrierroute: spelling fix in comment
by Henning Westerholt 10 Jul '25

10 Jul '25

Module: kamailio Branch: master Commit: 558e6458c083d12087556808aced05589cd59669 URL: https://github.com/kamailio/kamailio/commit/558e6458c083d12087556808aced055… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-07-10T08:36:17Z carrierroute: spelling fix in comment --- Modified: src/modules/carrierroute/cr_rule.h --- Diff: https://github.com/kamailio/kamailio/commit/558e6458c083d12087556808aced055… Patch: https://github.com/kamailio/kamailio/commit/558e6458c083d12087556808aced055… --- diff --git a/src/modules/carrierroute/cr_rule.h b/src/modules/carrierroute/cr_rule.h index f0517e2f165..96f69f58633 100644 --- a/src/modules/carrierroute/cr_rule.h +++ b/src/modules/carrierroute/cr_rule.h @@ -48,7 +48,7 @@ struct route_rule double orig_prob; /*!< The original probability for that rule, only useful when using crc32 hashing */ str host; /*!< The new target host for the request */ int strip; /*!< the number of digits to be stripped off from uri before prepending prefix */ - str local_prefix; /*!< the pefix to be attached to the new destination */ + str local_prefix; /*!< the prefix to be attached to the new destination */ str local_suffix; /*!< the suffix to be appended to the localpart of the new destination */ str comment; /*!< A comment for the route rule */ str prefix; /*!< The prefix for which the route is valid */

1 0

[kamailio/kamailio] Topos not working Properly for ACK (Issue #4208)
by Pandi1981 10 Jul '25

10 Jul '25

Pandi1981 created an issue (kamailio/kamailio#4208) We are facing an issue while using TOPOS in Kamailio P-CSCF (running under Docker). Kamailio version: 6.0.0 operating system: "10 (buster)" **Call Flow & Issue Description:** Note: Public IP and Private IP mapping using the below configuration. ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ listen=udp:LOCAL_IP:5060 name "internal" listen=udp:LOCAL_IP:6080 advertise PUBLIC_IP:6080 name "external" ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ **Call Flow (INVITE):** A-Party --> Kamailio P-CSCF (Public IP) --> Kamailio P-CSCF (Private IP) --> Kamailio S-CSCF (Private IP) B-Party <-- Kamailio P-CSCF (Public IP) <-- Kamailio P-CSCF (Private IP) <-- Kamailio S-CSCF (Private IP) Everything works correctly up to the point where the 200 OK is received by the A-Party. However, when the A-Party sends the ACK, it is received by the Kamailio P-CSCF but is not forwarded to the B-Party. Upon investigation, we found that when Kamailio receives the ACK from the A-Party, the rr (Record-Route) module fails to recognize its own Route header and thus falls back to strict routing instead of performing loose routing. This results in an incorrect rewrite of the Request-URI of the ACK, causing it to fail to reach the B-Party. Essentially, the Route header is not popped as expected and the ACK routing breaks. TOPOS module is configured as follows: loadmodule "topos_redis.so" loadmodule "ndb_redis.so" loadmodule "topos.so" modparam("ndb_redis", "server", "name=tps;addr=127.0.0.1;port=6379;db=0") modparam("topos", "storage", "redis") modparam("topos_redis", "serverid", "tps") modparam("topos", "rr_update", 1) modparam("topos", "contact_mode", 1) modparam("topos", "xavu_field_a_contact", "a_contact") modparam("topos", "xavu_field_b_contact", "b_contact") modparam("topos", "xavu_cfg", "tps") Please guide us how to achieve and what we are doing wrong. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4208 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4208(a)github.com>

3 7

[kamailio/kamailio] tls: Create one shared context and reuse it. (PR #3972)
by Xenofon Karamanos 10 Jul '25

10 Jul '25

#### 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) - [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 - [x] Related to issue #3823 #### Description  This PR aims to implement what was discussed in [mailing list](https://lists.kamailio.org/mailman3/hyperkitty/list/sr-dev@lists.kama… regarding some `tls.reload` and increasing memory usage. It adds a new parameter `enable_shared_ctx` in `tls` module that if set to 0, preserves the old behavior and if set to 1 (other than 0 tbh), it creates a single SSL context that is being shared. This have the effect of using way less memory when initialized as well, but also minimizes (can't say it fixes the problem) the `tls.reload` memory increase. I have also added a small markdown (comparison.md) file, where some comparisons where made between enabled/disabled shared context and with/without CA file (where the initial problem was occurring by the reporter). Feedback would be necessary to verify whether this patch, acts as expected and kamailio works as intented. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3972 -- Commit Summary -- * tls: Add parameter for shared contexts * tls: Comparison for enable_shared_ctx -- File Changes -- A comparison.md (15) M src/modules/tls/tls_domain.c (172) M src/modules/tls/tls_mod.c (11) M src/modules/tls/tls_mod.h (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3972.patch https://github.com/kamailio/kamailio/pull/3972.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3972 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3972(a)github.com>

4 13

[kamailio/kamailio] KEMI app_python3 crash (Issue #4314)
by Denys Pozniak 09 Jul '25

09 Jul '25

denyspozniak created an issue (kamailio/kamailio#4314) ### Description The code below causes Kamailio to crash when sending a simple OPTIONS request. I agree that this is not the most proper way to call the sl function, but it's probably worth paying attention to this issue. #### Reproduction ```import sys import KSR as KSR def mod_init(): return kamailio() class kamailio: def __init__(self): KSR.info('== kamailio init ==\n') def child_init(self, rank): return 0 def ksr_request_route(self, msg): KSR.x.modf("sl_send_reply", "200", "OK"); return 1 ``` #### Debugging Data [(gdb) bt full.txt](https://github.com/user-attachments/files/21149798/gdb.bt.full.tx… [(gdb) info locals.txt](https://github.com/user-attachments/files/21149836/gdb.info.loc… ``` (gdb) list 154 in sl_funcs.c ``` #### Log Messages ``` Program received signal SIGSEGV, Segmentation fault. sl_reply_helper (msg=msg@entry=0x7ffff6fada80, code=200, reason=reason@entry=0x7ffff75d2070 "OK", tag=tag@entry=0x0) at sl_funcs.c:159 159 sl_funcs.c: No such file or directory. ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 6.1.0-dev1 (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_SEND_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 9.4.0 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4314 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4314(a)github.com>

1 0

git:master:c8e6a738: modules: readme files regenerated - rtpengine ... [skip ci]
by Kamailio Dev 09 Jul '25

09 Jul '25

Module: kamailio Branch: master Commit: c8e6a738d58f143a5be6dcd4564614d2ce88590b URL: https://github.com/kamailio/kamailio/commit/c8e6a738d58f143a5be6dcd4564614d… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-07-09T13:16:09+02:00 modules: readme files regenerated - rtpengine ... [skip ci] --- Modified: src/modules/rtpengine/README --- Diff: https://github.com/kamailio/kamailio/commit/c8e6a738d58f143a5be6dcd4564614d… Patch: https://github.com/kamailio/kamailio/commit/c8e6a738d58f143a5be6dcd4564614d…

1 0

[kamailio/kamailio] Fix documentation on rtpengine_subscribe (Issue #4308)
by tsearle 09 Jul '25

09 Jul '25

tsearle created an issue (kamailio/kamailio#4308) while rtpengine_subscribe_request is the correct method name in kemi, when using the kamailio scripting engine the correct name is rtpengine_subscribe_offer the documentation for the module needs to be updated to reflect this -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4308 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4308(a)github.com>

2 6

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev