sr-dev January 2025

sr-dev@lists.kamailio.org

21 participants
301 discussions

Start a nNew thread

[kamailio/kamailio] Fix Makefile.defs and influxdbc for build on FreeBSD (PR #4099)
by Boris Korzun 15 Jan '25

15 Jan '25

4 7

[kamailio/kamailio] lib/srdb1/schema: ims_dialog definitions (PR #4101)
by Victor Seva 15 Jan '25

15 Jan '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:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [x] Related to issue #4100 #### Description db_schema for ims_dialog module You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4101 -- Commit Summary -- * lib/srdb1/schema: ims_dialog definitions * utils/kamctl: regenerate db table definitions -- File Changes -- A src/lib/srdb1/schema/dialog_in.xml (180) A src/lib/srdb1/schema/dialog_out.xml (116) A src/lib/srdb1/schema/kamailio-ims_dialog.xml (14) A utils/kamctl/db_berkeley/kamailio/dialog_in (10) A utils/kamctl/db_berkeley/kamailio/dialog_out (10) M utils/kamctl/db_berkeley/kamailio/version (6) A utils/kamctl/db_redis/kamailio/dialog_in (2) A utils/kamctl/db_redis/kamailio/dialog_out (2) A utils/kamctl/db_sqlite/ims_dialog-create.sql (55) A utils/kamctl/dbtext/kamailio/dialog_in (1) A utils/kamctl/dbtext/kamailio/dialog_out (1) M utils/kamctl/dbtext/kamailio/version (4) A utils/kamctl/mongodb/kamailio/dialog_in.json (101) A utils/kamctl/mongodb/kamailio/dialog_out.json (61) M utils/kamctl/mongodb/kamailio/version-create.mongo (2) M utils/kamctl/mysql/ims_dialog-create.sql (87) A utils/kamctl/oracle/ims_dialog-create.sql (79) A utils/kamctl/postgres/ims_dialog-create.sql (55) A utils/kamctl/xhttp_pi/ims_dialog-mod (192) A utils/kamctl/xhttp_pi/ims_dialog-table (50) M utils/kamctl/xhttp_pi/pi_framework.xml (242) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4101.patch https://github.com/kamailio/kamailio/pull/4101.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4101 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4101(a)github.com>

2 8

git:master:c8290b8f: utils/kamctl: regenerate db table definitions
by Victor Seva 15 Jan '25

15 Jan '25

Module: kamailio Branch: master Commit: c8290b8f4ca35f88265073301de4c38e40516206 URL: https://github.com/kamailio/kamailio/commit/c8290b8f4ca35f88265073301de4c38… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-01-15T17:28:12+01:00 utils/kamctl: regenerate db table definitions --- Added: utils/kamctl/db_berkeley/kamailio/dialog_in Added: utils/kamctl/db_berkeley/kamailio/dialog_out Added: utils/kamctl/db_redis/kamailio/dialog_in Added: utils/kamctl/db_redis/kamailio/dialog_out Added: utils/kamctl/db_sqlite/ims_dialog-create.sql Added: utils/kamctl/dbtext/kamailio/dialog_in Added: utils/kamctl/dbtext/kamailio/dialog_out Added: utils/kamctl/mongodb/kamailio/dialog_in.json Added: utils/kamctl/mongodb/kamailio/dialog_out.json Added: utils/kamctl/oracle/ims_dialog-create.sql Added: utils/kamctl/postgres/ims_dialog-create.sql Added: utils/kamctl/xhttp_pi/ims_dialog-mod Added: utils/kamctl/xhttp_pi/ims_dialog-table Modified: utils/kamctl/db_berkeley/kamailio/version Modified: utils/kamctl/dbtext/kamailio/version Modified: utils/kamctl/mongodb/kamailio/version-create.mongo Modified: utils/kamctl/mysql/ims_dialog-create.sql Modified: utils/kamctl/xhttp_pi/pi_framework.xml --- Diff: https://github.com/kamailio/kamailio/commit/c8290b8f4ca35f88265073301de4c38… Patch: https://github.com/kamailio/kamailio/commit/c8290b8f4ca35f88265073301de4c38…

1 0

git:master:ef6df942: lib/srdb1/schema: ims_dialog definitions
by Victor Seva 15 Jan '25

15 Jan '25

Module: kamailio Branch: master Commit: ef6df94258c7ebe542feca9af241de41ca048785 URL: https://github.com/kamailio/kamailio/commit/ef6df94258c7ebe542feca9af241de4… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-01-15T17:28:12+01:00 lib/srdb1/schema: ims_dialog definitions fixes #4100 --- Added: src/lib/srdb1/schema/dialog_in.xml Added: src/lib/srdb1/schema/dialog_out.xml Added: src/lib/srdb1/schema/kamailio-ims_dialog.xml --- Diff: https://github.com/kamailio/kamailio/commit/ef6df94258c7ebe542feca9af241de4… Patch: https://github.com/kamailio/kamailio/commit/ef6df94258c7ebe542feca9af241de4…

1 0

git:master:6839f1a2: cmake/kamctl: Build and install to correct paths
by Xenofon Karamanos 15 Jan '25

15 Jan '25

Module: kamailio Branch: master Commit: 6839f1a2e090f06b0b7ca9289dd3b7279903a1e2 URL: https://github.com/kamailio/kamailio/commit/6839f1a2e090f06b0b7ca9289dd3b72… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xk(a)gilawa.com> Date: 2025-01-15T14:42:31Z cmake/kamctl: Build and install to correct paths - Some filebased databases require the kamailio subfolder --- Modified: utils/kamctl/CMakeLists.txt --- Diff: https://github.com/kamailio/kamailio/commit/6839f1a2e090f06b0b7ca9289dd3b72… Patch: https://github.com/kamailio/kamailio/commit/6839f1a2e090f06b0b7ca9289dd3b72… --- diff --git a/utils/kamctl/CMakeLists.txt b/utils/kamctl/CMakeLists.txt index 215399be4ae..c5eb602a18a 100644 --- a/utils/kamctl/CMakeLists.txt +++ b/utils/kamctl/CMakeLists.txt @@ -274,8 +274,10 @@ else() OR db_name STREQUAL "dbtext" OR db_name STREQUAL "mongodb") set(prefix '') + set(folder_suffix "${MAIN_NAME}") else() set(prefix "${table}-") + set(folder_suffix '') endif() add_custom_command( @@ -283,7 +285,8 @@ else() PRE_BUILD COMMAND XML_CATALOG_FILES=${CATALOG} ${XSLTPROC_EXECUTABLE} ${XSLTPROC_FLAGS} - --stringparam dir ${CMAKE_CURRENT_BINARY_DIR}/${db_name_folder} + --stringparam dir + ${CMAKE_CURRENT_BINARY_DIR}/${db_name_folder}/${folder_suffix} --stringparam prefix ${prefix} --stringparam db ${db_name} ${xsl_file} "kamailio-${table}.xml" WORKING_DIRECTORY "${CMAKE_SOURCE_DIR}/src/lib/srdb1/schema"

1 0

git:master:3cda5a83: cmake: Remove LOCK_METHOD as an option
by Xenofon Karamanos 15 Jan '25

15 Jan '25

Module: kamailio Branch: master Commit: 3cda5a83cc45da2b0a9d6bb18f1b7f85849c526f URL: https://github.com/kamailio/kamailio/commit/3cda5a83cc45da2b0a9d6bb18f1b7f8… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <xk(a)gilawa.com> Date: 2025-01-15T10:52:50Z cmake: Remove LOCK_METHOD as an option - If USE_FAST_LOCK is set to ON, try to use FAST_LOCK. If not avalaible on platoform, fallback to a platform specific one. - If USE_FAST_LOCK is set to OFF, fallback directly to platform specific one. - Fix wrong IF(NOT <option>) evaluation - CMake lint files --- Modified: cmake/defs.cmake Modified: cmake/os-specific/darwin.cmake Modified: cmake/os-specific/dragonfly.cmake Modified: cmake/os-specific/freebsd.cmake Modified: cmake/os-specific/linux.cmake --- Diff: https://github.com/kamailio/kamailio/commit/3cda5a83cc45da2b0a9d6bb18f1b7f8… Patch: https://github.com/kamailio/kamailio/commit/3cda5a83cc45da2b0a9d6bb18f1b7f8…

1 0

[kamailio/kamailio] cmake: add FreeBSD support (PR #4104)
by Boris Korzun 15 Jan '25

15 Jan '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) - [ ] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist: - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description I've tested cmake-build in the FreeBSD environment, as requested by @miconda in #4099. And founded some troubles. First of all are: specific flags for FreeBSD and arch-name for FreeBSD cmake-environment. I've fixed it in the PR. But there're some unresolved problems I want to discuss... A `LOCK_METHOD` problem: There's a `LOCK_METHOD` variable in defs.cmake: https://github.com/kamailio/kamailio/blob/6b0b8cb84b7d0e965d50bdf0dfa542300… And the explicit usage `LOCK_METHOD` as `USE_FUTEX` in a compile_definitions: https://github.com/kamailio/kamailio/blob/6b0b8cb84b7d0e965d50bdf0dfa542300… I suppose it's incorrect while some OSs (FreeBSD, ex.) doesn't tested with futexlock.h earlier. A resolv-link problem: There're an explicit links of the resolv library: https://github.com/kamailio/kamailio/blob/6b0b8cb84b7d0e965d50bdf0dfa542300… https://github.com/kamailio/kamailio/blob/6b0b8cb84b7d0e965d50bdf0dfa542300… I think it's incorrect while some OSs (FreeBSD, ex.) uses builtin resolv (libc, etc). Initially сmake is built successful after the above fixes. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4104 -- Commit Summary -- * cmake: add amd64 arch fallback * cmake: add FreeBSD support -- File Changes -- M cmake/compiler-specific.cmake (2) M cmake/defs.cmake (4) M cmake/os-specific.cmake (3) A cmake/os-specific/freebsd.cmake (37) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4104.patch https://github.com/kamailio/kamailio/pull/4104.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4104 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4104(a)github.com>

3 11

git:5.8:50e69e9a: sanity: log as debug when Content-Length is missing for datagram protocols
by Daniel-Constantin Mierla 15 Jan '25

15 Jan '25

Module: kamailio Branch: 5.8 Commit: 50e69e9a269e5d5dba0251f893abc34907a12970 URL: https://github.com/kamailio/kamailio/commit/50e69e9a269e5d5dba0251f893abc34… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-01-15T09:25:11+01:00 sanity: log as debug when Content-Length is missing for datagram protocols - the header is optional for datagram protocols (RFC3261, 20.14) (cherry picked from commit e38c7f8bac1aef9890d266fa466cea1561346732) --- Modified: src/modules/sanity/sanity.c --- Diff: https://github.com/kamailio/kamailio/commit/50e69e9a269e5d5dba0251f893abc34… Patch: https://github.com/kamailio/kamailio/commit/50e69e9a269e5d5dba0251f893abc34… --- diff --git a/src/modules/sanity/sanity.c b/src/modules/sanity/sanity.c index fa3d0493883..44765f7a17f 100644 --- a/src/modules/sanity/sanity.c +++ b/src/modules/sanity/sanity.c @@ -625,7 +625,11 @@ int check_cl(sip_msg_t *msg) } LM_DBG("check_cl passed\n"); } else { - LM_WARN("content length header missing in request\n"); + if(msg->rcv.proto != PROTO_UDP && msg->rcv.proto != PROTO_SCTP) { + LM_WARN("content length header missing in request\n"); + } else { + LM_DBG("content length header missing in request\n"); + } } return SANITY_CHECK_PASSED;

1 0

git:master:e38c7f8b: sanity: log as debug when Content-Length is missing for datagram protocols
by Daniel-Constantin Mierla 15 Jan '25

15 Jan '25

Module: kamailio Branch: master Commit: e38c7f8bac1aef9890d266fa466cea1561346732 URL: https://github.com/kamailio/kamailio/commit/e38c7f8bac1aef9890d266fa466cea1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-01-15T07:26:58+01:00 sanity: log as debug when Content-Length is missing for datagram protocols - the header is optional for datagram protocols (RFC3261, 20.14) --- Modified: src/modules/sanity/sanity.c --- Diff: https://github.com/kamailio/kamailio/commit/e38c7f8bac1aef9890d266fa466cea1… Patch: https://github.com/kamailio/kamailio/commit/e38c7f8bac1aef9890d266fa466cea1… --- diff --git a/src/modules/sanity/sanity.c b/src/modules/sanity/sanity.c index 45864b47523..582a1aaada0 100644 --- a/src/modules/sanity/sanity.c +++ b/src/modules/sanity/sanity.c @@ -627,7 +627,11 @@ int check_cl(sip_msg_t *msg) } LM_DBG("check_cl passed\n"); } else { - LM_WARN("content length header missing in request\n"); + if(msg->rcv.proto != PROTO_UDP && msg->rcv.proto != PROTO_SCTP) { + LM_WARN("content length header missing in request\n"); + } else { + LM_DBG("content length header missing in request\n"); + } } return SANITY_CHECK_PASSED;

1 0

git:master:0498446f: cmake: add FreeBSD support
by Xenofon Karamanos 14 Jan '25

14 Jan '25

Module: kamailio Branch: master Commit: 0498446fd24462a7de72f73aae37a76ceabc4276 URL: https://github.com/kamailio/kamailio/commit/0498446fd24462a7de72f73aae37a76… Author: drTr0jan <drtr0jan(a)yandex.ru> Committer: Xenofon Karamanos <xenakiskaramanos(a)gmail.com> Date: 2025-01-14T19:05:39+02:00 cmake: add FreeBSD support --- Added: cmake/os-specific/freebsd.cmake Modified: cmake/os-specific.cmake --- Diff: https://github.com/kamailio/kamailio/commit/0498446fd24462a7de72f73aae37a76… Patch: https://github.com/kamailio/kamailio/commit/0498446fd24462a7de72f73aae37a76… --- diff --git a/cmake/os-specific.cmake b/cmake/os-specific.cmake index d70a44cf379..ab67a4f7d68 100644 --- a/cmake/os-specific.cmake +++ b/cmake/os-specific.cmake @@ -12,6 +12,9 @@ elseif(CMAKE_SYSTEM_NAME STREQUAL "DragonFly") elseif(CMAKE_SYSTEM_NAME STREQUAL "Darwin") # Mac OS X specific flags include(${OS_SPECIFIC_DIR}/darwin.cmake) +elseif(CMAKE_SYSTEM_NAME STREQUAL "FreeBSD") + # FreeBSD specific flags + include(${OS_SPECIFIC_DIR}/freebsd.cmake) elseif() message(FATAL_ERROR "Unsupported system: ${CMAKE_SYSTEM_NAME}") endif() diff --git a/cmake/os-specific/freebsd.cmake b/cmake/os-specific/freebsd.cmake new file mode 100644 index 00000000000..03d1f2032c3 --- /dev/null +++ b/cmake/os-specific/freebsd.cmake @@ -0,0 +1,37 @@ +message( + STATUS + "Configuring for FreeBSD" +) + +target_compile_definitions( + common + INTERFACE HAVE_SOCKADDR_SA_LEN + HAVE_GETHOSTBYNAME2 + HAVE_UNION_SEMUN + HAVE_SCHED_YIELD + HAVE_MSGHDR_MSG_CONTROL + HAVE_CONNECT_ECONNRESET_BUG + HAVE_TIMEGM + HAVE_IP_MREQN +) + +if(${RAW_SOCKS}) + target_compile_definitions(common INTERFACE USE_RAW_SOCKS) +endif() + +message(STATUS "USE_FAST_LOCK = ${USE_FAST_LOCK}") +if(NOT ${USE_FAST_LOCK}) + target_compile_definitions(common INTERFACE USE_PTHREAD_MUTEX) +endif() + +if(NOT DEFINED ${NO_SELECT}) + target_compile_definitions(common INTERFACE HAVE_SELECT) +endif() + +if(NOT DEFINED ${NO_KQUEUE}) + target_compile_definitions(common INTERFACE HAVE_KQUEUE) +endif() + +if(NOT DEFINED RUN_PREFIX) + set(RUN_PREFIX "/var") +endif()

1 0

← Newer
1
...
21
22
23
24
25
26
27
...
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