sr-dev October 2023

sr-dev@lists.kamailio.org

15 participants
291 discussions

Re: [kamailio/kamailio] usrloc: docs for ka_reply_codes parameter (c1bc5f7)
by Victor Seva 26 Oct '23

26 Oct '23

which responses are ?? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/c1bc5f7095659182401e6dd2b25bfaf… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/c1bc5f7095659182401e6dd2b25bfaf7f963466a/130981246(a)github.com>

1 0

git:master:55b3ce67: modules: readme files regenerated - usrloc ... [skip ci]
by Kamailio Dev 26 Oct '23

26 Oct '23

Module: kamailio Branch: master Commit: 55b3ce67709e56be116bb7077931ba7f4c79dff5 URL: https://github.com/kamailio/kamailio/commit/55b3ce67709e56be116bb7077931ba7… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2023-10-26T13:47:02+02:00 modules: readme files regenerated - usrloc ... [skip ci] --- Modified: src/modules/usrloc/README --- Diff: https://github.com/kamailio/kamailio/commit/55b3ce67709e56be116bb7077931ba7… Patch: https://github.com/kamailio/kamailio/commit/55b3ce67709e56be116bb7077931ba7… --- diff --git a/src/modules/usrloc/README b/src/modules/usrloc/README index 00b0743e9a0..fa0db1dcc66 100644 --- a/src/modules/usrloc/README +++ b/src/modules/usrloc/README @@ -97,8 +97,9 @@ Carsten Bock 3.55. ka_timeout (int) 3.56. ka_loglevel (int) 3.57. ka_logmsg (str) - 3.58. load_rank (int) - 3.59. db_clean_tcp (int) + 3.58. ka_reply_codes (str) + 3.59. load_rank (int) + 3.60. db_clean_tcp (int) 4. RPC Commands @@ -203,8 +204,9 @@ Carsten Bock 1.55. Set ka_timeout parameter 1.56. ka_loglevel parameter usage 1.57. ka_logmsg parameter usage - 1.58. load_rank parameter usage - 1.59. db_clean_tcp parameter usage + 1.58. ka_reply_codes parameter usage + 1.59. load_rank parameter usage + 1.60. db_clean_tcp parameter usage Chapter 1. Admin Guide @@ -278,8 +280,9 @@ Chapter 1. Admin Guide 3.55. ka_timeout (int) 3.56. ka_loglevel (int) 3.57. ka_logmsg (str) - 3.58. load_rank (int) - 3.59. db_clean_tcp (int) + 3.58. ka_reply_codes (str) + 3.59. load_rank (int) + 3.60. db_clean_tcp (int) 4. RPC Commands @@ -416,8 +419,9 @@ Chapter 1. Admin Guide 3.55. ka_timeout (int) 3.56. ka_loglevel (int) 3.57. ka_logmsg (str) - 3.58. load_rank (int) - 3.59. db_clean_tcp (int) + 3.58. ka_reply_codes (str) + 3.59. load_rank (int) + 3.60. db_clean_tcp (int) 3.1. nat_bflag (int) @@ -1228,19 +1232,35 @@ modparam("usrloc", "ka_loglevel", 1) modparam("usrloc", "ka_logmsg", " to-uri: [$tu] remote-addr: [$sas]") ... -3.58. load_rank (int) +3.58. ka_reply_codes (str) + + Comma separated list of reply code values or reply code classes to be + considered valid for keepalive processing. If the reply code does not + match any of them, then the reply is ignored, not considered a valid + response to keepalive request. It could be useful to skip replies from + intermediary proxies that could not forward the request. + + Default value is ��0�� (checking of reply code is not done, are responses + are considered valid). + + Example 1.58. ka_reply_codes parameter usage +... +modparam("usrloc", "ka_reply_codes", "2,405,486") +... + +3.59. load_rank (int) Allows to set the rank of the child SIP worker to load the location records. Default value is ��1�� (PROC_SIPINIT). - Example 1.58. load_rank parameter usage + Example 1.59. load_rank parameter usage ... modparam("usrloc", "load_rank", 1) ... -3.59. db_clean_tcp (int) +3.60. db_clean_tcp (int) If set to 1, when Kamailio starts it removes the contacts with transport TCP, TLS or WSS, no longer loading them. Useful when end @@ -1251,7 +1271,7 @@ modparam("usrloc", "load_rank", 1) Default value is ��0�� (do not clean tcp contacts). - Example 1.59. db_clean_tcp parameter usage + Example 1.60. db_clean_tcp parameter usage ... modparam("usrloc", "db_clean_tcp", 1) ...

1 0

git:master:c1bc5f70: usrloc: docs for ka_reply_codes parameter
by Daniel-Constantin Mierla 26 Oct '23

26 Oct '23

Module: kamailio Branch: master Commit: c1bc5f7095659182401e6dd2b25bfaf7f963466a URL: https://github.com/kamailio/kamailio/commit/c1bc5f7095659182401e6dd2b25bfaf… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2023-10-26T13:41:12+02:00 usrloc: docs for ka_reply_codes parameter --- Modified: src/modules/usrloc/doc/usrloc_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/c1bc5f7095659182401e6dd2b25bfaf… Patch: https://github.com/kamailio/kamailio/commit/c1bc5f7095659182401e6dd2b25bfaf… --- diff --git a/src/modules/usrloc/doc/usrloc_admin.xml b/src/modules/usrloc/doc/usrloc_admin.xml index 61491648a0c..5f244bc80e5 100644 --- a/src/modules/usrloc/doc/usrloc_admin.xml +++ b/src/modules/usrloc/doc/usrloc_admin.xml @@ -1504,6 +1504,29 @@ modparam("usrloc", "ka_logmsg", " to-uri: [$tu] remote-addr: [$sas]") </example> </section> + <section id="usrloc.p.ka_reply_codes"> + <title><varname>ka_reply_codes</varname> (str)</title> + <para> + Comma separated list of reply code values or reply code classes to be + considered valid for keepalive processing. If the reply code does not + match any of them, then the reply is ignored, not considered a valid + response to keepalive request. It could be useful to skip replies + from intermediary proxies that could not forward the request. + </para> + <para> + Default value is <quote>0</quote> (checking of reply code is not done, + are responses are considered valid). + </para> + <example> + <title><varname>ka_reply_codes</varname> parameter usage</title> + <programlisting format="linespecific"> +... +modparam("usrloc", "ka_reply_codes", "2,405,486") +... + </programlisting> + </example> + </section> + <section id="usrloc.p.load_rank"> <title><varname>load_rank</varname> (int)</title> <para>

1 0

git:master:8797e745: usrloc: new parameter to specify reply codes for keepalive handling
by Daniel-Constantin Mierla 26 Oct '23

26 Oct '23

Module: kamailio Branch: master Commit: 8797e7454f5948c0c8166bf2b938a489b3092f2d URL: https://github.com/kamailio/kamailio/commit/8797e7454f5948c0c8166bf2b938a48… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2023-10-26T13:34:51+02:00 usrloc: new parameter to specify reply codes for keepalive handling - comma separated list of reply codes or reply class to be considered valid for keepalive handling --- Modified: src/modules/usrloc/ul_keepalive.c Modified: src/modules/usrloc/ul_keepalive.h Modified: src/modules/usrloc/usrloc_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/8797e7454f5948c0c8166bf2b938a48… Patch: https://github.com/kamailio/kamailio/commit/8797e7454f5948c0c8166bf2b938a48… --- diff --git a/src/modules/usrloc/ul_keepalive.c b/src/modules/usrloc/ul_keepalive.c index 9897233ea53..02c21c0837e 100644 --- a/src/modules/usrloc/ul_keepalive.c +++ b/src/modules/usrloc/ul_keepalive.c @@ -315,6 +315,47 @@ unsigned long ul_ka_fromhex(str *shex, int *err) return v; } +/** + * + */ +#define UL_KA_REPLY_CODES_SIZE 32 +static int _ul_ka_reply_codes[UL_KA_REPLY_CODES_SIZE] = {0}; + +/** + * + */ +int ul_ka_parse_reply_codes(char *vcodes) +{ + int nb = 0; + char *p; + char *e; + + _ul_ka_reply_codes[0] = 0; + if(vcodes == NULL || strlen(vcodes) == 0) { + LM_ERR("invalid parameter\n"); + return -1; + } + p = vcodes; + while(nb < UL_KA_REPLY_CODES_SIZE && *p) { + _ul_ka_reply_codes[nb] = strtol(p, &e, 10); + if(_ul_ka_reply_codes[nb] > 0) { + nb++; + } else { + _ul_ka_reply_codes[nb] = 0; + } + while(*e == ',' || *e == ' ') { + e++; + } + p = e; + } + if(nb == UL_KA_REPLY_CODES_SIZE) { + LM_ERR("exceeded maximum number of reply code rules\n"); + return -1; + } + _ul_ka_reply_codes[nb] = 0; + return 0; +} + /** * */ @@ -329,6 +370,7 @@ int ul_ka_reply_received(sip_msg_t *msg) struct timeval tvm; struct timeval tvn; unsigned int tvdiff; + int i; if(msg->cseq == NULL) { if((parse_headers(msg, HDR_CSEQ_F, 0) == -1) || (msg->cseq == NULL)) { @@ -345,6 +387,19 @@ int ul_ka_reply_received(sip_msg_t *msg) return 1; } + for(i = 0; _ul_ka_reply_codes[i] != 0; i++) { + if(_ul_ka_reply_codes[i] == msg->first_line.u.reply.statuscode + || _ul_ka_reply_codes[i] + == (msg->first_line.u.reply.statuscode / 100)) { + break; + } + } + + if(i > 0 && _ul_ka_reply_codes[i] == 0) { + /* no match of status code */ + return 1; + } + /* there must be no second via */ if(!(parse_headers(msg, HDR_VIA2_F, 0) == -1 || (msg->via2 == 0) || (msg->via2->error != PARSE_OK))) { diff --git a/src/modules/usrloc/ul_keepalive.h b/src/modules/usrloc/ul_keepalive.h index b807d392c57..8e5e684b37a 100644 --- a/src/modules/usrloc/ul_keepalive.h +++ b/src/modules/usrloc/ul_keepalive.h @@ -35,5 +35,6 @@ int ul_ka_urecord(urecord_t *ur); int ul_ka_reply_received(sip_msg_t *msg); +int ul_ka_parse_reply_codes(char *vcodes); -#endif \ No newline at end of file +#endif diff --git a/src/modules/usrloc/usrloc_mod.c b/src/modules/usrloc/usrloc_mod.c index de75545bc78..37f4a903b86 100644 --- a/src/modules/usrloc/usrloc_mod.c +++ b/src/modules/usrloc/usrloc_mod.c @@ -220,6 +220,8 @@ int ul_hash_size = 10; int ul_db_insert_null = 0; int ul_db_timer_clean = 0; +char *ul_ka_reply_codes_str = "0"; + /* flags */ unsigned int ul_nat_bflag = (unsigned int)-1; unsigned int ul_init_flag = 0; @@ -301,6 +303,7 @@ static param_export_t params[] = { {"ka_timeout", PARAM_INT, &ul_keepalive_timeout}, {"ka_loglevel", PARAM_INT, &ul_ka_loglevel}, {"ka_logmsg", PARAM_STR, &ul_ka_logmsg}, + {"ka_reply_codes", PARAM_STRING, &ul_ka_reply_codes_str}, {"load_rank", PARAM_INT, &ul_load_rank}, {"db_clean_tcp", PARAM_INT, &ul_db_clean_tcp}, {0, 0, 0} @@ -335,6 +338,9 @@ static int mod_init(void) int i; udomain_t *d; + if(ul_ka_parse_reply_codes(ul_ka_reply_codes_str)) { + return -1; + } if(ul_rm_expired_delay != 0) { if(ul_db_mode != DB_ONLY) { LM_ERR("rm expired delay feature is available for db only mode\n");

1 0

[kamailio/kamailio] kamailio depends on obsolete pcre3 library (Issue #2946)
by Victor Seva 26 Oct '23

26 Oct '23

>From https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1000134: ``` Source: kamailio Severity: important User: matthew-pcredep(a)debian.org Usertags: obsolete-pcre3 Dear maintainer, Your package still depends on the old, obsolete PCRE3[0] libraries (i.e. libpcre3-dev). This has been end of life for a while now, and upstream do not intend to fix any further bugs in it. Accordingly, I would like to remove the pcre3 libraries from Debian, preferably in time for the release of Bookworm. The newer PCRE2 library was first released in 2015, and has been in Debian since stretch. Upstream's documentation for PCRE2 is available here: https://pcre.org/current/doc/html/ Many large projects that use PCRE have made the switch now (e.g. git, php); it does involve some work, but we are now at the stage where PCRE3 should not be used, particularly if it might ever be exposed to untrusted input. This mass bug filing was discussed on debian-devel@ in https://lists.debian.org/debian-devel/2021/11/msg00176.html Regards, Matthew [0] Historical reasons mean that old PCRE is packaged as pcre3 in Debian ``` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/2946

2 4

git:5.4:7d022d7d: pkg/kamailio/deb: refresh files [skip ci]
by Victor Seva 26 Oct '23

26 Oct '23

Module: kamailio Branch: 5.4 Commit: 7d022d7da5caab942de84c834e000b87ada6dce5 URL: https://github.com/kamailio/kamailio/commit/7d022d7da5caab942de84c834e000b8… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-10-26T10:39:47+02:00 pkg/kamailio/deb: refresh files [skip ci] related to #3617 --- Modified: pkg/kamailio/deb/sid/rules --- Diff: https://github.com/kamailio/kamailio/commit/7d022d7da5caab942de84c834e000b8… Patch: https://github.com/kamailio/kamailio/commit/7d022d7da5caab942de84c834e000b8… --- diff --git a/pkg/kamailio/deb/sid/rules b/pkg/kamailio/deb/sid/rules index 0460677fba3..ea773469ca9 100755 --- a/pkg/kamailio/deb/sid/rules +++ b/pkg/kamailio/deb/sid/rules @@ -34,7 +34,7 @@ EXTRA_EXCLUDED_MODULES=bdb dbtext oracle pa iptrtpproxy mi_xmlrpc dnssec # on which other depend first) PACKAGE_GROUPS=mysql postgres berkeley unixodbc radius presence \ ldap xml perl utils lua memcached \ - snmpstats carrierroute xmpp cpl redisgeoip geoip2 \ + snmpstats carrierroute xmpp cpl redis geoip geoip2 \ sqlite json mono ruby ims sctp \ tls outbound websocket autheph kazoo cnxcc \ erlang systemd phonenum mongodb rabbitmq python3 mqtt

1 0

git:5.4:09ca2a99: pkg/kamailio/deb: fix backports scripts [skip ci]
by Victor Seva 26 Oct '23

26 Oct '23

Module: kamailio Branch: 5.4 Commit: 09ca2a9988c523c32c558a3c74a486b122f48266 URL: https://github.com/kamailio/kamailio/commit/09ca2a9988c523c32c558a3c74a486b… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-10-26T10:39:33+02:00 pkg/kamailio/deb: fix backports scripts [skip ci] missing space was causing some modules not to be build fixes #3617 (cherry picked from commit 9326e98d2677ada18f90e28ef37f496ab4d2e985) (cherry picked from commit 44eed1c1ad533ce60741aeedf97d328f1fc0757f) --- Modified: pkg/kamailio/deb/debian/backports/sid --- Diff: https://github.com/kamailio/kamailio/commit/09ca2a9988c523c32c558a3c74a486b… Patch: https://github.com/kamailio/kamailio/commit/09ca2a9988c523c32c558a3c74a486b… --- diff --git a/pkg/kamailio/deb/debian/backports/sid b/pkg/kamailio/deb/debian/backports/sid index deb134222af..da837bb9301 100755 --- a/pkg/kamailio/deb/debian/backports/sid +++ b/pkg/kamailio/deb/debian/backports/sid @@ -32,7 +32,7 @@ sed -i -e '/dh-systemd/d' ${DIST}/control sed -i -e '/^ python,/d' -e '/python-dev/d' \ -e '/^Package: kamailio-python-modules/,/^$/d' \ ${DIST}/control -sed -i -e 's/ python [ ,$]*//' ${DIST}/rules +sed -i -e 's/ python [ ,$]*/ /' ${DIST}/rules sed -i -e '/^EXCLUDED_MODULES=/s/$/ python/' ${DIST}/rules # no lsb-base

1 0

git:5.5:1f576f75: pkg/kamailio/deb: refresh files [skip ci]
by Victor Seva 26 Oct '23

26 Oct '23

Module: kamailio Branch: 5.5 Commit: 1f576f7538934bf91d43c1e27d9c66251ffc4c4b URL: https://github.com/kamailio/kamailio/commit/1f576f7538934bf91d43c1e27d9c662… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-10-26T10:37:39+02:00 pkg/kamailio/deb: refresh files [skip ci] related to #3617 --- Modified: pkg/kamailio/deb/sid/rules --- Diff: https://github.com/kamailio/kamailio/commit/1f576f7538934bf91d43c1e27d9c662… Patch: https://github.com/kamailio/kamailio/commit/1f576f7538934bf91d43c1e27d9c662… --- diff --git a/pkg/kamailio/deb/sid/rules b/pkg/kamailio/deb/sid/rules index b9dde8844d8..fdeba60fcbb 100755 --- a/pkg/kamailio/deb/sid/rules +++ b/pkg/kamailio/deb/sid/rules @@ -34,7 +34,7 @@ EXTRA_EXCLUDED_MODULES=bdb dbtext oracle pa iptrtpproxy mi_xmlrpc dnssec # on which other depend first) PACKAGE_GROUPS=mysql postgres berkeley unixodbc radius presence \ ldap xml perl utils lua memcached \ - snmpstats carrierroute xmpp cpl redisgeoip geoip2 \ + snmpstats carrierroute xmpp cpl redis geoip geoip2 \ sqlite json mono ruby ims sctp \ tls outbound websocket autheph kazoo cnxcc \ erlang systemd phonenum mongodb rabbitmq python3 mqtt secsipid lwsc

1 0

git:5.5:44eed1c1: pkg/kamailio/deb: fix backports scripts [skip ci]
by Victor Seva 26 Oct '23

26 Oct '23

Module: kamailio Branch: 5.5 Commit: 44eed1c1ad533ce60741aeedf97d328f1fc0757f URL: https://github.com/kamailio/kamailio/commit/44eed1c1ad533ce60741aeedf97d328… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-10-26T10:36:51+02:00 pkg/kamailio/deb: fix backports scripts [skip ci] missing space was causing some modules not to be build fixes #3617 (cherry picked from commit 9326e98d2677ada18f90e28ef37f496ab4d2e985) --- Modified: pkg/kamailio/deb/debian/backports/sid --- Diff: https://github.com/kamailio/kamailio/commit/44eed1c1ad533ce60741aeedf97d328… Patch: https://github.com/kamailio/kamailio/commit/44eed1c1ad533ce60741aeedf97d328… --- diff --git a/pkg/kamailio/deb/debian/backports/sid b/pkg/kamailio/deb/debian/backports/sid index deb134222af..da837bb9301 100755 --- a/pkg/kamailio/deb/debian/backports/sid +++ b/pkg/kamailio/deb/debian/backports/sid @@ -32,7 +32,7 @@ sed -i -e '/dh-systemd/d' ${DIST}/control sed -i -e '/^ python,/d' -e '/python-dev/d' \ -e '/^Package: kamailio-python-modules/,/^$/d' \ ${DIST}/control -sed -i -e 's/ python [ ,$]*//' ${DIST}/rules +sed -i -e 's/ python [ ,$]*/ /' ${DIST}/rules sed -i -e '/^EXCLUDED_MODULES=/s/$/ python/' ${DIST}/rules # no lsb-base

1 0

[kamailio/kamailio] Debian 12 packaging: missing files for kamailio-redis-modules (Issue #3617)
by bonan 26 Oct '23

26 Oct '23

### Description I installed kamailio-redis-modules for kamailio 5.6.4 on debian 12 (`kamailio-redis-modules_5.6.4+bpo12_amd64.deb`) The package doesn't contain the modules, only changelog and copyright The issue also seems to affect `kamailio-redis-modules_5.6.4+ubuntu22.04_amd64.deb` #### Reproduction ```sh wget "https://deb.kamailio.org/kamailio56/pool/main/k/kamailio/kamailio-redis-mod…" dpkg-deb -c "kamailio-redis-modules_5.6.4+bpo12_amd64.deb" ``` Result: ``` drwxr-xr-x root/root 0 2023-02-27 12:48 ./ drwxr-xr-x root/root 0 2023-02-27 12:48 ./usr/ drwxr-xr-x root/root 0 2023-02-27 12:48 ./usr/share/ drwxr-xr-x root/root 0 2023-02-27 12:48 ./usr/share/doc/ drwxr-xr-x root/root 0 2023-02-27 12:48 ./usr/share/doc/kamailio-redis-modules/ -rw-r--r-- root/root 487 2023-02-27 12:48 ./usr/share/doc/kamailio-redis-modules/changelog.Debian.gz -rw-r--r-- root/root 121021 2023-02-27 12:48 ./usr/share/doc/kamailio-redis-modules/changelog.gz -rw-r--r-- root/root 15067 2023-02-27 12:48 ./usr/share/doc/kamailio-redis-modules/copyright ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.6.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, 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 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 [hostname] 6.1.0-13-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.55-1 (2023-09-29) x86_64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3617 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3617(a)github.com>

2 2

← Newer
1
...
14
15
16
17
18
19
20
...
30
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev October 2023