<!-- 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
Add global config var
```
enable_tls_thread=1 #0 | 1 | 2
```
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3745
-- Commit Summary --
* step-1
* step-2
* step-3
-- File Changes --
M src/core/cfg.lex (2)
M src/core/cfg.y (5)
M src/core/globals.h (1)
M src/core/rthreads.h (19)
M src/main.c (1)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3745.patchhttps://github.com/kamailio/kamailio/pull/3745.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3745
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3745(a)github.com>
fixes code generation when python keywords are used
fixes formating issues of generated code
fixes crash from pv.setl reporting parameters as null instead of 'none'
made example more complete
<!-- 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
- [ ] 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/3751
-- Commit Summary --
* tools/kemi: update kemi_mock for python
-- File Changes --
M misc/tools/kemi/python_mock/kemi_mock.py (51)
M misc/tools/kemi/python_mock/test.py (12)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3751.patchhttps://github.com/kamailio/kamailio/pull/3751.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3751
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3751(a)github.com>
Module: kamailio
Branch: master
Commit: 6918c9e34b826f8d8c7fb1746851055fa35f8ece
URL: https://github.com/kamailio/kamailio/commit/6918c9e34b826f8d8c7fb1746851055…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2024-02-09T10:01:34+01:00
modules: readme files regenerated - ldap ... [skip ci]
---
Modified: src/modules/ldap/README
---
Diff: https://github.com/kamailio/kamailio/commit/6918c9e34b826f8d8c7fb1746851055…
Patch: https://github.com/kamailio/kamailio/commit/6918c9e34b826f8d8c7fb1746851055…
---
diff --git a/src/modules/ldap/README b/src/modules/ldap/README
index 921f45822f1..d69005f864e 100644
--- a/src/modules/ldap/README
+++ b/src/modules/ldap/README
@@ -31,6 +31,7 @@ Christian Schlatter
4. Parameters
4.1. config_file (string)
+ 4.2. connect_mode (int)
5. Functions
@@ -82,12 +83,13 @@ Christian Schlatter
1.6. ldap_client_bind_timeout example
1.7. Example LDAP Configuration File
1.8. config_file parameter usage
- 1.9. Example Usage of ldap_url
- 1.10. Example Usage
+ 1.9. Set connect_mode parameter
+ 1.10. Example Usage of ldap_url
1.11. Example Usage
1.12. Example Usage
1.13. Example Usage
1.14. Example Usage
+ 1.15. Example Usage
2.1. Example code fragment to load LDAP module API
2.2. Example LDAP module API function call
@@ -114,6 +116,7 @@ Chapter 1. Admin Guide
4. Parameters
4.1. config_file (string)
+ 4.2. connect_mode (int)
5. Functions
@@ -418,6 +421,7 @@ ldap_client_bind_timeout = 500
4. Parameters
4.1. config_file (string)
+ 4.2. connect_mode (int)
4.1. config_file (string)
@@ -428,6 +432,17 @@ ldap_client_bind_timeout = 500
Example 1.8. config_file parameter usage
modparam("ldap", "config_file", "/usr/local/etc/kamailio/ldap.ini")
+4.2. connect_mode (int)
+
+ Control if the module must stop loading when connecting to server fails
+ during start up. Values: 0 - stop loading; 1 - continue even if
+ connecting to database server fails..
+
+ Default value: 0
+
+ Example 1.9. Set connect_mode parameter
+modparam("ldap", "connect_mode", 1)
+
5. Functions
5.1. ldap_search(ldap_url)
@@ -458,7 +473,7 @@ modparam("ldap", "config_file", "/usr/local/etc/kamailio/ldap.ini")
Kamailio pseudo variables and AVPs included in ldap_url do get
substituted with their value.
- Example 1.9. Example Usage of ldap_url
+ Example 1.10. Example Usage of ldap_url
Search with LDAP session named sipaccounts, base
ou=sip,dc=example,dc=com, one level deep using search filter
@@ -492,7 +507,7 @@ ldap://ldap_1/dc=example,dc=com?
This function can be used from REQUEST_ROUTE, FAILURE_ROUTE,
BRANCH_ROUTE, and ONREPLY_ROUTE.
- Example 1.10. Example Usage
+ Example 1.11. Example Usage
...
# ldap search
if (!ldap_search("ldap://sipaccounts/ou=sip,dc=example,dc=com??one?(cn=$rU)"))
@@ -570,7 +585,7 @@ ldap_result("telephoneNumber/$avp(s:tel_number)");
This function can be used from REQUEST_ROUTE, FAILURE_ROUTE,
BRANCH_ROUTE, and ONREPLY_ROUTE.
- Example 1.11. Example Usage
+ Example 1.12. Example Usage
...
# ldap_search call
@@ -639,7 +654,7 @@ ldap_result("SIPIdentitySIPURI/$avp(i:10)", "/^[^@]+@(.+)$/\1/");
This function can be used from REQUEST_ROUTE, FAILURE_ROUTE,
BRANCH_ROUTE, and ONREPLY_ROUTE.
- Example 1.12. Example Usage
+ Example 1.13. Example Usage
...
# ldap_search call
...
@@ -688,7 +703,7 @@ if (!ldap_result_check("sn/$ru", "/^sip:([^@]).*$/\1/"))
This function can be used from REQUEST_ROUTE, FAILURE_ROUTE,
BRANCH_ROUTE, and ONREPLY_ROUTE.
- Example 1.13. Example Usage
+ Example 1.14. Example Usage
...
# ldap_search call
...
@@ -745,7 +760,7 @@ if (ldap_result_next())
This function can be used from REQUEST_ROUTE, FAILURE_ROUTE,
BRANCH_ROUTE, and ONREPLY_ROUTE.
- Example 1.14. Example Usage
+ Example 1.15. Example Usage
...
if (!ldap_filter_url_encode("cn=$avp(s:name)", "$avp(s:name_esc)"))
{
<!-- 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 aims to give the option for ldap to allow kamailio to start-up even if server is not reachable/available when starting.
It follows similar logic to https://github.com/kamailio/kamailio/commit/2df43b992b1e6932974bd0c928c0465… for SQL.
A new config parameter for `ldap` module is defined, `connect_mode`, that defaults to 0, preserving the old behavior. If it's set `1`, `kamailio` will not exit and allow `ldap` to try and reconnect when necessary.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3747
-- Commit Summary --
* ldap: option to start even when connecting to server fails
* ldap/docs: Add connect_mode docs
-- File Changes --
M src/modules/ldap/doc/ldap_admin.xml (18)
M src/modules/ldap/ldap_mod.c (15)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3747.patchhttps://github.com/kamailio/kamailio/pull/3747.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3747
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3747(a)github.com>
### Description
Via sngrep I can see call was terminated via BYE message but was not terminated on RTPengine hosts. The `rtpengine-ctl list sessions all` show calls that was terminated properly via BYE message.
In the kamailio config used `rtpengine_manage` function.
I do not have any idea how to troubleshoot this.
Could you suggest next step and I will add more details for this issue
### Additional Information
* **Kamailio Version**
Used 5.6.4 with small customization.
```
version: kamailio 5.6.4 (aarch64/linux) 06ad45
flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT-NOSMP, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: 06ad45
compiled on 16:37:11 Aug 7 2023 with gcc 8.5.0
```
* **Operating System**:
CentOS 8 Stream, aarch64
```
[root@sbc-a0 ~]# uname -a
Linux sbc-a0.nga911.com 4.19.113-300.el8.aarch64 #1 SMP Fri May 12 11:47:07 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux
[root@sbc-a0 ~]# cat /etc/os-release
NAME="CentOS Stream"
VERSION="8"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="8"
PLATFORM_ID="platform:el8"
PRETTY_NAME="CentOS Stream 8"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:8"
HOME_URL="https://centos.org/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux 8"
REDHAT_SUPPORT_PRODUCT_VERSION="CentOS Stream"
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3749
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3749(a)github.com>