sr-dev January 2025

sr-dev@lists.kamailio.org

21 participants
301 discussions

[kamailio/kamailio] kamcmd tls.reload problem (Issue #4033)
by ynasida 21 Jan '25

21 Jan '25

Hello, I am using letsencrypt cert and key and do not want to restart kamailio every 3 months to load new ones. I know that there is: kamcmd tls.reload method but it has an error for me. error: 500 - Error while fixing TLS configuration (consult server log) I am checking the logs and see: kamailio[3865480]: INFO: tls [tls_domain.c:345]: ksr_tls_fill_missing(): TLSs<default>: tls_method=3 kamailio[3865480]: INFO: tls [tls_domain.c:357]: ksr_tls_fill_missing(): TLSs<default>: certificate='/etc/kamailio/certs/my_cert.crt' kamailio[3865480]: INFO: tls [tls_domain.c:364]: ksr_tls_fill_missing(): TLSs<default>: ca_list='(null)' kamailio[3865480]: INFO: tls [tls_domain.c:371]: ksr_tls_fill_missing(): TLSs<default>: ca_path='(null)' kamailio[3865480]: INFO: tls [tls_domain.c:378]: ksr_tls_fill_missing(): TLSs<default>: crl='(null)' kamailio[3865480]: INFO: tls [tls_domain.c:382]: ksr_tls_fill_missing(): TLSs<default>: require_certificate=0 kamailio[3865480]: INFO: tls [tls_domain.c:390]: ksr_tls_fill_missing(): TLSs<default>: cipher_list='(null)' kamailio[3865480]: INFO: tls [tls_domain.c:397]: ksr_tls_fill_missing(): TLSs<default>: private_key='/etc/kamailio/certs/private.key' kamailio[3865480]: INFO: tls [tls_domain.c:401]: ksr_tls_fill_missing(): TLSs<default>: verify_certificate=0 kamailio[3865480]: INFO: tls [tls_domain.c:406]: ksr_tls_fill_missing(): TLSs<default>: verify_depth=9 kamailio[3865480]: INFO: tls [tls_domain.c:410]: ksr_tls_fill_missing(): TLSs<default>: verify_client=0 kamailio[3865480]: NOTICE: tls [tls_domain.c:1168]: ksr_tls_fix_domain(): registered server_name callback handler for socket [:0], server_name='<default>' ... kamailio[3865480]: ERROR: tls [tls_domain.c:590]: load_cert(): TLSs<default>: Unable to load certificate file '/etc/kamailio/certs/my_cert.crt' kamailio[3865480]: ERROR: tls [tls_util.h:49]: tls_err_ret(): load_cert:error:03000072:digital envelope routines::decode error (sni: unknown) kamailio[3865480]: ERROR: tls [tls_util.h:49]: tls_err_ret(): load_cert:error:0A00018F:SSL routines::ee key too small (sni: unknown) Any advice ? It's interesting that there are not any TLS errors in case I restart kamailio. I can make TLS calls without problems. deb 12.5 version: kamailio 5.7.4 (x86_64/linux) -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4033 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4033(a)github.com>

4 5

[kamailio/kamailio] Close parenthesis on error description (#2589)
by Javier Valencia 20 Jan '25

20 Jan '25

Close parentheses on error description when *Not enough parameters* on *htable_rpc_delete*. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2589 -- Commit Summary -- * Close parenthesis on error description -- File Changes -- M src/modules/htable/htable.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2589.patch https://github.com/kamailio/kamailio/pull/2589.diff -- 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/pull/2589

3 4

git:master:024c649c: htable: close parentheses on error description
by Henning Westerholt 20 Jan '25

20 Jan '25

Module: kamailio Branch: master Commit: 024c649c36bbe1d5ba4b306de53102cec4096734 URL: https://github.com/kamailio/kamailio/commit/024c649c36bbe1d5ba4b306de53102c… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-01-20T19:36:11Z htable: close parentheses on error description --- Modified: src/modules/htable/htable.c --- Diff: https://github.com/kamailio/kamailio/commit/024c649c36bbe1d5ba4b306de53102c… Patch: https://github.com/kamailio/kamailio/commit/024c649c36bbe1d5ba4b306de53102c… --- diff --git a/src/modules/htable/htable.c b/src/modules/htable/htable.c index bd0a7bf6e57..d458e83141a 100644 --- a/src/modules/htable/htable.c +++ b/src/modules/htable/htable.c @@ -1561,7 +1561,7 @@ static void htable_rpc_delete(rpc_t *rpc, void *c) int res; if(rpc->scan(c, "SS", &htname, &keyname) < 2) { - rpc->fault(c, 500, "Not enough parameters (htable name & key name"); + rpc->fault(c, 500, "Not enough parameters (htable name & key name)"); return; } ht = ht_get_table(&htname);

1 0

git:master:e8389b36: cdp: remove level and function name from log messages
by Victor Seva 20 Jan '25

20 Jan '25

Module: kamailio Branch: master Commit: e8389b36aff772d1a419a5b9d28ede0e86b2109e URL: https://github.com/kamailio/kamailio/commit/e8389b36aff772d1a419a5b9d28ede0… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-01-20T18:37:51+01:00 cdp: remove level and function name from log messages --- Modified: src/modules/cdp/authstatemachine.c Modified: src/modules/cdp/common.c Modified: src/modules/cdp/config.c Modified: src/modules/cdp/configparser.c Modified: src/modules/cdp/diameter_avp.c Modified: src/modules/cdp/diameter_comm.c Modified: src/modules/cdp/diameter_msg.c Modified: src/modules/cdp/diameter_peer.c Modified: src/modules/cdp/peermanager.c Modified: src/modules/cdp/peerstatemachine.c Modified: src/modules/cdp/receiver.c Modified: src/modules/cdp/routing.c Modified: src/modules/cdp/session.c Modified: src/modules/cdp/tcp_accept.c Modified: src/modules/cdp/worker.c --- Diff: https://github.com/kamailio/kamailio/commit/e8389b36aff772d1a419a5b9d28ede0… Patch: https://github.com/kamailio/kamailio/commit/e8389b36aff772d1a419a5b9d28ede0…

1 0

git:master:b96bc11a: core: parser/sdp - check if still before end of the value
by Daniel-Constantin Mierla 20 Jan '25

20 Jan '25

Module: kamailio Branch: master Commit: b96bc11a54f11ec688429c057bf5a90c40c147b3 URL: https://github.com/kamailio/kamailio/commit/b96bc11a54f11ec688429c057bf5a90… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-01-20T17:10:05+01:00 core: parser/sdp - check if still before end of the value --- Modified: src/core/parser/sdp/sdp_helpr_funcs.c --- Diff: https://github.com/kamailio/kamailio/commit/b96bc11a54f11ec688429c057bf5a90… Patch: https://github.com/kamailio/kamailio/commit/b96bc11a54f11ec688429c057bf5a90… --- diff --git a/src/core/parser/sdp/sdp_helpr_funcs.c b/src/core/parser/sdp/sdp_helpr_funcs.c index 5c7076823c4..483cfdec70e 100644 --- a/src/core/parser/sdp/sdp_helpr_funcs.c +++ b/src/core/parser/sdp/sdp_helpr_funcs.c @@ -413,7 +413,7 @@ int extract_ice_option(str *body, sdp_stream_cell_t *stream) trim_len(ice_opt->option.len, ice_opt->option.s, ice_opt->option); length = 0; - if(*ptr_src == 32) + if(ptr_src < end && *ptr_src == 32) ptr_src++; /* skip space */ }

1 0

git:master:6750e2e2: core: socket info - set receive timeout for netlink socket
by Daniel-Constantin Mierla 20 Jan '25

20 Jan '25

Module: kamailio Branch: master Commit: 6750e2e2f1d6c10f6b5e66897d38657a576a9732 URL: https://github.com/kamailio/kamailio/commit/6750e2e2f1d6c10f6b5e66897d38657… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-01-20T12:58:18+01:00 core: socket info - set receive timeout for netlink socket - GH #4098 --- Modified: src/core/socket_info.c --- Diff: https://github.com/kamailio/kamailio/commit/6750e2e2f1d6c10f6b5e66897d38657… Patch: https://github.com/kamailio/kamailio/commit/6750e2e2f1d6c10f6b5e66897d38657… --- diff --git a/src/core/socket_info.c b/src/core/socket_info.c index b431ea395db..24243fea153 100644 --- a/src/core/socket_info.c +++ b/src/core/socket_info.c @@ -1378,6 +1378,7 @@ static int nl_bound_sock(void) { int sock = -1; struct sockaddr_nl la; + struct timeval recvtimeout; sock = socket(AF_NETLINK, SOCK_RAW, NETLINK_ROUTE); if(sock < 0) { @@ -1396,6 +1397,15 @@ static int nl_bound_sock(void) goto error; } + recvtimeout.tv_sec = 4; + recvtimeout.tv_usec = 0; + if(setsockopt( + sock, SOL_SOCKET, SO_RCVTIMEO, &recvtimeout, sizeof(recvtimeout)) + < 0) { + LM_ERR("failed to set receive timeout\n"); + goto error; + } + return sock; error: if(sock >= 0) @@ -1448,6 +1458,10 @@ static int get_flags(int family) goto error; } rtn = recv(nl_sock, p, sizeof(buf) - nll, 0); + if(rtn <= 0) { + LM_ERR("failed to receive data (%d/%d)\n", rtn, errno); + goto error; + } nlp = (struct nlmsghdr *)p; if(nlp->nlmsg_type == NLMSG_DONE) { LM_DBG("done\n"); @@ -1550,6 +1564,10 @@ static int build_iface_list(void) goto error; } rtn = recv(nl_sock, p, sizeof(buf) - nll, 0); + if(rtn <= 0) { + LM_ERR("failed to receive data (%d/%d)\n", rtn, errno); + goto error; + } LM_DBG("received %d byles \n", rtn); nlp = (struct nlmsghdr *)p; if(nlp->nlmsg_type == NLMSG_DONE) {

1 0

[kamailio/kamailio] cmake - make clean deletes readme files from modules source folders (Issue #4084)
by Daniel-Constantin Mierla 20 Jan '25

20 Jan '25

Executing `make clean` in the `build` folder deletes the `README` files from the modules folders. This is probably because the README is an output of a `cmake-make` command. What would be the best option to skip this removing? I found: - https://stackoverflow.com/questions/6259372/cmake-preventing-make-clean-fro… Which indicates adding: ``` SET_DIRECTORY_PROPERTIES(PROPERTIES CLEAN_NO_CUSTOM 1) ``` In the CMakeLists.txt inside the module folders. But maybe it is another place where it can be done once for all modules, or another option... -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4084 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4084(a)github.com>

3 9

git:master:e50a24aa: cmake/os-specific/darwin.cmake: use -undefined dynamic_lookup for OS v24+
by Daniel-Constantin Mierla 20 Jan '25

20 Jan '25

Module: kamailio Branch: master Commit: e50a24aa4a4fcc6c0310f9f2d5f130197372b2f9 URL: https://github.com/kamailio/kamailio/commit/e50a24aa4a4fcc6c0310f9f2d5f1301… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-01-20T11:05:08+01:00 cmake/os-specific/darwin.cmake: use -undefined dynamic_lookup for OS v24+ - the option -undefined suppress has been deprecated --- Modified: cmake/os-specific/darwin.cmake --- Diff: https://github.com/kamailio/kamailio/commit/e50a24aa4a4fcc6c0310f9f2d5f1301… Patch: https://github.com/kamailio/kamailio/commit/e50a24aa4a4fcc6c0310f9f2d5f1301… --- diff --git a/cmake/os-specific/darwin.cmake b/cmake/os-specific/darwin.cmake index bd02a290b53..a43c075142b 100644 --- a/cmake/os-specific/darwin.cmake +++ b/cmake/os-specific/darwin.cmake @@ -25,11 +25,20 @@ if(NOT ${USE_FAST_LOCK}) target_compile_definitions(common INTERFACE USE_PTHREAD_MUTEX USE_SYSV_SEM) endif() +if (CMAKE_HOST_SYSTEM_VERSION VERSION_LESS 24) + message(STATUS "Running on Big Sur") set(CMAKE_MODULE_LINKER_FLAGS "${CMAKE_MODULE_LINKER_FLAGS} -bundle -flat_namespace -undefined suppress") set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -dynamiclib -flat_namespace -undefined suppress" ) +else () +set(CMAKE_MODULE_LINKER_FLAGS + "${CMAKE_MODULE_LINKER_FLAGS} -bundle -flat_namespace -undefined dynamic_lookup") +set(CMAKE_SHARED_LINKER_FLAGS + "${CMAKE_SHARED_LINKER_FLAGS} -dynamiclib -flat_namespace -undefined dynamic_lookup" +) +endif () if(NOT NO_SELECT) target_compile_definitions(common INTERFACE HAVE_SELECT)

1 0

git:master:a1bdef91: cmake: messages about not finding docs-related tools made status
by Daniel-Constantin Mierla 20 Jan '25

20 Jan '25

Module: kamailio Branch: master Commit: a1bdef919414173e8327a8dd65ecaa2fca279d6a URL: https://github.com/kamailio/kamailio/commit/a1bdef919414173e8327a8dd65ecaa2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-01-20T10:54:14+01:00 cmake: messages about not finding docs-related tools made status - they should not be needed for development and usual installation, warning makes it look like something relevant cannot be done --- Modified: cmake/modules-docs.cmake --- Diff: https://github.com/kamailio/kamailio/commit/a1bdef919414173e8327a8dd65ecaa2… Patch: https://github.com/kamailio/kamailio/commit/a1bdef919414173e8327a8dd65ecaa2… --- diff --git a/cmake/modules-docs.cmake b/cmake/modules-docs.cmake index 10b1b70a8f2..721ba7d0b45 100644 --- a/cmake/modules-docs.cmake +++ b/cmake/modules-docs.cmake @@ -6,12 +6,12 @@ find_program(LYNX_EXECUTABLE lynx QUIET) find_program(DOCBOOK2X_EXECUTABLE docbook2x-man QUIET) if(BUILD_DOC AND (NOT XSLTPROC_EXECUTABLE OR NOT LYNX_EXECUTABLE)) - message(WARNING "xsltproc or lynx not found but required for doc generation.") + message(STATUS "xsltproc or lynx not found but required for doc generation.") set(BUILD_DOC OFF) endif() if(BUILD_DOC AND (NOT DOCBOOK2X_EXECUTABLE)) - message(WARNING "docbook2x-man not found but required for man generation.") + message(STATUS "docbook2x-man not found but required for man generation.") set(BUILD_DOC OFF) endif()

1 0

git:master:09628a8e: cmake: fix typo, missing KRADIUS
by Victor Seva 20 Jan '25

20 Jan '25

Module: kamailio Branch: master Commit: 09628a8e6060a21805fccccc1f530062fca1bb75 URL: https://github.com/kamailio/kamailio/commit/09628a8e6060a21805fccccc1f53006… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-01-20T09:18:45+01:00 cmake: fix typo, missing KRADIUS --- Modified: .pre-commit-config.yaml Modified: cmake/groups.cmake --- Diff: https://github.com/kamailio/kamailio/commit/09628a8e6060a21805fccccc1f53006… Patch: https://github.com/kamailio/kamailio/commit/09628a8e6060a21805fccccc1f53006… --- diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 7892175df39..c5df7975c1f 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -16,4 +16,10 @@ repos: rev: v0.6.13 hooks: - id: cmake-format + verbose: true + args: + - -c cmake/cmake-format.py - id: cmake-lint + verbose: true + args: + - -c cmake/cmake-format.py diff --git a/cmake/groups.cmake b/cmake/groups.cmake index b3562cbbbcc..9cd7ddf77dc 100644 --- a/cmake/groups.cmake +++ b/cmake/groups.cmake @@ -720,7 +720,7 @@ list( KPOSTGRES KCPL KXML - KRAIDUS + KRADIUS KUNIXODBC KPERL KSNMPSTATS

1 0

← Newer
1
...
19
20
21
22
23
24
25
...
31
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2025