Module: kamailio
Branch: master
Commit: 6235369e1deab4b87c999d5f00fb59b6609b3df6
URL: https://github.com/kamailio/kamailio/commit/6235369e1deab4b87c999d5f00fb59b…
Author: Xenofon Karamanos <xk(a)gilawa.com>
Committer: Xenofon Karamanos <xk(a)gilawa.com>
Date: 2024-12-03T14:19:43Z
cmake: Update platform-specific flags
- Update the platform-specific flags in the "os-specific.cmake" file for Linux and DragonFly BSD.
- Add new files "dragonfly.cmake" and "linux.cmake" in the "os-specific" directory to define the flags for DragonFly BSD and Linux, respectively.
- Add a new file "macos.cmake" in the "os-specific" directory to define the flags for Mac OS X.
- Update the "defs.cmake" file to print the availability of fast lock.
---
Added: cmake/os-specific/dragonfly.cmake
Added: cmake/os-specific/linux.cmake
Added: cmake/os-specific/macos.cmake
Modified: cmake/defs.cmake
Modified: cmake/os-specific.cmake
---
Diff: https://github.com/kamailio/kamailio/commit/6235369e1deab4b87c999d5f00fb59b…
Patch: https://github.com/kamailio/kamailio/commit/6235369e1deab4b87c999d5f00fb59b…
### Description
When calling inline functions using `app_python3s_exec()`, setting the level to critical using either `KSR.xlog.xcrit("foo\n")` or using `KSR.xlog.xlog("L_CRIT", "foo\n")` Do not set the level on the log line in a manner consistent with other log levels.
#### Reproduction
Kamailio code:
```
request_route {
app_python3s_exec("log_test");
. . .
}
```
Python code to contrast how info level logging and critical level logging differ:
```
def log_test():
KSR.xlog.xinfo("This line log level: xinfo = INFO\n");
KSR.xlog.xcrit("This line log level: xcrit = CRITICAL\n");
KSR.xlog.xlog("L_INFO", "This line log level: xlog = L_INFO\n");
KSR.xlog.xlog("L_CRIT", "This line log level: xlog = L_CRIT\n");
```
#### Log Messages
Log output. Note how the info level lines begin with INFO, but the critical level lines begin with an empty string:
```
2(8) INFO: <script>: This line log level: xinfo = INFO
2(8) : <script>: This line log level: xcrit = CRITICAL
2(8) INFO: <script>: This line log level: xlog = L_INFO
2(8) : <script>: This line log level: xlog = L_CRIT
```
### Possible Solutions
The critical level lines should begin with `CRITICAL` - this is what happens if the xlog function is called in Kamailio script. The output should look like this:
```
2(8) INFO: <script>: This line log level: xinfo = INFO
2(8) CRITICAL: <script>: This line log level: xcrit = CRITICAL
2(8) INFO: <script>: This line log level: xlog = L_INFO
2(8) CRITICAL: <script>: This line log level: xlog = L_CRIT
```
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.8.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, MEM_JOIN_FREE, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_SEND_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: unknown
compiled on 15:03:24 Nov 18 2024 with gcc 13.2.1
```
* **Operating System**:
Tested in docker on Windows 11 (WSL2) using alpine linux as a base. `/etc/os-releas`:
```
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.20.2
PRETTY_NAME="Alpine Linux v3.20"
HOME_URL="https://alpinelinux.org/"
BUG_REPORT_URL="https://gitlab.alpinelinux.org/alpine/aports/-/issues"
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4048
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4048(a)github.com>
Module: kamailio
Branch: master
Commit: 299db405e54741e4dc1ec04e6658d6bc99901edf
URL: https://github.com/kamailio/kamailio/commit/299db405e54741e4dc1ec04e6658d6b…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2024-12-02T18:00:18+01:00
src/Makefiles: filter out CMakeLists.txt from modules discovery
---
Modified: src/Makefile
---
Diff: https://github.com/kamailio/kamailio/commit/299db405e54741e4dc1ec04e6658d6b…
Patch: https://github.com/kamailio/kamailio/commit/299db405e54741e4dc1ec04e6658d6b…
---
diff --git a/src/Makefile b/src/Makefile
index 28106b06852..4ba662535bb 100644
--- a/src/Makefile
+++ b/src/Makefile
@@ -118,10 +118,10 @@ endif
endif
else # config.mak doesn't need to be used
ifneq (,$(filter cfg config cfg-defs,$(word 1,$(MAKECMDGOALS))))
-# needed here to avoid starting a config submake
-# (e.g. rm -f config.mak; make config.mak), which would either require
+# needed here to avoid starting a config submake
+# (e.g. rm -f config.mak; make config.mak), which would either require
# double Makefile.defs defines execution (suboptimal), would loose
-# $(value ...) expansion or would cause some warning (if Makefile.defs exec.
+# $(value ...) expansion or would cause some warning (if Makefile.defs exec.
# is skipped in the "main" makefile invocation).
$(shell rm -rf config.mak)
config_mak=0
@@ -154,12 +154,13 @@ export extra_defs
# When group_include is used, we want: include_modules (based on group_include)
# - exclude_modules
-ifneq ($(modules_configured),1)
+ifneq ($(modules_configured),1)
#modules_all=$(filter-out modules/CVS,$(wildcard modules/*))
# create modules*_all vars
$(foreach mods,$(modules_dirs),$(eval \
- $(mods)_all=$$(filter-out $(mods)/CVS,$$(wildcard $(mods)/*))))
+ $(mods)_all=$$(filter-out $(mods)/CVS $(mods)/CMakeLists.txt, \
+ $$(wildcard $(mods)/*))))
#debugging
#$(foreach mods,$(modules_dirs),$(info "$(mods)_all=$($(mods)_all)"))
@@ -168,7 +169,7 @@ $(foreach mods,$(modules_dirs),$(eval \
$(mods)=$$(filter-out $$(addprefix $(mods)/, \
$$(exclude_modules) $$(static_modules)), \
$$(addprefix $(mods)/, $$(include_modules) )) ))
-else
+else
# Standard, old resultant set
$(foreach mods,$(modules_dirs),$(eval \
$(mods)_noinc=$$(filter-out $$(addprefix $(mods)/, \
@@ -214,14 +215,14 @@ C_INSTALL_UTILS= ../utils/kamctl
else
C_INSTALL_UTILS=
endif
-# list of scripts that should be installed along the core
+# list of scripts that should be installed along the core
# (here a script is something that doesn't have a Makefile)
C_INSTALL_SCRIPTS=
# list of extra configs that should be installed along the core
# Note: all the paths of the form /usr/*lib/$(CFG_NAME)/<module_dir>
# will be updated to the directory where the modules will be installed.
C_INSTALL_CFGS=
-# list of files that should be installed in the arch-independent
+# list of files that should be installed in the arch-independent
# directory (by default /usr/local/share/$(MAIN_NAME)))
C_INSTALL_SHARE=
@@ -237,11 +238,11 @@ tar_extra_args+=$(addprefix --exclude=$(notdir $(CURDIR))/, \
ifeq ($(CORE_TLS), 1)
tar_extra_args+=
else
- tar_extra_args+=--exclude=$(notdir $(CURDIR))/tls/*
+ tar_extra_args+=--exclude=$(notdir $(CURDIR))/tls/*
endif
ifneq ($(nodeb),)
- tar_extra_args+=--exclude=$(notdir $(CURDIR))/debian
+ tar_extra_args+=--exclude=$(notdir $(CURDIR))/debian
tar_name:=$(tar_name)_nodeb
endif
@@ -513,7 +514,7 @@ $(foreach mods,$(modules_dirs),$(eval $(call MODULES_RULES_template,$(mods))))
modules-all every-module: $(modules_dirs)
$(extra_objs):
- @echo "Extra objs: $(extra_objs)"
+ @echo "Extra objs: $(extra_objs)"
@for r in $(static_modules_path) "" ; do \
if [ -n "$$r" -a -r "$$r/Makefile" ]; then \
$(call oecho, "" ;) \
@@ -670,10 +671,10 @@ mk-install_dirs: $(cfg_prefix)/$(cfg_dir) $(bin_prefix)/$(bin_dir) \
$(modules_prefix)/$(modules_dir)$(mods) \
$(doc_prefix)/$(doc_dir)$(mods) )
-$(cfg_prefix)/$(cfg_dir):
+$(cfg_prefix)/$(cfg_dir):
mkdir -p $(cfg_prefix)/$(cfg_dir)
-$(run_prefix)/$(run_dir):
+$(run_prefix)/$(run_dir):
mkdir -p $(run_prefix)/$(run_dir)
$(bin_prefix)/$(bin_dir):
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
This PR adds initial support for building kamailio with CMake.
See [docs](https://github.com/kamailio/kamailio-wiki/pull/63) for more information on how to build and use CMake.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4043
-- Commit Summary --
* core/resovle: Fix compile if DNS_USE_CACHE not defined
* core/forward: Define ret even if USE_TCP is not defined
* Add build folder to ignore
* Add CMake support
* Core compiles and runs successfully on Linux
* Add some notes regarding definitions
* Add missing definitions
* CMake: Clean up and minor fixes
* CMake: Add install targets for binary,cfg,kamctl and kamdbctl
* CMake: Install utils (kamctl,kamdbctl and required) with sed changes
* CMake: Move notes out of pkg
* CMake: Add uninstall target
* CMake: FIx core building and installing
* CMake: Add lib support
* CMake: Add initial module support
* CMake: Fix formating
* Update gitignore/Add cmake-format file
* Add aarch64 options
* Make cross-compile available
* Format CMakeLists with cmake-format
* Add build docs support
* cmake: add github action
* cmake: Remove windows from matrix
* cmake: Add aarch64 cross-compile
* cmake: rename arm to aarch64 arch
* cmake: Add clang support
* cmake: Remove duplicate definition
* cmake: Upgrade warning to errors to match clang
* cmake: Add target host definition
* cmake: Allow only by trigger
* cmake: Add cmake to most (all) modules
* cmake: declare dproto
* cmake: Add MOD_NAMEID definitions to modules
* cmake: modify build_rpath
* Change rpath
* cmake: Move kamailio to root folder
* cmake: Modify run path of modules
* cmake: var/run path change
* cmake: make it abs path
* cmake: Add option to select groups
* cmake: Add all modules group
* cmake: Default group set to default modules
* cmake: groups update, fixup me
* cmake: Add Find MariaDB and MySQL cmake
* cmake: Add absolute path for /var/run (fixup with other commit)
* cmake: Only search for mariadb
* cmake: Update mysql cmake
* cmake: Remove app_mono
* cmake: modify FindMariaDBCLient.cmake
* cmake: Add description to use_mcast option
* cmake: Fix format
* cmake: Add config files to uninstall target
* cmake: Add FindRadius to discover radius client
* cmake: Modify dependencies so parser c files are generated when needed
* cmake: Policy to link against targets defined in other CMakelists.
* cmake: Define MOD_NAME for all modules
* cmake: Remove RPath from module's Cmake and use PRIVATE on all
* cmake: Add support to exclude modules
* cmake: Modules with external deps ready
* cmake: Add srdb2 CMakeLists.txt
* cmake: Add various Find[Module] for cmake
* cmake: Add FindXXX to dedicated folder
* cmake: Add CPack for packaging (experimental)
* camke: Clean up some commented sections
* cmake/modules: Add NETSNMP
* cmake/libs: Modify targets with lib_ and add more libs
* cmake/modules: Update ALL modules CMakeLists.txt
-- File Changes --
A .github/workflows/cmake_build.yml (84)
M .gitignore (2)
A CMakeLists.txt (61)
A CMakelists.notes (54)
A cmake/aarch64-toolchain.cmake (28)
A cmake/cmake-format.py (240)
A cmake/cmake_uninstall.cmake.in (32)
A cmake/modules/FindBerkeleyDB.cmake (182)
A cmake/modules/FindErlang.cmake (151)
A cmake/modules/FindLdap.cmake (119)
A cmake/modules/FindLibev.cmake (60)
A cmake/modules/FindMariaDBClient.cmake (68)
A cmake/modules/FindMySQL.cmake (964)
A cmake/modules/FindNETSNMP.cmake (76)
A cmake/modules/FindOracle.cmake (115)
A cmake/modules/FindRadius.cmake (51)
A cmake/modules_docs.cmake (74)
A src/CMakeLists.txt (164)
A src/compiler-specific.cmake (55)
A src/core/CMakeLists.txt (57)
A src/core/autover.h.in (6)
M src/core/forward.h (6)
M src/core/resolve.c (8)
A src/defs.cmake (318)
A src/lib/CMakeLists.txt (30)
A src/lib/ims/CMakeLists.txt (22)
A src/lib/print/CMakeLists.txt (18)
A src/lib/srdb1/CMakeLists.txt (17)
A src/lib/srdb2/CMakeLists.txt (15)
A src/lib/trie/CMakeLists.txt (16)
A src/modules/CMakeLists.txt (143)
A src/modules/acc/CMakeLists.txt (10)
A src/modules/acc_diameter/CMakeLists.txt (8)
A src/modules/acc_json/CMakeLists.txt (12)
A src/modules/acc_radius/CMakeLists.txt (11)
A src/modules/alias_db/CMakeLists.txt (10)
A src/modules/app_java/CMakeLists.txt (14)
A src/modules/app_jsdt/CMakeLists.txt (8)
A src/modules/app_lua/CMakeLists.txt (22)
A src/modules/app_perl/CMakeLists.txt (14)
A src/modules/app_python/CMakeLists.txt (11)
A src/modules/app_python3/CMakeLists.txt (10)
A src/modules/app_python3s/CMakeLists.txt (10)
A src/modules/app_ruby/CMakeLists.txt (10)
A src/modules/app_ruby_proc/CMakeLists.txt (12)
A src/modules/async/CMakeLists.txt (8)
A src/modules/auth/CMakeLists.txt (9)
A src/modules/auth_db/CMakeLists.txt (10)
A src/modules/auth_diameter/CMakeLists.txt (8)
A src/modules/auth_ephemeral/CMakeLists.txt (8)
A src/modules/auth_radius/CMakeLists.txt (11)
A src/modules/auth_xkeys/CMakeLists.txt (8)
A src/modules/avp/CMakeLists.txt (8)
A src/modules/avpops/CMakeLists.txt (10)
A src/modules/benchmark/CMakeLists.txt (8)
A src/modules/blst/CMakeLists.txt (8)
A src/modules/call_control/CMakeLists.txt (8)
A src/modules/call_obj/CMakeLists.txt (8)
A src/modules/carrierroute/CMakeLists.txt (11)
A src/modules/cdp/CMakeLists.txt (10)
A src/modules/cdp_avp/CMakeLists.txt (8)
A src/modules/cfg_db/CMakeLists.txt (10)
A src/modules/cfg_rpc/CMakeLists.txt (10)
A src/modules/cfgt/CMakeLists.txt (8)
A src/modules/cfgutils/CMakeLists.txt (8)
A src/modules/cnxcc/CMakeLists.txt (18)
A src/modules/corex/CMakeLists.txt (8)
A src/modules/counters/CMakeLists.txt (10)
A src/modules/cplc/CMakeLists.txt (12)
A src/modules/crypto/CMakeLists.txt (10)
A src/modules/ctl/CMakeLists.txt (12)
A src/modules/db2_ldap/CMakeLists.txt (22)
A src/modules/db2_ops/CMakeLists.txt (10)
A src/modules/db_berkeley/CMakeLists.txt (12)
A src/modules/db_cluster/CMakeLists.txt (10)
A src/modules/db_flatstore/CMakeLists.txt (10)
A src/modules/db_mongodb/CMakeLists.txt (17)
A src/modules/db_mysql/CMakeLists.txt (19)
A src/modules/db_oracle/CMakeLists.txt (16)
A src/modules/db_perlvdb/CMakeLists.txt (13)
A src/modules/db_postgres/CMakeLists.txt (14)
A src/modules/db_redis/CMakeLists.txt (21)
A src/modules/db_sqlite/CMakeLists.txt (12)
A src/modules/db_text/CMakeLists.txt (10)
A src/modules/db_unixodbc/CMakeLists.txt (13)
A src/modules/debugger/CMakeLists.txt (8)
A src/modules/dialog/CMakeLists.txt (10)
A src/modules/dialplan/CMakeLists.txt (10)
A src/modules/dispatcher/CMakeLists.txt (10)
A src/modules/diversion/CMakeLists.txt (8)
A src/modules/dlgs/CMakeLists.txt (8)
A src/modules/dmq/CMakeLists.txt (10)
A src/modules/dmq_usrloc/CMakeLists.txt (8)
A src/modules/dnssec/CMakeLists.txt (19)
A src/modules/domain/CMakeLists.txt (10)
A src/modules/domainpolicy/CMakeLists.txt (10)
A src/modules/drouting/CMakeLists.txt (10)
A src/modules/enum/CMakeLists.txt (8)
A src/modules/erlang/CMakeLists.txt (11)
A src/modules/evapi/CMakeLists.txt (10)
A src/modules/evrexec/CMakeLists.txt (8)
A src/modules/exec/CMakeLists.txt (8)
A src/modules/file_out/CMakeLists.txt (8)
A src/modules/gcrypt/CMakeLists.txt (12)
A src/modules/geoip/CMakeLists.txt (12)
A src/modules/geoip2/CMakeLists.txt (13)
A src/modules/group/CMakeLists.txt (10)
A src/modules/groups.cmake (686)
A src/modules/gzcompress/CMakeLists.txt (12)
A src/modules/h350/CMakeLists.txt (8)
A src/modules/htable/CMakeLists.txt (10)
A src/modules/http_async_client/CMakeLists.txt (16)
A src/modules/http_client/CMakeLists.txt (10)
A src/modules/imc/CMakeLists.txt (10)
A src/modules/ims_auth/CMakeLists.txt (10)
A src/modules/ims_charging/CMakeLists.txt (10)
A src/modules/ims_dialog/CMakeLists.txt (10)
A src/modules/ims_diameter_server/CMakeLists.txt (10)
A src/modules/ims_icscf/CMakeLists.txt (12)
A src/modules/ims_ipsec_pcscf/CMakeLists.txt (14)
A src/modules/ims_isc/CMakeLists.txt (11)
A src/modules/ims_ocs/CMakeLists.txt (10)
A src/modules/ims_qos/CMakeLists.txt (10)
A src/modules/ims_qos_npn/CMakeLists.txt (10)
A src/modules/ims_registrar_pcscf/CMakeLists.txt (10)
A src/modules/ims_registrar_scscf/CMakeLists.txt (10)
A src/modules/ims_usrloc_pcscf/CMakeLists.txt (10)
A src/modules/ims_usrloc_scscf/CMakeLists.txt (10)
A src/modules/influxdbc/CMakeLists.txt (8)
A src/modules/ipops/CMakeLists.txt (8)
A src/modules/jansson/CMakeLists.txt (12)
A src/modules/janssonrpcc/CMakeLists.txt (16)
A src/modules/json/CMakeLists.txt (16)
A src/modules/jsonrpcc/CMakeLists.txt (16)
A src/modules/jsonrpcs/CMakeLists.txt (8)
A src/modules/jwt/CMakeLists.txt (16)
A src/modules/kafka/CMakeLists.txt (16)
A src/modules/kazoo/CMakeLists.txt (33)
A src/modules/keepalive/CMakeLists.txt (10)
A src/modules/kemix/CMakeLists.txt (8)
A src/modules/kex/CMakeLists.txt (8)
A src/modules/lcr/CMakeLists.txt (19)
A src/modules/ldap/CMakeLists.txt (10)
A src/modules/log_custom/CMakeLists.txt (8)
A src/modules/log_systemd/CMakeLists.txt (12)
A src/modules/lost/CMakeLists.txt (11)
A src/modules/lrkproxy/CMakeLists.txt (8)
A src/modules/lwsc/CMakeLists.txt (12)
A src/modules/mangler/CMakeLists.txt (8)
A src/modules/math/CMakeLists.txt (8)
A src/modules/matrix/CMakeLists.txt (10)
A src/modules/maxfwd/CMakeLists.txt (8)
A src/modules/mediaproxy/CMakeLists.txt (8)
A src/modules/memcached/CMakeLists.txt (12)
A src/modules/microhttpd/CMakeLists.txt (13)
A src/modules/misc_radius/CMakeLists.txt (12)
A src/modules/misctest/CMakeLists.txt (8)
A src/modules/mohqueue/CMakeLists.txt (10)
A src/modules/mqtt/CMakeLists.txt (14)
A src/modules/mqueue/CMakeLists.txt (10)
A src/modules/msilo/CMakeLists.txt (10)
A src/modules/msrp/CMakeLists.txt (8)
A src/modules/mtree/CMakeLists.txt (10)
A src/modules/nat_traversal/CMakeLists.txt (8)
A src/modules/nathelper/CMakeLists.txt (8)
A src/modules/nats/CMakeLists.txt (12)
A src/modules/ndb_cassandra/CMakeLists.txt (12)
A src/modules/ndb_mongodb/CMakeLists.txt (12)
A src/modules/ndb_redis/CMakeLists.txt (12)
A src/modules/nghttp2/CMakeLists.txt (20)
A src/modules/nosip/CMakeLists.txt (8)
A src/modules/nsq/CMakeLists.txt (29)
A src/modules/outbound/CMakeLists.txt (10)
A src/modules/p_usrloc/CMakeLists.txt (10)
A src/modules/path/CMakeLists.txt (8)
A src/modules/pdb/CMakeLists.txt (8)
A src/modules/pdt/CMakeLists.txt (10)
A src/modules/peering/CMakeLists.txt (11)
A src/modules/permissions/CMakeLists.txt (10)
A src/modules/phonenum/CMakeLists.txt (49)
A src/modules/pike/CMakeLists.txt (8)
A src/modules/pipelimit/CMakeLists.txt (10)
A src/modules/posops/CMakeLists.txt (8)
A src/modules/prefix_route/CMakeLists.txt (10)
A src/modules/presence/CMakeLists.txt (14)
A src/modules/presence_conference/CMakeLists.txt (10)
A src/modules/presence_dialoginfo/CMakeLists.txt (10)
A src/modules/presence_mwi/CMakeLists.txt (8)
A src/modules/presence_profile/CMakeLists.txt (8)
A src/modules/presence_reginfo/CMakeLists.txt (10)
A src/modules/presence_xml/CMakeLists.txt (13)
A src/modules/print/CMakeLists.txt (8)
A src/modules/print_lib/CMakeLists.txt (11)
A src/modules/pua/CMakeLists.txt (12)
A src/modules/pua_bla/CMakeLists.txt (10)
A src/modules/pua_dialoginfo/CMakeLists.txt (10)
A src/modules/pua_json/CMakeLists.txt (16)
A src/modules/pua_reginfo/CMakeLists.txt (10)
A src/modules/pua_rpc/CMakeLists.txt (9)
A src/modules/pua_usrloc/CMakeLists.txt (10)
A src/modules/pua_xmpp/CMakeLists.txt (10)
A src/modules/pv/CMakeLists.txt (9)
A src/modules/pv_headers/CMakeLists.txt (8)
A src/modules/pvtpl/CMakeLists.txt (8)
A src/modules/qos/CMakeLists.txt (8)
A src/modules/rabbitmq/CMakeLists.txt (26)
A src/modules/ratelimit/CMakeLists.txt (8)
A src/modules/regex/CMakeLists.txt (17)
A src/modules/registrar/CMakeLists.txt (9)
A src/modules/rls/CMakeLists.txt (15)
A src/modules/rr/CMakeLists.txt (8)
A src/modules/rtimer/CMakeLists.txt (8)
A src/modules/rtjson/CMakeLists.txt (8)
A src/modules/rtp_media_server/CMakeLists.txt (24)
A src/modules/rtpengine/CMakeLists.txt (10)
A src/modules/rtpproxy/CMakeLists.txt (10)
A src/modules/ruxc/CMakeLists.txt (18)
A src/modules/sanity/CMakeLists.txt (8)
A src/modules/sca/CMakeLists.txt (10)
A src/modules/sctp/CMakeLists.txt (11)
A src/modules/sdpops/CMakeLists.txt (8)
A src/modules/seas/CMakeLists.txt (8)
A src/modules/secfilter/CMakeLists.txt (10)
A src/modules/secsipid/CMakeLists.txt (8)
A src/modules/secsipid_proc/CMakeLists.txt (21)
A src/modules/sipcapture/CMakeLists.txt (10)
A src/modules/sipdump/CMakeLists.txt (8)
A src/modules/sipjson/CMakeLists.txt (8)
A src/modules/siprepo/CMakeLists.txt (8)
A src/modules/sipt/CMakeLists.txt (8)
A src/modules/siptrace/CMakeLists.txt (10)
A src/modules/siputils/CMakeLists.txt (10)
A src/modules/sl/CMakeLists.txt (8)
A src/modules/slack/CMakeLists.txt (10)
A src/modules/sms/CMakeLists.txt (8)
A src/modules/smsops/CMakeLists.txt (8)
A src/modules/snmpstats/CMakeLists.txt (23)
A src/modules/speeddial/CMakeLists.txt (10)
A src/modules/sqlops/CMakeLists.txt (10)
A src/modules/ss7ops/CMakeLists.txt (8)
A src/modules/sst/CMakeLists.txt (8)
A src/modules/statistics/CMakeLists.txt (8)
A src/modules/statsc/CMakeLists.txt (8)
A src/modules/statsd/CMakeLists.txt (8)
A src/modules/stirshaken/CMakeLists.txt (12)
A src/modules/stun/CMakeLists.txt (8)
A src/modules/sworker/CMakeLists.txt (8)
A src/modules/systemdops/CMakeLists.txt (12)
A src/modules/tcpops/CMakeLists.txt (8)
A src/modules/textops/CMakeLists.txt (8)
A src/modules/textopsx/CMakeLists.txt (8)
A src/modules/timer/CMakeLists.txt (8)
A src/modules/tls/CMakeLists.txt (46)
A src/modules/tls_wolfssl/CMakeLists.txt (15)
A src/modules/tlsa/CMakeLists.txt (35)
A src/modules/tm/CMakeLists.txt (12)
A src/modules/tmrec/CMakeLists.txt (8)
A src/modules/tmx/CMakeLists.txt (9)
A src/modules/topoh/CMakeLists.txt (8)
A src/modules/topos/CMakeLists.txt (10)
A src/modules/topos_redis/CMakeLists.txt (12)
A src/modules/tsilo/CMakeLists.txt (12)
A src/modules/uac/CMakeLists.txt (10)
A src/modules/uac_redirect/CMakeLists.txt (8)
A src/modules/uid_auth_db/CMakeLists.txt (9)
A src/modules/uid_avp_db/CMakeLists.txt (9)
A src/modules/uid_domain/CMakeLists.txt (10)
A src/modules/uid_gflags/CMakeLists.txt (10)
A src/modules/uid_uri_db/CMakeLists.txt (10)
A src/modules/uri_db/CMakeLists.txt (10)
A src/modules/userblocklist/CMakeLists.txt (10)
A src/modules/usrloc/CMakeLists.txt (10)
A src/modules/utils/CMakeLists.txt (12)
A src/modules/uuid/CMakeLists.txt (12)
A src/modules/websocket/CMakeLists.txt (17)
A src/modules/xcap_client/CMakeLists.txt (12)
A src/modules/xcap_server/CMakeLists.txt (12)
A src/modules/xhttp/CMakeLists.txt (8)
A src/modules/xhttp_pi/CMakeLists.txt (12)
A src/modules/xhttp_prom/CMakeLists.txt (8)
A src/modules/xhttp_rpc/CMakeLists.txt (8)
A src/modules/xlog/CMakeLists.txt (8)
A src/modules/xmlops/CMakeLists.txt (10)
A src/modules/xmlrpc/CMakeLists.txt (12)
A src/modules/xmpp/CMakeLists.txt (13)
A src/modules/xprint/CMakeLists.txt (8)
A src/os-specific.cmake (97)
A utils/kamctl/CMakeLists.txt (196)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4043.patchhttps://github.com/kamailio/kamailio/pull/4043.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4043
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4043(a)github.com>
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [x] Related to issue #3814
#### Description
<!-- Describe your changes in detail -->
When the fixup is done in main using fix_rls function (fixing the route lists in cfg file), all the children inherit the memory allocated as a copy that needs separate management.
The function responsible for the fixup (fix_actions) is also used to call the free_fixup function with an added argument.
Then on main, we call the free_rls ( as fix_rls) on two separate occasions. Once in cleanup() function for the main process and once in the sig_usr() function for each of the children.
This seems to work fine and after debugging with `file_out` fixups, I can no longer see memory leaks due to the malloced memory in one of the fixups.
- main.c: Call free_rls() in cleanup() function to clean route lists and fixed up parameters in main process.
- main.c: Call free_rls() in sig_usr() function to clean route lists and fixed up parameters in child processes.
My question is, after a parameter is fixed, is it saved somewhere else, therefore we can immediately free the fix-up resources and not on shutdown?
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4023
-- Commit Summary --
* core: Add fix_actions function parameter in fix_actions to free fixed up resources
-- File Changes --
M src/core/route.c (61)
M src/core/route.h (3)
M src/core/rvalue.c (2)
M src/core/switch.c (4)
M src/main.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4023.patchhttps://github.com/kamailio/kamailio/pull/4023.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4023
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4023(a)github.com>
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
#### Description
While working on an IMS setup, I and @lynxis encountered some cases where it was unclear why some of the related modules would not load. They just did not without printing any messages as to why. I have identified a few cases where logging was missing and added it, in the hope that it might be helpful.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4035
-- Commit Summary --
* ims_scscf: Add more logging to ipsec_create()
* ims_ipsec_pcscf: Add error logging
-- File Changes --
M src/modules/ims_ipsec_pcscf/cmd.c (14)
M src/modules/ims_ipsec_pcscf/ims_ipsec_pcscf_mod.c (9)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4035.patchhttps://github.com/kamailio/kamailio/pull/4035.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4035
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4035(a)github.com>
- print entire socket address for ease of debugging with empty mask ip
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [ ] 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)
- [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)
- [ ] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4039
-- Commit Summary --
* topoh: improve debug info for topoh crash
-- File Changes --
M src/modules/topoh/topoh_mod.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4039.patchhttps://github.com/kamailio/kamailio/pull/4039.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4039
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4039(a)github.com>
```
make[2]: --libs: No such file or directory
CC (gcc) [M db_mongodb.so] db_mongodb_mod.o
CC (gcc) [M db_mongodb.so] mongodb_connection.o
CC (gcc) [M db_mongodb.so] mongodb_dbase.o
mongodb_dbase.c: In function ‘db_mongodb_convert_result’:
mongodb_dbase.c:776:25: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
776 | jstr = bson_as_json(itdoc, NULL);
| ^~~~
In file included from /usr/include/libmongoc-1.0/mongoc/mongoc.h:22,
from /usr/include/libmongoc-1.0/mongoc.h:18,
from mongodb_connection.h:26,
from mongodb_dbase.c:27:
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
mongodb_dbase.c: In function ‘db_mongodb_query’:
mongodb_dbase.c:1003:17: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
1003 | jstr = bson_as_json(seldoc, NULL);
| ^~~~
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
mongodb_dbase.c:1039:25: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
1039 | jstr = bson_as_json(mgcon->colsdoc, NULL);
| ^~~~
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
mongodb_dbase.c: In function ‘db_mongodb_insert’:
mongodb_dbase.c:1177:17: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
1177 | jstr = bson_as_json(doc, NULL);
| ^~~~
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
mongodb_dbase.c: In function ‘db_mongodb_delete’:
mongodb_dbase.c:1271:17: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
1271 | jstr = bson_as_json(doc, NULL);
| ^~~~
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
make[3]: Entering directory '/root/rpmbuild/BUILD/kamailio-6.0.0-dev3/src/lib/srdb1'
make[3]: 'libsrdb1.so.1.0' is up to date.
make[3]: Leaving directory '/root/rpmbuild/BUILD/kamailio-6.0.0-dev3/src/lib/srdb1'
LD (gcc) [M db_mongodb.so] db_mongodb.so
make[2]: --libs: No such file or directory
make[2]: --libs: No such file or directory
CC (gcc) [M ndb_mongodb.so] mongodb_client.o
mongodb_client.c: In function ‘mongodbc_exec_cmd’:
mongodb_client.c:266:17: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
266 | rpl->jsonrpl.s = bson_as_json(&reply, NULL);
| ^~~
In file included from /usr/include/libmongoc-1.0/mongoc/mongoc.h:22,
from /usr/include/libmongoc-1.0/mongoc.h:18,
from mongodb_client.h:25,
from mongodb_client.c:34:
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
mongodb_client.c:271:25: warning: ‘mongoc_collection_command’ is deprecated: Use mongoc_collection_command_simple instead [-Wdeprecated-declarations]
271 | rpl->cursor = mongoc_collection_command(rpl->collection,
| ^~~
In file included from /usr/include/libmongoc-1.0/mongoc/mongoc-client.h:26,
from /usr/include/libmongoc-1.0/mongoc/mongoc-bulkwrite.h:22,
from /usr/include/libmongoc-1.0/mongoc/mongoc.h:28:
/usr/include/libmongoc-1.0/mongoc/mongoc-collection.h:51:1: note: declared here
51 | mongoc_collection_command (mongoc_collection_t *collection,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
mongodb_client.c:306:17: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
306 | rpl->jsonrpl.s = bson_as_json(cdoc, NULL);
| ^~~
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
mongodb_client.c: In function ‘mongodbc_next_reply’:
mongodb_client.c:482:9: warning: ‘bson_as_json’ is deprecated: Use bson_as_legacy_extended_json instead [-Wdeprecated-declarations]
482 | rpl->jsonrpl.s = bson_as_json(cdoc, NULL);
| ^~~
/usr/include/libbson-1.0/bson/bson.h:535:1: note: declared here
535 | bson_as_json (const bson_t *bson, size_t *length) BSON_GNUC_DEPRECATED_FOR (bson_as_legacy_extended_json);
| ^~~~~~~~~~~~
CC (gcc) [M ndb_mongodb.so] ndb_mongodb_mod.o
LD (gcc) [M ndb_mongodb.so] ndb_mongodb.so
make[2]: --libs: No such file or directory
make[2]: --libs: No such file or directory
CC (gcc) [M db_mysql.so] db_mysql.o
CC (gcc) [M db_mysql.so] km_db_mysql.o
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4037
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4037(a)github.com>
<!--
Kamailio Project uses GitHub Issues only for bugs in the code or feature requests. Please use this template only for bug reports.
If you have questions about using Kamailio or related to its configuration file, ask on sr-users mailing list:
* https://lists.kamailio.org/mailman3/postorius/lists/sr-users.lists.kamailio…
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list:
* https://lists.kamailio.org/mailman3/postorius/lists/sr-dev.lists.kamailio.o…
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot the issue.
Note that an issue report may be closed automatically after about 2 months
if there is no interest from developers or community users on pursuing it, being
considered expired. In such case, it can be reopened by writing a comment that includes
the token `/notexpired`. About two weeks before considered expired, the issue is
marked with the label `stale`, trying to notify the submitter and everyone else
that might be interested in it. To remove the label `stale`, write a comment that
includes the token `/notstale`. Also, any comment postpone the `expire` timeline,
being considered that there is interest in pursuing the issue.
If there is no content to be filled in a section, the entire section can be removed.
You can delete the comments from the template sections when filling.
You can delete next line and everything above before submitting (it is a comment).
-->
### Description
The Python3 modules violate Python's exception handling contract.
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
### Troubleshooting
#### Reproduction
```
KSR.pc.sets("$fU",None)
```
#### Debugging Data
```
Unhandled exception in the Python code:
TypeError: kemi-param-ss() argument 2 must be str, not None
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/dist-packages/kamailio/trace.py", line 212, in pac
return proc(*a, **k)
^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/kamailio/_basic.py", line 181, in ksr_request_route
self.do_route(msg)
File "/usr/local/lib/python3.11/dist-packages/kamailio/_basic.py", line 261, in do_route
self.route_static(msg)
File "/usr/local/lib/python3.11/dist-packages/kamailio/_basic.py", line 287, in route_static
if self.fix_addrs(msg):
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/kamailio/_basic.py", line 334, in fix_addrs
PV.tU = dnr
^^^^^
File "/usr/local/lib/python3.11/dist-packages/kamailio/var.py", line 46, in __setattr__
return self.__setitem__(k, v)
^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/dist-packages/kamailio/var.py", line 73, in __setitem__
KSR.pv.sets("$fU", v)
SystemError: <built-in function sets> returned a result with an exception set
```
### Possible Solutions
See PR #4044
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
seen with 5.6 and 5.8
```
* **Operating System**:
Debian Stable but doesn't matter
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4045
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4045(a)github.com>
i am using the 10 algorithm for the call load distribution it is good but when i put more load on it at create a race condition
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4047
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4047(a)github.com>
hey kindly help me with error as i am using the call load distribution algorithm how i resolve this error
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4046
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4046(a)github.com>
```
CC (gcc) [M lost.so] pidf.o
CC (gcc) [M lost.so] response.o
CC (gcc) [M lost.so] utilities.o
utilities.c: In function ‘lost_held_post_request’:
utilities.c:1420:9: warning: ‘xmlKeepBlanksDefault’ is deprecated [-Wdeprecated-declarations]
1420 | xmlKeepBlanksDefault(1);
| ^~~~~~~~~~~~~~~~~~~~
In file included from pidf.h:37,
from utilities.c:49:
/usr/include/libxml2/libxml/parser.h:957:17: note: declared here
957 | xmlKeepBlanksDefault (int val);
| ^~~~~~~~~~~~~~~~~~~~
utilities.c: In function ‘lost_held_location_request’:
utilities.c:1518:9: warning: ‘xmlKeepBlanksDefault’ is deprecated [-Wdeprecated-declarations]
1518 | xmlKeepBlanksDefault(1);
| ^~~~~~~~~~~~~~~~~~~~
/usr/include/libxml2/libxml/parser.h:957:17: note: declared here
957 | xmlKeepBlanksDefault (int val);
| ^~~~~~~~~~~~~~~~~~~~
utilities.c: In function ‘lost_find_service_request’:
utilities.c:1671:9: warning: ‘xmlKeepBlanksDefault’ is deprecated [-Wdeprecated-declarations]
1671 | xmlKeepBlanksDefault(1);
| ^~~~~~~~~~~~~~~~~~~~
/usr/include/libxml2/libxml/parser.h:957:17: note: declared here
957 | xmlKeepBlanksDefault (int val);
| ^~~~~~~~~~~~~~~~~~~~
utilities.c: In function ‘lost_parse_geo’:
utilities.c:1050:41: warning: ‘snprintf’ output may be truncated before the last format character [-Wformat-truncation=]
1050 | snprintf(loc->latitude, len, "%s", (char *)bufLat);
| ^
utilities.c:1050:9: note: ‘snprintf’ output between 1 and 128 bytes into a destination of size 127
1050 | snprintf(loc->latitude, len, "%s", (char *)bufLat);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
utilities.c:1060:42: warning: ‘snprintf’ output may be truncated before the last format character [-Wformat-truncation=]
1060 | snprintf(loc->longitude, len, "%s", (char *)bufLon);
| ^
utilities.c:1060:9: note: ‘snprintf’ output between 1 and 128 bytes into a destination of size 127
1060 | snprintf(loc->longitude, len, "%s", (char *)bufLon);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
utilities.c:1074:49: warning: ‘snprintf’ output may be truncated before the last format character [-Wformat-truncation=]
1074 | snprintf(loc->altitude, len, "%s", (char *)bufAlt);
| ^
utilities.c:1074:17: note: ‘snprintf’ output between 1 and 128 bytes into a destination of size 127
1074 | snprintf(loc->altitude, len, "%s", (char *)bufAlt);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
LD (gcc) [M lost.so] lost.so
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4038
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4038(a)github.com>
kamcmd tm.t_uac_start INVITE sip:1346@192.168.10.22:5080 . . 'From: <sip:1000@192.168.10.11>\r\n' 'To: <sip:1346@192.168.10.22:5080>\r\n'
error: 400 - Invalid headers
the log is :
/usr/bin/kamailio[2773]: ERROR: <core> [core/parser/msg_parser.c:260]: get_hdr_field(): no eol - bad body for <From> (hdr type: 4) [<sip:1000@192.168.10.11>\r\n]
/usr/bin/kamailio[2773]: ERROR: <core> [core/parser/msg_parser.c:339]: parse_headers(): bad header field [From: <sip:1000@192.168.10.11>\r\n]
kamailio -v
version: kamailio 5.7.0 (x86_64/linux) f1e91d
please help me, thank you
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4042
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4042(a)github.com>
route[HANDLE_INVITE] {
if (!has_totag()) {
if (!t_newtran()) {
xlog("L_ERROR", "Failed to create new transaction\n");
sl_reply_error();
exit;
}
if (!is_present_hf("X-VICIdial-Client-Id")) {
xlog("L_WARN", "INVITE received without X-VICIdial-Client-Id header\n");
t_reply("400", "Bad Request - Missing Client ID");
exit;
}
if ($rU == $null) {
xlog("L_WARN", "INVITE received with empty Request-URI username\n");
t_reply("400", "Bad Request - Missing User");
exit;
}
$var(full_client_id) = $hdr(X-VICIdial-Client-Id);
# Extract parent ID
if ($var(full_client_id) =~ "^(CID_[0-9]+)-[a-z]$") {
$var(parent_client_id) = $(var(full_client_id){re.subst,/^(CID_[0-9]+)-[a-z]$/\1/});
} else {
$var(parent_client_id) = $var(full_client_id);
}
# CPS Check and Rate Limiting
lock("cps_counter");
# Get current CPS count and max limit
$var(current_cps) = $sht(client_limits=>$var(parent_client_id));
$var(max_cps) = $sht(client_max_cps=>$var(parent_client_id));
xlog("L_INFO", "CPS Check - Client: $var(parent_client_id), Current: $var(current_cps), Max: $var(max_cps)\n");
# Check for configured limit
if($var(max_cps) == $null) {
unlock("cps_counter");
xlog("L_WARN", "No CPS limit configured for client $var(parent_client_id)\n");
t_reply("403", "Client Not Authorized");
exit;
}
# Get concurrent calls
$var(concurrent_calls) = $sht(client_calls=>$var(parent_client_id));
if($var(concurrent_calls) == $null) {
$var(concurrent_calls) = 0;
}
# Strict limit check
if($var(concurrent_calls) >= $var(max_cps)) {
unlock("cps_counter");
xlog("L_WARN", "Concurrent call limit reached for $var(parent_client_id): $var(concurrent_calls)/$var(max_cps)\n");
t_reply("503", "Concurrent Call Limit Exceeded");
exit;
}
# Allow call and increment counter
if($var(current_cps) == $null) {
$sht(client_limits=>$var(parent_client_id)) = 1;
} else {
$sht(client_limits=>$var(parent_client_id)) = $var(current_cps) + 1;
}
xlog("L_INFO", "Call allowed for $var(parent_client_id): Concurrent=$var(concurrent_calls+1)/$var(max_cps)\n");
unlock("cps_counter");
$var(user) = $rU;
# Extract parent client ID if this is a child ID
if ($var(full_client_id) =~ "^(CID_[0-9]+)-[a-z]$") {
$var(parent_client_id) = $(var(full_client_id){re.subst,/^(CID_[0-9]+)-[a-z]$/\1/});
$var(is_child) = 1;
} else {
$var(parent_client_id) = $var(full_client_id);
$var(is_child) = 0;
}
$avp(full_client_id) = $var(full_client_id);
$avp(parent_client_id) = $var(parent_client_id);
$avp(user) = $var(user);
$var(call_id) = $ci;
route(GENERATE_SERIAL_NUMBER);
# Track client calls
lock("client_counter");
# Update parent counter
$var(parent_calls) = $sht(client_calls=>$var(parent_client_id));
if($var(parent_calls) == $null) {
$var(parent_calls) = 0;
}
$sht(client_calls=>$var(parent_client_id)) = $var(parent_calls) + 1;
# If it's a child ID, update its individual counter too
if ($var(is_child) == 1) {
$var(child_calls) = $sht(client_calls=>$var(full_client_id));
if($var(child_calls) == $null) {
$var(child_calls) = 0;
}
$sht(client_calls=>$var(full_client_id)) = $var(child_calls) + 1;
}
# Store mappings
$sht(client_calls=>$var(call_id)) = $var(parent_client_id);
$sht(client_original=>$var(call_id)) = $var(full_client_id);
unlock("client_counter");
# Find destination with lowest active calls
$var(min_calls) = 999;
$var(selected_dst) = "";
lock("calls_counter");
if (ds_select_dst("4", "4")) {
$var(count) = 0; # Add counter
$var(current_calls) = $sht(active_calls=>$du);
if ($var(current_calls) == $null) {
$var(current_calls) = 0;
}
if ($var(current_calls) < $var(min_calls)) {
$var(min_calls) = $var(current_calls);
$var(selected_dst) = $du;
}
while(ds_next_dst() && $var(count) < 350) { # Set higher than 280
$var(count) = $var(count) + 1;
$var(current_calls) = $sht(active_calls=>$du);
if ($var(current_calls) == $null) {
$var(current_calls) = 0;
}
if ($var(current_calls) < $var(min_calls)) {
$var(min_calls) = $var(current_calls);
$var(selected_dst) = $du;
}
}
}
if ($var(selected_dst) != "" && $var(min_calls) < 25) {
$du = $var(selected_dst);
# Add this block - Double check the count hasn't changed
$var(current_count) = $sht(active_calls=>$du);
if ($var(current_count) >= 25) {
unlock("calls_counter");
xlog("L_ERROR", "Count increased during selection for $du to $var(current_count)\n");
t_reply("503", "Service Unavailable");
exit;
}
# Store Call-ID to destination mapping
$sht(active_calls=>$var(call_id)) = $du;
# Increment call counter
$sht(active_calls=>$du) = $var(min_calls) + 1;
unlock("calls_counter");
route(RELAY_INVITE);
} else {
unlock("calls_counter");
xlog("L_NOTICE", "Unlocked $ci");
$du = "sip:34.125.5.64:5060";
t_on_failure("FINAL_FAILURE");
if (!t_relay()) {
t_reply("500", "Internal Server Error");
}
}
} else {
route(RELAY);
}
} here is my invite rout configuration if you look at to the cps limit this is occultly the call per second limit on the client id when i set it 200 and the start calling up to 3000 per seconds so some time or for a bit of time it exceed the limit 201 or 202 then it start enforce 200 so if anyone have the best idea or suggestion regarding to this with the good ms time
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/4040
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/4040(a)github.com>
Dear Kamailio Team,
My name is Marletta, and I am a technician.
I am working on a project involving Kamailio and need some information or assistance to configure its integration with Microsoft Teams and Asterisk.
Specifically, I would like to know:
1. Is this integration feasible?
2. Which certificates should be used?
3. How can I validate the communication between Kamailio and Teams?
I have consulted the official documentation and forums but could not find comprehensive answers.
Would it be possible to receive your support or be directed to relevant resources or experts?
Thank you in advance for your time and for the work you do to support this remarkable open-source community.
I remain available to provide further details or clarifications.
Best regards / 此致
[cid:image001.png@01DB3F60.F5D74DE0]
Giuseppe Marletta
Viale Europa, 128 - 65010 SPOLTORE (PE)
Tel [+39] 085 4450011- Fax [+39] 085 4459477
PI e C.F. 01635460684
www.tecnojest.com<http://www.tecnojest.com/>
[Image result for you tube icon]<https://www.youtube.com/@tecnojestsrl>[cid:image003.png@01DB3F60.F5D74DE0]<http://www.tecnojest.com/> [cid:image004.png@01DB3F60.F5D74DE0] <https://www.linkedin.com/company/tecnojests.r.l./>
Le informazioni contenute nella presente e-mail e nei documenti eventualmente allegati possono essere confidenziali e sono comunque riservate al destinatario della stessa. La loro diffusione, distribuzione e/o copiatura da parte di terzi è proibita. I contenuti testuali sono divulgati a mero titolo informativo, gli stessi non possono essere rivendicati come impegni, ordini o commesse se non accompagnati da specifico documento firmato a norma di legge. Se avete ricevuto questa comunicazione per errore, Vi preghiamo di informare immediatamente il mittente del messaggio e di distruggere questa e-mail.
This e-mail may contain confidential and/or privileged information. If you are not the intended recipient (or have received this e-mail in error) please notify the sender immediately and destroy this e-mail. Any unauthorized copying, disclosure or distribution of the material in this e-mail is strictly forbidden.
Module: kamailio
Branch: master
Commit: 77e7494d29d891c44a49377a462b0c82e6769f1e
URL: https://github.com/kamailio/kamailio/commit/77e7494d29d891c44a49377a462b0c8…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2024-11-26T10:46:10+01:00
modules: readme files regenerated - dispatcher ... [skip ci]
---
Modified: src/modules/dispatcher/README
---
Diff: https://github.com/kamailio/kamailio/commit/77e7494d29d891c44a49377a462b0c8…
Patch: https://github.com/kamailio/kamailio/commit/77e7494d29d891c44a49377a462b0c8…
---
diff --git a/src/modules/dispatcher/README b/src/modules/dispatcher/README
index ef5c56f6df6..44439a450d7 100644
--- a/src/modules/dispatcher/README
+++ b/src/modules/dispatcher/README
@@ -1678,6 +1678,11 @@ onreply_route {
The parameters can include variables.
+ The returns 1 (cfg-true) when the oc attrbutes are udpated for a
+ destination record; -1 (cfg-false) when there is an internal error; -2
+ (cfg-false) when no destination is updated (e.g., not found or the
+ ocseq condition fails).
+
This function can be used from ANY_ROUTE.
Example 1.61. ds_oc_set_attrs() usage
Module: kamailio
Branch: master
Commit: 76f2d91dc00fa33368d4df0d5a25a2dbbbc9f8c7
URL: https://github.com/kamailio/kamailio/commit/76f2d91dc00fa33368d4df0d5a25a2d…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2024-11-26T10:39:35+01:00
dispatcher: updated docs for ds_oc_set_attrs()
---
Modified: src/modules/dispatcher/doc/dispatcher_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/76f2d91dc00fa33368d4df0d5a25a2d…
Patch: https://github.com/kamailio/kamailio/commit/76f2d91dc00fa33368d4df0d5a25a2d…
---
diff --git a/src/modules/dispatcher/doc/dispatcher_admin.xml b/src/modules/dispatcher/doc/dispatcher_admin.xml
index 5e7cb95953c..f311be2f1ff 100644
--- a/src/modules/dispatcher/doc/dispatcher_admin.xml
+++ b/src/modules/dispatcher/doc/dispatcher_admin.xml
@@ -2073,6 +2073,10 @@ onreply_route {
</listitem>
</itemizedlist>
<para>The parameters can include variables.</para>
+ <para>The returns 1 (cfg-true) when the oc attrbutes are udpated for a
+ destination record; -1 (cfg-false) when there is an internal error;
+ -2 (cfg-false) when no destination is updated (e.g., not found or the
+ ocseq condition fails).</para>
<para>
This function can be used from ANY_ROUTE.
</para>
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [ ] 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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
Various fixes for issues discovered with the Coverity static analysis tooling.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4030
-- Commit Summary --
* cdp: fixes for various issues found with coverity
-- File Changes --
M src/modules/cdp/authstatemachine.c (2)
M src/modules/cdp/common.c (8)
M src/modules/cdp/diameter_peer.c (6)
M src/modules/cdp/receiver.c (4)
M src/modules/cdp/session.c (12)
M src/modules/cdp/session.h (13)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4030.patchhttps://github.com/kamailio/kamailio/pull/4030.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4030
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4030(a)github.com>
Module: kamailio
Branch: master
Commit: 1ceeba7364b745b6ca2840e830d006a37460d10e
URL: https://github.com/kamailio/kamailio/commit/1ceeba7364b745b6ca2840e830d006a…
Author: Dragos Vingarzan <vingarzan(a)gmail.com>
Committer: Dragos Vingarzan <dragos(a)neatpath.net>
Date: 2024-11-25T13:33:40+01:00
cdp: fixed issues discovered during static code analysis
This fixes:
- 4-bytes (typically uint32_t) used for time_t storage (Y2K38)
- print of time_t with %d instead of %ld
- reuse of same loop index variable in a sub-loop
- use of random(), when kam_rand() could theoretically be better
- copy-paste mistake in sp2/sp parameter, where sp2 was meant and sp is probably NULL
---
Modified: src/modules/cdp/authstatemachine.c
Modified: src/modules/cdp/common.c
Modified: src/modules/cdp/diameter_peer.c
Modified: src/modules/cdp/receiver.c
Modified: src/modules/cdp/session.c
Modified: src/modules/cdp/session.h
---
Diff: https://github.com/kamailio/kamailio/commit/1ceeba7364b745b6ca2840e830d006a…
Patch: https://github.com/kamailio/kamailio/commit/1ceeba7364b745b6ca2840e830d006a…
- Added parameter to control the interval
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [ ] 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)
- [x] New feature (non-breaking change which adds new functionality)
- [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
This PR adds a timer that its interval can be controlled with the param `ping_interval`.
The timer pings all registered rtpengines and disables if any are found not responsive and vice versa.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4016
-- Commit Summary --
* Add timer to ping rtpengines
-- File Changes --
M src/modules/rtpengine/rtpengine.c (50)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4016.patchhttps://github.com/kamailio/kamailio/pull/4016.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4016
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4016(a)github.com>
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [X] PR should be backported to stable branches (v5.8.2, v5.8.3, v5.8.4)
- [X] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
Always use to-tag for NG message with rtpp-flags,
if presented. This change is caused by the processing logic being moved to rtpengine with rtpp-flags.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4036
-- Commit Summary --
* rtpengine: use to-tag for NG message when rtpp-flags
-- File Changes --
M src/modules/rtpengine/rtpengine.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4036.patchhttps://github.com/kamailio/kamailio/pull/4036.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4036
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4036(a)github.com>
Hi team, i got this issue when i change network connection
connect and disconnect wifi or lan
this is log
09-25 11:31:17.826 4510 4510 E kamailio: ERROR: <core> [core/tcp_main.c:3955]: handle_new_connect(): error while accepting connection(22): Invalid argument
09-25 11:31:22.019 4510 4510 I chatty : uid=0(root) kamailio identical 45884 lines
09-25 11:31:22.022 4510 4510 E kamailio: ERROR: <core> [core/tcp_main.c:3955]: handle_new_connect(): error while accepting connection(22): Invalid argument
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3983
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3983(a)github.com>
Module: kamailio
Branch: master
Commit: 0583ae7f70d3a88b21f826033052753fb02b48f8
URL: https://github.com/kamailio/kamailio/commit/0583ae7f70d3a88b21f826033052753…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2024-11-21T14:01:10+01:00
modules: readme files regenerated - sqlops ... [skip ci]
---
Modified: src/modules/sqlops/README
---
Diff: https://github.com/kamailio/kamailio/commit/0583ae7f70d3a88b21f826033052753…
Patch: https://github.com/kamailio/kamailio/commit/0583ae7f70d3a88b21f826033052753…
---
diff --git a/src/modules/sqlops/README b/src/modules/sqlops/README
index d55fab0e081..e6b91d07f53 100644
--- a/src/modules/sqlops/README
+++ b/src/modules/sqlops/README
@@ -209,10 +209,12 @@ modparam("sqlops", "log_buf_size", 4096)
3.5. connect_mode (int)
- Control how the module will connect to database. Values: 0 - connect at
- start up or fail 1 - connect at start up but continue even if
- connecting to database server fails 2 - do not connect until needed
- (this will limit the amount of idle connections)
+ Control how the module will connect to database. Values:
+ * 0 connect at start up or fail
+ * 1 connect at start up but continue even if connecting to database
+ server fails
+ * 2 do not connect until needed (this will limit the amount of idle
+ connections)
Default value is 0.
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### 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] Documentation
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4032
-- Commit Summary --
* sqlops: new connect mode documentation update
-- File Changes --
M src/modules/sqlops/doc/sqlops_admin.xml (20)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4032.patchhttps://github.com/kamailio/kamailio/pull/4032.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4032
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4032(a)github.com>
Module: kamailio
Branch: master
Commit: 378e857aa4eb17fe454b0f96fd4cd0200d31caf4
URL: https://github.com/kamailio/kamailio/commit/378e857aa4eb17fe454b0f96fd4cd02…
Author: Julien Chavanton <jchavanton(a)gmail.com>
Committer: GitHub <noreply(a)github.com>
Date: 2024-11-21T07:53:11-05:00
sqlops: new connect mode documentation update (#4032)
---
Modified: src/modules/sqlops/doc/sqlops_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/378e857aa4eb17fe454b0f96fd4cd02…
Patch: https://github.com/kamailio/kamailio/commit/378e857aa4eb17fe454b0f96fd4cd02…
---
diff --git a/src/modules/sqlops/doc/sqlops_admin.xml b/src/modules/sqlops/doc/sqlops_admin.xml
index beccbcab6dd..70cfe569253 100644
--- a/src/modules/sqlops/doc/sqlops_admin.xml
+++ b/src/modules/sqlops/doc/sqlops_admin.xml
@@ -221,10 +221,24 @@ modparam("sqlops", "log_buf_size", 4096)
<para>
Control how the module will connect to database.
Values:
- 0 - connect at start up or fail
- 1 - connect at start up but continue even if connecting to database server fails
- 2 - do not connect until needed (this will limit the amount of idle connections)
</para>
+ <itemizedlist>
+ <listitem>
+ <para>
+ <emphasis>0</emphasis> connect at start up or fail
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ <emphasis>1</emphasis> connect at start up but continue even if connecting to database server fails
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ <emphasis>2</emphasis> do not connect until needed (this will limit the amount of idle connections)
+ </para>
+ </listitem>
+ </itemizedlist>
<para>
<emphasis>
Default value is 0.
Module: kamailio
Branch: master
Commit: 7215f2554e3c2551870902fd061f20d5a4160bca
URL: https://github.com/kamailio/kamailio/commit/7215f2554e3c2551870902fd061f20d…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2024-11-20T18:31:10+01:00
modules: readme files regenerated - sqlops ... [skip ci]
---
Modified: src/modules/sqlops/README
---
Diff: https://github.com/kamailio/kamailio/commit/7215f2554e3c2551870902fd061f20d…
Patch: https://github.com/kamailio/kamailio/commit/7215f2554e3c2551870902fd061f20d…
---
diff --git a/src/modules/sqlops/README b/src/modules/sqlops/README
index e8b7b072fc6..d55fab0e081 100644
--- a/src/modules/sqlops/README
+++ b/src/modules/sqlops/README
@@ -209,9 +209,10 @@ modparam("sqlops", "log_buf_size", 4096)
3.5. connect_mode (int)
- Control if the module must stop loading when connecting to database
- server fails during start up. Values: 0 - stop loading; 1 - continue
- even if connecting to database server fails.
+ Control how the module will connect to database. Values: 0 - connect at
+ start up or fail 1 - connect at start up but continue even if
+ connecting to database server fails 2 - do not connect until needed
+ (this will limit the amount of idle connections)
Default value is 0.
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [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] New feature (non-breaking change which adds new functionality)
#### Checklist:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [x] Tested changes locally
#### Description
<!-- Describe your changes in detail -->
Very often I only need one connection from a worker process, to cache fill provisioning data to Kamailio.
This new sqlops connect mode will reduce the amount of idle connection to the database server significantly.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4027
-- Commit Summary --
* sqlops: new connect mode to limit the amount of connections
-- File Changes --
M src/modules/sqlops/doc/sqlops_admin.xml (8)
M src/modules/sqlops/sql_api.c (5)
M src/modules/sqlops/sqlops.c (17)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4027.patchhttps://github.com/kamailio/kamailio/pull/4027.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4027
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4027(a)github.com>
<!-- Kamailio Pull Request Template -->
<!--
IMPORTANT:
- for detailed contributing guidelines, read:
https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md
- pull requests must be done to master branch, unless they are backports
of fixes from master branch to a stable branch
- backports to stable branches must be done with 'git cherry-pick -x ...'
- code is contributed under BSD for core and main components (tm, sl, auth, tls)
- code is contributed GPLv2 or a compatible license for the other components
- GPL code is contributed with OpenSSL licensing exception
-->
#### Pre-Submission Checklist
<!-- Go over all points below, and after creating the PR, tick all the checkboxes that apply -->
<!-- All points should be verified, otherwise, read the CONTRIBUTING guidelines from above-->
<!-- If you're unsure about any of these, don't hesitate to ask on sr-dev mailing list -->
- [x ] Commit message has the format required by CONTRIBUTING guide
- [x ] Commits are split per component (core, individual modules, libs, utils, ...)
- [xx 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:
<!-- Go over all points below, and after creating the PR, tick the checkboxes that apply -->
- [ ] PR should be backported to stable branches
- [ ] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4031
-- Commit Summary --
* cnxcc: add fixup_free function
* corex: add fixup_free to file_read function
* crypto: add fixup_free function
* dispatcher: add fixup_free to several exported functions
* dlgs: fix some fixup_free
* enum: add fixup_free* to exported functions
* evapi: add fixup_free* to exported functions
* exec: add fixup_free* to exported functions
-- File Changes --
M src/modules/cnxcc/cnxcc_mod.c (20)
M src/modules/corex/corex_mod.c (20)
M src/modules/crypto/crypto_mod.c (18)
M src/modules/dispatcher/dispatcher.c (42)
M src/modules/dlgs/dlgs_mod.c (6)
M src/modules/enum/enum_mod.c (10)
M src/modules/evapi/evapi_mod.c (29)
M src/modules/exec/exec_mod.c (21)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4031.patchhttps://github.com/kamailio/kamailio/pull/4031.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4031
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4031(a)github.com>