- sr-dev - kamailio.org

git:master:465fda0f: core: implement pop of lists from msg env
by Daniel-Constantin Mierla 30 May '25

30 May '25

Module: kamailio Branch: master Commit: 465fda0f790e44e5484b2ef41363fb23fece3f41 URL: https://github.com/kamailio/kamailio/commit/465fda0f790e44e5484b2ef41363fb2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-05-30T12:07:46+02:00 core: implement pop of lists from msg env --- Modified: src/core/srapi.c --- Diff: https://github.com/kamailio/kamailio/commit/465fda0f790e44e5484b2ef41363fb2… Patch: https://github.com/kamailio/kamailio/commit/465fda0f790e44e5484b2ef41363fb2… --- diff --git a/src/core/srapi.c b/src/core/srapi.c index 8d7bd1ca620..a16acf97d21 100644 --- a/src/core/srapi.c +++ b/src/core/srapi.c @@ -79,7 +79,6 @@ void ksr_msg_env_push(ksr_msg_env_t *menv) { menv->route_type = get_route_type(); - /* make available the avp list from transaction */ menv->avps_uri_from = set_avp_list(AVP_TRACK_FROM | AVP_CLASS_URI, NULL); menv->avps_uri_to = set_avp_list(AVP_TRACK_TO | AVP_CLASS_URI, NULL); menv->avps_user_from = set_avp_list(AVP_TRACK_FROM | AVP_CLASS_USER, NULL); @@ -100,5 +99,16 @@ void ksr_msg_env_push(ksr_msg_env_t *menv) void ksr_msg_env_pop(ksr_msg_env_t *menv) { set_route_type(menv->route_type); + + set_avp_list(AVP_TRACK_FROM | AVP_CLASS_URI, menv->avps_uri_from); + set_avp_list(AVP_TRACK_TO | AVP_CLASS_URI, menv->avps_uri_to); + set_avp_list(AVP_TRACK_FROM | AVP_CLASS_USER, menv->avps_user_from); + set_avp_list(AVP_TRACK_TO | AVP_CLASS_USER, menv->avps_user_to); + set_avp_list(AVP_TRACK_FROM | AVP_CLASS_DOMAIN, menv->avps_domain_from); + set_avp_list(AVP_TRACK_TO | AVP_CLASS_DOMAIN, menv->avps_domain_to); + xavp_set_list(menv->xavps); + xavu_set_list(menv->xavus); + xavi_set_list(menv->xavis); + return; }

1 0

git:master:7fed8f52: jansson: docs - added missing semicolon in the example
by Daniel-Constantin Mierla 30 May '25

30 May '25

Module: kamailio Branch: master Commit: 7fed8f52391aaef43d5cb672f4821d50d4855830 URL: https://github.com/kamailio/kamailio/commit/7fed8f52391aaef43d5cb672f4821d5… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-05-30T11:53:06+02:00 jansson: docs - added missing semicolon in the example --- Modified: src/modules/jansson/doc/jansson_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/7fed8f52391aaef43d5cb672f4821d5… Patch: https://github.com/kamailio/kamailio/commit/7fed8f52391aaef43d5cb672f4821d5… --- diff --git a/src/modules/jansson/doc/jansson_admin.xml b/src/modules/jansson/doc/jansson_admin.xml index daa5e9ea51e..71ffa92bff9 100644 --- a/src/modules/jansson/doc/jansson_admin.xml +++ b/src/modules/jansson/doc/jansson_admin.xml @@ -94,7 +94,7 @@ if(!jansson_get("inner.deep.list[3]", "$var(myjson)", "$var(n)")) { } xlog("L_INFO", "foo is $var(n)"); ... -jansson_get("test", "{\"test\":\"abc\",\"idx\":20}", "$var(n)") +jansson_get("test", "{\"test\":\"abc\",\"idx\":20}", "$var(n)"); ... </programlisting> </example> @@ -313,4 +313,3 @@ xlog("foo.foz is $var(foofoz)"); </section> </section> </chapter> -

1 0

[kamailio/kamailio] Sca in cluster/load balancer (Issue #4267)
by Maurotb 29 May '25

29 May '25

Maurotb created an issue (kamailio/kamailio#4267) We have two kamailio in load balancing, mysql is in cluster mode. We need to use sca, but kamailio db cluster is not supported (i need to specify connection string to sql not cluster name) and if phone1 is registered on kamailio1 and phone2 on kamailio2, sca not work. It is possibile to implement cluster? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4267 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4267(a)github.com>

2 2

[kamailio/kamailio] Disable IPSec in P-CSCF (Issue #4269)
by navjyoty 29 May '25

29 May '25

navjyoty created an issue (kamailio/kamailio#4269) I'm working on a custom IMS deployment using the pcscf module in Kamailio. For our setup, we do not want Kamailio to handle IPsec or generate Security-Server headers in 401/407 responses. Is there a clean way to disable IPsec functionality entirely in the pcscf module? Specifically, I would like to: Prevent the module from requiring IPsec setup during UE registration Avoid any modification of SIP messages related to IPsec (e.g., removing Security-Verify, Security-Client, or Security-Server headers) Use only TLS or UDP without IPsec involvement Has anyone achieved this via configuration or by disabling specific features? Any guidance or examples would be greatly appreciated. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4269 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4269(a)github.com>

2 2

[kamailio/kamailio] SIP Race condition when using dlg.end_dlg (Issue #4268)
by IgorG 29 May '25

29 May '25

IgorrG created an issue (kamailio/kamailio#4268) ### Description We use RPC dlg.end_dlg command to terminate dialog. In most cases it does operate properly and send BYE from kamailio to both sides of dialog (as show on screenshot). ![Image](https://github.com/user-attachments/assets/b49164a5-6442-4c66-bd4d-f5412678247f) We found that in some cases we have only first BYE sent to calling party. Second BYE never sent, when called party tries to terminate call by timeout it get responded with 481. ![Image](https://github.com/user-attachments/assets/10c90b4c-30a3-4123-8e2b-2ca64cdec3c3) ![Image](https://github.com/user-attachments/assets/f0facce8-3acf-4049-8a23-ad559874af7b) In logs we have such warning: мая 27 15:11:46 service-proxy.iqtek.ru kamailio[3518388]: 2(3518388) WARNING: {1 1 BYE 541d3f2c-43fd-43b6-af24-bad1fbf85eb6} dialog [dlg_handlers.c:1343]: dlg_onroute(): unable to find dialog for BYE with route param 'e1e.4f22' [3614:8948] and call-id '541d3f2c-43fd-43b6-af24-bad1fbf85eb6' We also found that in this case this warning shown in logs 0.04 seconds after kamailio receive 200ok. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` # ./kamailio -V version: kamailio 5.8.6 (x86_64/linux) fb71db 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: fb71db compiled on 17:38:27 May 26 2025 with gcc 12.2.0 ``` * **Operating System**: ``` # lsb_release -a No LSB modules are available. Distributor ID: Debian Description: Debian GNU/Linux 12 (bookworm) Release: 12 Codename: bookworm # uname -a Linux rtp-kamailio2 6.1.0-32-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.129-1 (2025-03-06) x86_64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4268 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4268(a)github.com>

1 0

git:master:54a330bb: core: helper functions to get/set msg env lists
by Daniel-Constantin Mierla 28 May '25

28 May '25

Module: kamailio Branch: master Commit: 54a330bb707fb6b4943ba26356891696681a02e7 URL: https://github.com/kamailio/kamailio/commit/54a330bb707fb6b4943ba2635689169… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-05-28T21:28:47+02:00 core: helper functions to get/set msg env lists --- Modified: src/core/srapi.c Modified: src/core/srapi.h --- Diff: https://github.com/kamailio/kamailio/commit/54a330bb707fb6b4943ba2635689169… Patch: https://github.com/kamailio/kamailio/commit/54a330bb707fb6b4943ba2635689169… --- diff --git a/src/core/srapi.c b/src/core/srapi.c index 99fbf49d8a9..8d7bd1ca620 100644 --- a/src/core/srapi.c +++ b/src/core/srapi.c @@ -19,7 +19,9 @@ #include <stdio.h> #include <stdlib.h> + #include "dprint.h" +#include "route.h" #include "srapi.h" @@ -69,3 +71,34 @@ sr_cfgenv_t *sr_cfgenv_get(void) { return &_sr_cfgenv; } + +/** + * + */ +void ksr_msg_env_push(ksr_msg_env_t *menv) +{ + menv->route_type = get_route_type(); + + /* make available the avp list from transaction */ + menv->avps_uri_from = set_avp_list(AVP_TRACK_FROM | AVP_CLASS_URI, NULL); + menv->avps_uri_to = set_avp_list(AVP_TRACK_TO | AVP_CLASS_URI, NULL); + menv->avps_user_from = set_avp_list(AVP_TRACK_FROM | AVP_CLASS_USER, NULL); + menv->avps_user_to = set_avp_list(AVP_TRACK_TO | AVP_CLASS_USER, NULL); + menv->avps_domain_from = + set_avp_list(AVP_TRACK_FROM | AVP_CLASS_DOMAIN, NULL); + menv->avps_domain_to = set_avp_list(AVP_TRACK_TO | AVP_CLASS_DOMAIN, NULL); + menv->xavps = xavp_set_list(NULL); + menv->xavus = xavu_set_list(NULL); + menv->xavis = xavi_set_list(NULL); + + return; +} + +/** + * + */ +void ksr_msg_env_pop(ksr_msg_env_t *menv) +{ + set_route_type(menv->route_type); + return; +} diff --git a/src/core/srapi.h b/src/core/srapi.h index 1f5f8e7d36b..b573493a62b 100644 --- a/src/core/srapi.h +++ b/src/core/srapi.h @@ -20,6 +20,8 @@ #define __SRAPI_H__ #include "str.h" +#include "usr_avp.h" +#include "xavp.h" #include "parser/msg_parser.h" typedef void (*sr_generate_callid_f)(str *); @@ -39,4 +41,21 @@ typedef struct sr_cfgenv void sr_cfgenv_init(void); sr_cfgenv_t *sr_cfgenv_get(void); +typedef struct ksr_msg_env +{ + int route_type; + avp_list_t *avps_user_from; + avp_list_t *avps_user_to; + avp_list_t *avps_domain_from; + avp_list_t *avps_domain_to; + avp_list_t *avps_uri_from; + avp_list_t *avps_uri_to; + sr_xavp_t **xavps; + sr_xavp_t **xavus; + sr_xavp_t **xavis; +} ksr_msg_env_t; + +void ksr_msg_env_push(ksr_msg_env_t *menv); +void ksr_msg_env_pop(ksr_msg_env_t *menv); + #endif

1 0

git:master:f0291728: ims_ipsec_pcscf: jump to error for clean up
by Daniel-Constantin Mierla 28 May '25

28 May '25

Module: kamailio Branch: master Commit: f0291728b7fb623faf1d4de0f3152e782a6782ae URL: https://github.com/kamailio/kamailio/commit/f0291728b7fb623faf1d4de0f3152e7… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-05-28T21:28:46+02:00 ims_ipsec_pcscf: jump to error for clean up --- Modified: src/modules/ims_ipsec_pcscf/cmd.c --- Diff: https://github.com/kamailio/kamailio/commit/f0291728b7fb623faf1d4de0f3152e7… Patch: https://github.com/kamailio/kamailio/commit/f0291728b7fb623faf1d4de0f3152e7… --- diff --git a/src/modules/ims_ipsec_pcscf/cmd.c b/src/modules/ims_ipsec_pcscf/cmd.c index e15de4e2de8..4672a45906a 100644 --- a/src/modules/ims_ipsec_pcscf/cmd.c +++ b/src/modules/ims_ipsec_pcscf/cmd.c @@ -272,7 +272,7 @@ static int fill_contact( if((srcip = pkg_malloc(50)) == NULL) { LM_ERR("Error allocating memory for source IP address\n"); - return -1; + goto error; } memcpy(srcip, alias_s.s, ci->received_host.len);

1 0

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

28 May '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 12

[kamailio/kamailio] core: local TCP socket is bound on listening address (PR #3925)
by sergey-safarov 28 May '25

28 May '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 - [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: - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description As a local IP address for TCP sending operation the Kamailio service is taking the same network_interface/IP_address, which is used by the service for TCP listening. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3925 -- Commit Summary -- * core: local TCP socket is bound on listening address -- File Changes -- M src/core/tcp_main.c (24) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3925.patch https://github.com/kamailio/kamailio/pull/3925.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3925 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3925(a)github.com>

5 16

Re: [kamailio/kamailio] pkg/kamailio: switched RPM packaging to use cmake (9a1b9d0)
by Xenofon Karamanos 27 May '25

27 May '25

Same -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/9a1b9d08a112745663f37f0dfcbe86a… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/9a1b9d08a112745663f37f0dfcbe86a741486da3/158142745(a)github.com>

1 0

Re: [kamailio/kamailio] pkg/kamailio: switched RPM packaging to use cmake (9a1b9d0)
by Xenofon Karamanos 27 May '25

27 May '25

@sergey-safarov I look a bit on this commit and some comments i can give are: 1) The config files of these modules are already installed by default when they are included to be built. No need for extra step. 2) These *-cfg components are no longer available after the refactoring to use build groups instead. Can you maybe confirm that? I expect that it silently does nothing instead of complaining... -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/9a1b9d08a112745663f37f0dfcbe86a… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/9a1b9d08a112745663f37f0dfcbe86a741486da3/158142732(a)github.com>

1 0

git:master:6ccf41b3: modules: readme files regenerated - pdb ... [skip ci]
by Kamailio Dev 27 May '25

27 May '25

Module: kamailio Branch: master Commit: 6ccf41b3e2c27b9f790aa9cc881eeca1054cb20c URL: https://github.com/kamailio/kamailio/commit/6ccf41b3e2c27b9f790aa9cc881eeca… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-05-27T16:01:09+02:00 modules: readme files regenerated - pdb ... [skip ci] --- Modified: src/modules/pdb/README --- Diff: https://github.com/kamailio/kamailio/commit/6ccf41b3e2c27b9f790aa9cc881eeca… Patch: https://github.com/kamailio/kamailio/commit/6ccf41b3e2c27b9f790aa9cc881eeca… --- diff --git a/src/modules/pdb/README b/src/modules/pdb/README index 52ebab2e8ae..0c2ec72e122 100644 --- a/src/modules/pdb/README +++ b/src/modules/pdb/README @@ -12,6 +12,11 @@ Pawel Kuzak 1&1 Internet AG <pawel.kuzak(a)1und1.de> +Muhammad Shahzad Shafi + + 1&1 Internet AG + <muhammad.shafi(a)1und1.de> + Copyright © 2009 1&1 Internet AG __________________________________________________________________ @@ -38,8 +43,9 @@ Pawel Kuzak 5. RPC Commands 5.1. pdb.status - 5.2. pdb.activate - 5.3. pdb.deactivate + 5.2. pdb.timeout + 5.3. pdb.activate + 5.4. pdb.deactivate List of Examples @@ -48,8 +54,9 @@ Pawel Kuzak 1.3. Set ll_info parameter 1.4. pdb_query usage 1.5. pdb.status usage - 1.6. pdb.activate usage - 1.7. pdb.deactivate usage + 1.6. pdb.timeout usage + 1.7. pdb.activate usage + 1.8. pdb.deactivate usage Chapter 1. Admin Guide @@ -74,8 +81,9 @@ Chapter 1. Admin Guide 5. RPC Commands 5.1. pdb.status - 5.2. pdb.activate - 5.3. pdb.deactivate + 5.2. pdb.timeout + 5.3. pdb.activate + 5.4. pdb.deactivate 1. Overview @@ -178,8 +186,9 @@ cr_route("$avp(i:82)", "$rd", "$rU", "$rU", "call_id"); 5. RPC Commands 5.1. pdb.status - 5.2. pdb.activate - 5.3. pdb.deactivate + 5.2. pdb.timeout + 5.3. pdb.activate + 5.4. pdb.deactivate 5.1. pdb.status @@ -191,22 +200,35 @@ cr_route("$avp(i:82)", "$rd", "$rU", "$rU", "call_id"); kamcmd pdb.status ... -5.2. pdb.activate +5.2. pdb.timeout + + Prints the current PDB query timeout value. This can also be used to + set the PDB query timeout. + + Example 1.6. pdb.timeout usage +... +# get the PDB query timeout +kamcmd pdb.timeout +# set the PDB query timeout to 10ms +kamcmd pdb.timeout 10 +... + +5.3. pdb.activate Activates the module. This is the default after loading the module. - Example 1.6. pdb.activate usage + Example 1.7. pdb.activate usage ... kamcmd pdb.activate ... -5.3. pdb.deactivate +5.4. pdb.deactivate Deactivates the module. No more queries are performed until it is activated again. As long as the module is deactivated, the pdb_query function will return -1. - Example 1.7. pdb.deactivate usage + Example 1.8. pdb.deactivate usage ... kamcmd pdb.deactivate ...

1 0

[kamailio/kamailio] PDB: new RPC command to get/set the PDB query timeout value (PR #4261)
by Muhammad Shahzad Shafi 27 May '25

27 May '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:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  Adds RPC command to view and/or change the timeout value for PDB queries. The documentation is also updated accordingly. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4261 -- Commit Summary -- * pdb: add new RPC command to view/change PDB query timeout * pdb: add documentation for new RPC command to view/change PDB query timeout * Merge branch 'kamailio:master' into master -- File Changes -- M src/modules/pdb/doc/pdb.xml (6) M src/modules/pdb/doc/pdb_admin.xml (18) M src/modules/pdb/pdb.c (35) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4261.patch https://github.com/kamailio/kamailio/pull/4261.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4261 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4261(a)github.com>

2 5

git:master:db903613: pdb: add documentation for new RPC command to view/change PDB query timeout
by Henning Westerholt 27 May '25

27 May '25

Module: kamailio Branch: master Commit: db903613a10ae93ce0b8ff93835486953df4437e URL: https://github.com/kamailio/kamailio/commit/db903613a10ae93ce0b8ff938354869… Author: Muhammad Shafi <muhammad.shafi(a)1und1.de> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-05-27T15:56:13+02:00 pdb: add documentation for new RPC command to view/change PDB query timeout --- Modified: src/modules/pdb/doc/pdb.xml Modified: src/modules/pdb/doc/pdb_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/db903613a10ae93ce0b8ff938354869… Patch: https://github.com/kamailio/kamailio/commit/db903613a10ae93ce0b8ff938354869… --- diff --git a/src/modules/pdb/doc/pdb.xml b/src/modules/pdb/doc/pdb.xml index 835353ca359..686cf6ab0be 100644 --- a/src/modules/pdb/doc/pdb.xml +++ b/src/modules/pdb/doc/pdb.xml @@ -24,6 +24,12 @@ <affiliation><orgname>1&1 Internet AG</orgname></affiliation> <email>pawel.kuzak(a)1und1.de</email> </editor> + <editor> + <firstname>Muhammad Shahzad</firstname> + <surname>Shafi</surname> + <affiliation><orgname>1&1 Internet AG</orgname></affiliation> + <email>muhammad.shafi(a)1und1.de</email> + </editor> </authorgroup> <copyright> <year>2009</year> diff --git a/src/modules/pdb/doc/pdb_admin.xml b/src/modules/pdb/doc/pdb_admin.xml index 19b00bd99d8..50fb4190dae 100644 --- a/src/modules/pdb/doc/pdb_admin.xml +++ b/src/modules/pdb/doc/pdb_admin.xml @@ -163,6 +163,24 @@ cr_route("$avp(i:82)", "$rd", "$rU", "$rU", "call_id"); <programlisting format="linespecific"> ... &kamcmd; pdb.status +... + </programlisting> + </example> + </section> + <section id="pdb.r.timeout"> + <title>pdb.timeout</title> + <para> + Prints the current PDB query timeout value. + This can also be used to set the PDB query timeout. + </para> + <example> + <title><function>pdb.timeout</function> usage</title> + <programlisting format="linespecific"> +... +# get the PDB query timeout +&kamcmd; pdb.timeout +# set the PDB query timeout to 10ms +&kamcmd; pdb.timeout 10 ... </programlisting> </example>

1 0

git:master:e2ab3174: pdb: add new RPC command to view/change PDB query timeout
by Henning Westerholt 27 May '25

27 May '25

Module: kamailio Branch: master Commit: e2ab3174fb8f013e0a536c0cd5b049f3e2015eec URL: https://github.com/kamailio/kamailio/commit/e2ab3174fb8f013e0a536c0cd5b049f… Author: Muhammad Shafi <muhammad.shafi(a)1und1.de> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-05-27T15:56:13+02:00 pdb: add new RPC command to view/change PDB query timeout --- Modified: src/modules/pdb/pdb.c --- Diff: https://github.com/kamailio/kamailio/commit/e2ab3174fb8f013e0a536c0cd5b049f… Patch: https://github.com/kamailio/kamailio/commit/e2ab3174fb8f013e0a536c0cd5b049f… --- diff --git a/src/modules/pdb/pdb.c b/src/modules/pdb/pdb.c index 492fa99ec09..77806e49df6 100644 --- a/src/modules/pdb/pdb.c +++ b/src/modules/pdb/pdb.c @@ -27,6 +27,8 @@ #include "../../core/sr_module.h" +#include "../../core/cfg/cfg.h" +#include "../../core/cfg/cfg_ctx.h" #include "../../core/mem/mem.h" #include "../../core/mem/shm_mem.h" #include "../../core/rpc_lookup.h" @@ -806,6 +808,36 @@ static void pdb_rpc_status(rpc_t *rpc, void *ctx) (*active) ? "active" : "inactive"); } +static void pdb_rpc_timeout(rpc_t *rpc, void *ctx) +{ + void *vh; + int new_val = -1; + int old_val = cfg_get(pdb, pdb_cfg, timeout); + + str gname = str_init("pdb"); + str vname = str_init("timeout"); + + if(rpc->add(ctx, "{", &vh) < 0) { + rpc->fault(ctx, 500, "Server error"); + return; + } + + if(rpc->scan(ctx, "*d", &new_val) < 1) { + if(rpc->struct_add(vh, "d", "timeout", old_val) < 0) { + rpc->fault(ctx, 500, "Internal error reply structure"); + } + return; + } + + if(rpc->struct_add(vh, "dd", "old_timeout", old_val, "new_timeout", new_val) + < 0) { + rpc->fault(ctx, 500, "Internal error reply structure"); + return; + } + + cfg_set_now_int(ctx, &gname, NULL, &vname, new_val); +} + static void pdb_rpc_activate(rpc_t *rpc, void *ctx) { if(active == NULL) { @@ -826,11 +858,14 @@ static void pdb_rpc_deactivate(rpc_t *rpc, void *ctx) static const char *pdb_rpc_status_doc[2] = {"Get the pdb status.", 0}; +static const char *pdb_rpc_timeout_doc[2] = {"Set the pdb_query timeout.", 0}; + static const char *pdb_rpc_activate_doc[2] = {"Activate pdb.", 0}; static const char *pdb_rpc_deactivate_doc[2] = {"Deactivate pdb.", 0}; rpc_export_t pdb_rpc[] = {{"pdb.status", pdb_rpc_status, pdb_rpc_status_doc, 0}, + {"pdb.timeout", pdb_rpc_timeout, pdb_rpc_timeout_doc, 0}, {"pdb.activate", pdb_rpc_activate, pdb_rpc_activate_doc, 0}, {"pdb.deactivate", pdb_rpc_deactivate, pdb_rpc_deactivate_doc, 0}, {0, 0, 0, 0}};

1 0

[kamailio/kamailio] tls: add WSS to RPC funtions (PR #4259)
by Benjamin 27 May '25

27 May '25

- tls_list() add PROTO_WSS to TLS_LIST RPC call to include WSS connections in tls.list - tls_kill() add PROTO_WSS to handle WSS connections Co-authored-by: Andreas Tarp <tarp(a)sipgate.de> #### 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 - [ ] 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 #4167 #### Description WSS connection is not shown in tls.list RPC command like all other TLS connections. Also tls_kill does not handle WSS connections. This is related to the issue 4167. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4259 -- Commit Summary -- * http_client: add information about parameter loading * http_client: docs - typos from previous commit * tls: add WSS to RPC funtions -- File Changes -- M src/modules/http_client/doc/http_client_admin.xml (7) M src/modules/tls/tls_rpc.c (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4259.patch https://github.com/kamailio/kamailio/pull/4259.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4259 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4259(a)github.com>

1 5

git:master:0039c50e: permissions: don't allow reloads in the middle of ongoind reload
by Victor Seva 27 May '25

27 May '25

Module: kamailio Branch: master Commit: 0039c50e83916c647a2ee38e6be2c39ad17adfcc URL: https://github.com/kamailio/kamailio/commit/0039c50e83916c647a2ee38e6be2c39… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-05-27T14:06:20+02:00 permissions: don't allow reloads in the middle of ongoind reload --- Modified: src/modules/permissions/rpc.c --- Diff: https://github.com/kamailio/kamailio/commit/0039c50e83916c647a2ee38e6be2c39… Patch: https://github.com/kamailio/kamailio/commit/0039c50e83916c647a2ee38e6be2c39… --- diff --git a/src/modules/permissions/rpc.c b/src/modules/permissions/rpc.c index 6519c6343c0..7646ee0672e 100644 --- a/src/modules/permissions/rpc.c +++ b/src/modules/permissions/rpc.c @@ -45,7 +45,8 @@ int rpc_check_reload(rpc_t *rpc, void *ctx) rpc->fault(ctx, 500, "ongoing reload"); return -1; } - *perm_rpc_reload_time = time(NULL); + // we are reloading, don't allow new reloads + *perm_rpc_reload_time = time(NULL) + 86400; return 0; } @@ -61,10 +62,13 @@ void rpc_trusted_reload(rpc_t *rpc, void *c) if(reload_trusted_table_cmd() != 1) { rpc->fault(c, 500, "Reload failed."); - return; + goto done; } rpc->rpl_printf(c, "Reload OK"); +done: + // reloading is done + *perm_rpc_reload_time = time(NULL); return; } @@ -101,10 +105,13 @@ void rpc_address_reload(rpc_t *rpc, void *c) if(reload_address_table_cmd() != 1) { rpc->fault(c, 500, "Reload failed."); - return; + goto done; } rpc->rpl_printf(c, "Reload OK"); +done: + // reloading is done + *perm_rpc_reload_time = time(NULL); return; }

1 0

git:5.8:02edb040: sca: reserve subscription.rr.s separately since it could be updated
by Victor Seva 27 May '25

27 May '25

Module: kamailio Branch: 5.8 Commit: 02edb0408d907fa2a9031faf40bca899dbfcf050 URL: https://github.com/kamailio/kamailio/commit/02edb0408d907fa2a9031faf40bca89… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-05-27T11:43:53+02:00 sca: reserve subscription.rr.s separately since it could be updated (cherry picked from commit fd04bcc152dd6ef88fbb819b0b77aee7ca817878) (cherry picked from commit 14de4b1101f62c3463e5f05836e8618a3b0db96b) --- Modified: src/modules/sca/sca_subscribe.c --- Diff: https://github.com/kamailio/kamailio/commit/02edb0408d907fa2a9031faf40bca89… Patch: https://github.com/kamailio/kamailio/commit/02edb0408d907fa2a9031faf40bca89… --- diff --git a/src/modules/sca/sca_subscribe.c b/src/modules/sca/sca_subscribe.c index 9e212f01e2c..c460cccdb36 100644 --- a/src/modules/sca/sca_subscribe.c +++ b/src/modules/sca/sca_subscribe.c @@ -625,9 +625,6 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, len += sizeof(sca_subscription); len += sizeof(char) * (aor->len + subscriber->len); - if(!SCA_STR_EMPTY(rr)) { - len += sizeof(char) * rr->len; - } sub = (sca_subscription *)shm_malloc(len); if(sub == NULL) { @@ -659,18 +656,20 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, SCA_STR_COPY(&sub->target_aor, aor); len += aor->len; + // we shm_malloc this parts separately in case we need to update in-memory + // info for this subscriber. This is likely to happen if the + // subscriber goes off-line for some reason. if(!SCA_STR_EMPTY(rr)) { - sub->rr.s = (char *)sub + len; + sub->rr.s = (char *)shm_malloc(rr->len); + if(sub->rr.s == NULL) { + SHM_MEM_ERROR; + goto error; + } SCA_STR_COPY(&sub->rr, rr); - len += rr->len; } // dialog.id holds call-id + from-tag + to-tag; dialog.call_id, // dialog.from_tag, and dialog.to_tag point to offsets within // dialog.id. - // - // we shm_malloc this separately in case we need to update in-memory - // dialog saved for this subscriber. This is likely to happen if the - // subscriber goes off-line for some reason. len = sizeof(char) * (call_id->len + from_tag->len + to_tag->len); sub->dialog.id.s = (char *)shm_malloc(len); if(sub->dialog.id.s == NULL) { @@ -701,6 +700,9 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, error: if(sub != NULL) { + if(sub->rr.s != NULL) { + shm_free(sub->rr.s); + } if(sub->dialog.id.s != NULL) { shm_free(sub->dialog.id.s); } @@ -733,6 +735,10 @@ void sca_subscription_free(void *value) LM_DBG("Freeing %s subscription from %.*s\n", sca_event_name_from_type(sub->event), STR_FMT(&sub->subscriber)); + if(!SCA_STR_EMPTY(&sub->rr)) { + shm_free(sub->rr.s); + } + if(!SCA_STR_EMPTY(&sub->dialog.id)) { shm_free(sub->dialog.id.s); }

1 0

git:6.0:14de4b11: sca: reserve subscription.rr.s separately since it could be updated
by Victor Seva 27 May '25

27 May '25

Module: kamailio Branch: 6.0 Commit: 14de4b1101f62c3463e5f05836e8618a3b0db96b URL: https://github.com/kamailio/kamailio/commit/14de4b1101f62c3463e5f05836e8618… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-05-27T11:43:14+02:00 sca: reserve subscription.rr.s separately since it could be updated (cherry picked from commit fd04bcc152dd6ef88fbb819b0b77aee7ca817878) --- Modified: src/modules/sca/sca_subscribe.c --- Diff: https://github.com/kamailio/kamailio/commit/14de4b1101f62c3463e5f05836e8618… Patch: https://github.com/kamailio/kamailio/commit/14de4b1101f62c3463e5f05836e8618… --- diff --git a/src/modules/sca/sca_subscribe.c b/src/modules/sca/sca_subscribe.c index ca42fd138e9..5e34ceac6d2 100644 --- a/src/modules/sca/sca_subscribe.c +++ b/src/modules/sca/sca_subscribe.c @@ -626,9 +626,6 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, len += sizeof(sca_subscription); len += sizeof(char) * (aor->len + subscriber->len); - if(!SCA_STR_EMPTY(rr)) { - len += sizeof(char) * rr->len; - } sub = (sca_subscription *)shm_malloc(len); if(sub == NULL) { @@ -660,18 +657,20 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, SCA_STR_COPY(&sub->target_aor, aor); len += aor->len; + // we shm_malloc this parts separately in case we need to update in-memory + // info for this subscriber. This is likely to happen if the + // subscriber goes off-line for some reason. if(!SCA_STR_EMPTY(rr)) { - sub->rr.s = (char *)sub + len; + sub->rr.s = (char *)shm_malloc(rr->len); + if(sub->rr.s == NULL) { + SHM_MEM_ERROR; + goto error; + } SCA_STR_COPY(&sub->rr, rr); - len += rr->len; } // dialog.id holds call-id + from-tag + to-tag; dialog.call_id, // dialog.from_tag, and dialog.to_tag point to offsets within // dialog.id. - // - // we shm_malloc this separately in case we need to update in-memory - // dialog saved for this subscriber. This is likely to happen if the - // subscriber goes off-line for some reason. len = sizeof(char) * (call_id->len + from_tag->len + to_tag->len); sub->dialog.id.s = (char *)shm_malloc(len); if(sub->dialog.id.s == NULL) { @@ -702,6 +701,9 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, error: if(sub != NULL) { + if(sub->rr.s != NULL) { + shm_free(sub->rr.s); + } if(sub->dialog.id.s != NULL) { shm_free(sub->dialog.id.s); } @@ -734,6 +736,10 @@ void sca_subscription_free(void *value) LM_DBG("Freeing %s subscription from %.*s\n", sca_event_name_from_type(sub->event), STR_FMT(&sub->subscriber)); + if(!SCA_STR_EMPTY(&sub->rr)) { + shm_free(sub->rr.s); + } + if(!SCA_STR_EMPTY(&sub->dialog.id)) { shm_free(sub->dialog.id.s); }

1 0

[kamailio/kamailio] sca: reserve subscription.rr.s separately since it could be updated (PR #4251)
by Victor Seva 27 May '25

27 May '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 - [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:  - [x] PR should be backported to stable branches - [x] Tested changes locally #### Description sca was generating a crash since it is trying to shm_free a part of a bigger block at: https://github.com/kamailio/kamailio/blob/master/src/modules/sca/sca_subscr… So, let's reserve memory that field separately. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4251 -- Commit Summary -- * sca: reserve subscription.rr.s separately since it could be updated -- File Changes -- M src/modules/sca/sca_subscribe.c (24) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4251.patch https://github.com/kamailio/kamailio/pull/4251.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4251 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4251(a)github.com>

1 1

git:master:fd04bcc1: sca: reserve subscription.rr.s separately since it could be updated
by Victor Seva 27 May '25

27 May '25

Module: kamailio Branch: master Commit: fd04bcc152dd6ef88fbb819b0b77aee7ca817878 URL: https://github.com/kamailio/kamailio/commit/fd04bcc152dd6ef88fbb819b0b77aee… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-05-27T11:41:23+02:00 sca: reserve subscription.rr.s separately since it could be updated --- Modified: src/modules/sca/sca_subscribe.c --- Diff: https://github.com/kamailio/kamailio/commit/fd04bcc152dd6ef88fbb819b0b77aee… Patch: https://github.com/kamailio/kamailio/commit/fd04bcc152dd6ef88fbb819b0b77aee… --- diff --git a/src/modules/sca/sca_subscribe.c b/src/modules/sca/sca_subscribe.c index ca42fd138e9..5e34ceac6d2 100644 --- a/src/modules/sca/sca_subscribe.c +++ b/src/modules/sca/sca_subscribe.c @@ -626,9 +626,6 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, len += sizeof(sca_subscription); len += sizeof(char) * (aor->len + subscriber->len); - if(!SCA_STR_EMPTY(rr)) { - len += sizeof(char) * rr->len; - } sub = (sca_subscription *)shm_malloc(len); if(sub == NULL) { @@ -660,18 +657,20 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, SCA_STR_COPY(&sub->target_aor, aor); len += aor->len; + // we shm_malloc this parts separately in case we need to update in-memory + // info for this subscriber. This is likely to happen if the + // subscriber goes off-line for some reason. if(!SCA_STR_EMPTY(rr)) { - sub->rr.s = (char *)sub + len; + sub->rr.s = (char *)shm_malloc(rr->len); + if(sub->rr.s == NULL) { + SHM_MEM_ERROR; + goto error; + } SCA_STR_COPY(&sub->rr, rr); - len += rr->len; } // dialog.id holds call-id + from-tag + to-tag; dialog.call_id, // dialog.from_tag, and dialog.to_tag point to offsets within // dialog.id. - // - // we shm_malloc this separately in case we need to update in-memory - // dialog saved for this subscriber. This is likely to happen if the - // subscriber goes off-line for some reason. len = sizeof(char) * (call_id->len + from_tag->len + to_tag->len); sub->dialog.id.s = (char *)shm_malloc(len); if(sub->dialog.id.s == NULL) { @@ -702,6 +701,9 @@ sca_subscription *sca_subscription_create(str *aor, int event, str *subscriber, error: if(sub != NULL) { + if(sub->rr.s != NULL) { + shm_free(sub->rr.s); + } if(sub->dialog.id.s != NULL) { shm_free(sub->dialog.id.s); } @@ -734,6 +736,10 @@ void sca_subscription_free(void *value) LM_DBG("Freeing %s subscription from %.*s\n", sca_event_name_from_type(sub->event), STR_FMT(&sub->subscriber)); + if(!SCA_STR_EMPTY(&sub->rr)) { + shm_free(sub->rr.s); + } + if(!SCA_STR_EMPTY(&sub->dialog.id)) { shm_free(sub->dialog.id.s); }

1 0

[kamailio/kamailio] PDB: new RPC command to view/change PDB query timeout value (PR #4260)
by Muhammad Shahzad Shafi 27 May '25

27 May '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:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  This adds RPC command to view and/or change the timeout value for PDB queries. The documentation is also updated accordingly. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4260 -- Commit Summary -- * pdb: add new RPC command to view/change PDB query timeout * pdb: add documentation for new RPC command to view/change PDB query timeout * pdb: add new code contributor credits -- File Changes -- M src/modules/pdb/doc/pdb.xml (6) M src/modules/pdb/doc/pdb_admin.xml (18) M src/modules/pdb/pdb.c (34) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4260.patch https://github.com/kamailio/kamailio/pull/4260.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4260 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4260(a)github.com>

1 3

[kamailio/kamailio] PDB: add RPC command to get and set PDB query timeout (PR #4254)
by Muhammad Shahzad Shafi 27 May '25

27 May '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:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  This adds RPC command to view and/or change the timeout value for PDB queries. The documentation is also updated accordingly. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4254 -- Commit Summary -- * pdb: added rpc command to view/update the pdb query timeout * pdb: add documentation for rpc command to view/update the pdb query timeout -- File Changes -- M src/modules/pdb/doc/pdb.xml (55) M src/modules/pdb/doc/pdb_admin.xml (334) M src/modules/pdb/pdb.c (49) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4254.patch https://github.com/kamailio/kamailio/pull/4254.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4254 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4254(a)github.com>

3 8

[kamailio/kamailio] Adding kemi exports for play_dtmf event_callback and dlg_req_within functions (PR #4245)
by ggontariu 27 May '25

27 May '25

- [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) - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description Added missing kemi exports for RTP Engine module play_dtmf event_callback functions and Dialog module dlg_req_within function You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4245 -- Commit Summary -- * Add support for playing DTMF tones in RTP engine module using Kemi * Add support for KEMI callbacks in RTP engine events * Add `ki_dlg_req_within4` handling for dialog requests -- File Changes -- M src/modules/dialog/dialog.c (24) M src/modules/rtpengine/rtpengine.c (215) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4245.patch https://github.com/kamailio/kamailio/pull/4245.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4245 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4245(a)github.com>

2 17

git:master:472dbb59: dialog: Kemi export dlg_req_within
by Daniel-Constantin Mierla 27 May '25

27 May '25

Module: kamailio Branch: master Commit: 472dbb59006207629e9724b7d7155719cb653969 URL: https://github.com/kamailio/kamailio/commit/472dbb59006207629e9724b7d715571… Author: gabrielgontariu <114484712+gabrielgontariu(a)users.noreply.github.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-05-27T09:31:56+02:00 dialog: Kemi export dlg_req_within --- Modified: src/modules/dialog/dialog.c --- Diff: https://github.com/kamailio/kamailio/commit/472dbb59006207629e9724b7d715571… Patch: https://github.com/kamailio/kamailio/commit/472dbb59006207629e9724b7d715571… --- diff --git a/src/modules/dialog/dialog.c b/src/modules/dialog/dialog.c index 4539d1657e2..36af6f54404 100644 --- a/src/modules/dialog/dialog.c +++ b/src/modules/dialog/dialog.c @@ -2615,6 +2615,27 @@ static int ki_dlg_var_is_null(sip_msg_t *msg, str *name) return -1; } +/** + * + */ +static int ki_dlg_req_within4(struct sip_msg *msg, str *side, str *method, + str *content_type, str *content) +{ + int n = 0; + if(side && side->len > 0) { + if(side->len == 6 && strncasecmp(side->s, "caller", 6) == 0) { + n = 1; + } else if(side->len == 6 && strncasecmp(side->s, "callee", 6) == 0) { + n = 2; + } else { + return -1; + } + } + + return ki_dlg_req_with_headers_and_content( + msg, n, method, NULL, content_type, content); +} + /** * */ @@ -2780,6 +2801,11 @@ static sr_kemi_t sr_kemi_dialog_exports[] = { { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } }, + { str_init("dialog"), str_init("dlg_req_within4"), + SR_KEMIP_INT, ki_dlg_req_within4, + { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_STR, + SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, { {0, 0}, {0, 0}, 0, NULL, { 0, 0, 0, 0, 0, 0 } } };

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev