A user reached me to add a way to add the custom labels into the statsd
module, so a better way to report metrics to the observability platform.
I keep the same old functions and add a new parameter to the statsd
modules to both interfaces cfg and kemi.
The full documentation can be found here:
https://docs.datadoghq.com/developers/dogstatsd/datagram_shell/?tab=metrics
So, each function can be used like this: ``` statsd_set("fooo", 1,
"inbound"); statsd_gauge("NotFound", "+1", "outbound,carrierFoo");
statsd_gauge("AuthFailed", "+1", "carrier=foo,priority=10"); ```
Signed-off-by: Eloy Coto <eloy.coto(a)acalustra.com>
Tested-by: Alex Antonevych <alex.antonevych(a)replicant.ai>
<!-- Kamailio Pull Request Template -->
#### 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
- [ ] 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:
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
I'm waiting until first review on function arguments to create the docs
changes.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3529
-- Commit Summary --
* statsd: Add labels to metrics.
-- File Changes --
M src/modules/statsd/lib_statsd.c (102)
M src/modules/statsd/lib_statsd.h (10)
M src/modules/statsd/statsd.c (129)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3529.patchhttps://github.com/kamailio/kamailio/pull/3529.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3529
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3529(a)github.com>
Module: kamailio
Branch: master
Commit: bff7b033d817c6ed63f3de63dfa7f1b45ead2ff8
URL: https://github.com/kamailio/kamailio/commit/bff7b033d817c6ed63f3de63dfa7f1b…
Author: Eloy Coto <eloy.coto(a)acalustra.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-09-12T10:41:54+02:00
statsd: Add labels to metrics.
A user reached me to add a way to add the custom labels into the statsd
module, so a better way to report metrics to the observability platform.
I keep the same old functions and add a new parameter to the statsd
modules to both interfaces cfg and kemi.
The full documentation can be found here:
https://docs.datadoghq.com/developers/dogstatsd/datagram_shell/?tab=metrics
So, each function can be used like this: ``` statsd_set("fooo", 1,
"inbound"); statsd_gauge("NotFound", "+1", "outbound,carrierFoo");
statsd_gauge("AuthFailed", "+1", "carrier:foo,priority:10"); ```
Signed-off-by: Eloy Coto <eloy.coto(a)acalustra.com>
Tested-by: Alex Antonevych <alex.antonevych(a)replicant.ai>
Signed-off-by: Eloy Coto <eloy.coto(a)acalustra.com>
---
Added: src/modules/statsd/doc/statsd.txt
Modified: src/modules/statsd/lib_statsd.c
Modified: src/modules/statsd/lib_statsd.h
Modified: src/modules/statsd/statsd.c
---
Diff: https://github.com/kamailio/kamailio/commit/bff7b033d817c6ed63f3de63dfa7f1b…
Patch: https://github.com/kamailio/kamailio/commit/bff7b033d817c6ed63f3de63dfa7f1b…
<!-- 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)
- [ ] 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/3561
-- Commit Summary --
* core: Add option for increasing socket send buffer size
-- File Changes --
M src/core/cfg.lex (2)
M src/core/cfg.y (7)
M src/core/globals.h (1)
M src/core/udp_server.c (104)
M src/core/udp_server.h (1)
M src/main.c (28)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3561.patchhttps://github.com/kamailio/kamailio/pull/3561.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3561
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3561(a)github.com>
- the function call will suceed even if the prefix being inserted already exists in the tree
<!-- 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
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
PDT module initialization currently fails if you try to insert a prefix multiple times. We use kamailio as a loadbalancer and have had this change out in the field for quite some time now as our database often contains a prefix multiple times (associated with the same domain). Would it make sense to change the behavior in this case ?
Thanks,
Vladimir.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3485
-- Commit Summary --
* pdt: make add_to_tree() idempotent
-- File Changes --
M src/modules/pdt/pdtree.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3485.patchhttps://github.com/kamailio/kamailio/pull/3485.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3485
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3485(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
- [ ] 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
- [x] Related to issue #2946
#### Description
migrate lcr to pcre2
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3545
-- Commit Summary --
* lcr: pcre2 migration
-- File Changes --
M src/modules/lcr/Makefile (12)
M src/modules/lcr/hash.c (24)
M src/modules/lcr/hash.h (10)
M src/modules/lcr/lcr_mod.c (187)
M src/modules/lcr/lcr_mod.h (8)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3545.patchhttps://github.com/kamailio/kamailio/pull/3545.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3545
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3545(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
- [ ] 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
- [x] Related to issue #2946
#### Description
migrate dialplan to pcre2
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3543
-- Commit Summary --
* regex: migration to pcre2
* dialplan: migrate to pcre2
-- File Changes --
M src/modules/dialplan/Makefile (11)
M src/modules/dialplan/dialplan.c (5)
M src/modules/dialplan/dialplan.h (21)
M src/modules/dialplan/dp_db.c (103)
M src/modules/dialplan/dp_repl.c (56)
M src/modules/regex/Makefile (11)
M src/modules/regex/regex_mod.c (240)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3543.patchhttps://github.com/kamailio/kamailio/pull/3543.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3543
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3543(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
- [ ] 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/3551
-- Commit Summary --
* smsops: fixes as per static code analyzer report
-- File Changes --
M src/modules/smsops/smsops_impl.c (12)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3551.patchhttps://github.com/kamailio/kamailio/pull/3551.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3551
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3551(a)github.com>
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3559
-- Commit Summary --
* misc/ typos
-- File Changes --
M misc/examples/exec/exec_s5b.cfg (2)
M misc/examples/ims/icscf/README.md (2)
M misc/examples/ims/icscf/kamailio.cfg (4)
M misc/examples/ims/pcscf/README.md (2)
M misc/examples/ims/pcscf/kamailio.cfg (4)
M misc/examples/ims/pcscf/pcscf.cfg.sample (2)
M misc/examples/ims/pcscf/route/mo.cfg (2)
M misc/examples/ims/pcscf/sems/sems.conf (2)
M misc/examples/ims/scscf/README.md (2)
M misc/examples/ims/scscf/kamailio.cfg (6)
M misc/examples/ims/scscf/scscf.cfg.sample (2)
M misc/examples/kamailio/acc-mysql.cfg (2)
M misc/examples/kamailio/acc.cfg (2)
M misc/examples/kamailio/ctd.sh (4)
M misc/examples/kamailio/fork.cfg (2)
M misc/examples/kamailio/nathelper.cfg (2)
M misc/examples/kamailio/replicate.cfg (2)
M misc/examples/kamailio/web_im/README (2)
M misc/examples/kemi/kamailio-basic-kemi-ruby.rb (4)
M misc/examples/kemi/kamailio-basic-kemi.cfg (2)
M misc/examples/mixed/acc.cfg (2)
M misc/examples/mixed/kamailio-minimal-anycast.cfg (4)
M misc/examples/mixed/kamailio-minimal-proxy.cfg (4)
M misc/examples/mixed/nathelper.cfg (2)
M misc/examples/mixed/onr.cfg (4)
M misc/examples/mixed/replicate.cfg (2)
M misc/examples/mixed/sip-router.cfg.m4 (14)
M misc/examples/mixed/voicemail.cfg (2)
M misc/examples/obsoleted/imgw.cfg (4)
M misc/examples/obsoleted/secondary.cfg (2)
M misc/examples/obsoleted/sms.cfg (2)
M misc/examples/obsoleted/smsgw.cfg (2)
M misc/examples/obsoleted/test.cfg (2)
M misc/examples/obsoleted/tmtest.cfg (4)
M misc/examples/pkg/kamailio-basic.cfg (2)
M misc/examples/pkg/kamailio-oob.cfg (10)
M misc/examples/pkg/sip-router-basic.cfg (2)
M misc/examples/pkg/sip-router-oob.cfg (32)
M misc/examples/presence/full-no-failover.cfg (6)
M misc/examples/presence/no-db.cfg (4)
M misc/examples/scripts/ctd.sh (4)
M misc/examples/scripts/serresponse (8)
M misc/examples/scripts/web_im/README (2)
M misc/examples/webrtc/kamailio-default-websocket.cfg (4)
M misc/scripts/dbtext/ser_dbtext.sh (2)
M misc/scripts/mysql/sip-router_mysql.sh (4)
M misc/scripts/postgres/ser_postgres.sh (4)
M misc/scripts/serconf.sh (18)
M misc/tools/kemi/python_mock/README.md (4)
M misc/tools/kemi/python_mock/kemi_mock.py (2)
M misc/tools/pike_top/pike_top.c (12)
M misc/tools/profile/launch.sh (2)
M misc/tools/sipgrep/sipgrep (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3559.patchhttps://github.com/kamailio/kamailio/pull/3559.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3559
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3559(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.
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
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
### Troubleshooting
#### Reproduction
<!--
If the issue can be reproduced, describe how it can be done.
-->
#### Debugging Data
<!--
If you got a core dump, use gdb to extract troubleshooting data - full backtrace,
local variables and the list of the code at the issue location.
gdb /path/to/kamailio /path/to/corefile
bt full
info locals
list
If you are familiar with gdb, feel free to attach more of what you consider to
be relevant.
-->
```
(paste your debugging data here)
```
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
in $mb from xlog shows path header as Path: <sip:xx.xxx.xx.xx
suffix header part is not getting appended creating malformed packet
```
#### SIP Traffic
<!--
REGISTER sip:xx.xxxx.xx.xxxx.xx SIP/2.0
From: "+914962207321"<sip:+91111111111@xx.xxxx.xx.xxxx.xx>;tag=786f78-a7504e5-13c4-590628-2b088c7a-590628
To: "+914962207321"<sip:+911111111111@xx.xxxx.xx.xxxx.xx>
P-Access-Network-Info: ADSL;ue-ip=10.117.x.xxx;ue-port=5060;transport=udp;nat=no
Call-ID: 790930-a7504e5-13c4-590628-aede83e-590628
CSeq: 2 REGISTER
Via: SIP/2.0/UDP 127.0.0.1;branch=z9hG4bK1af.96eb50fa0163b0488a72566a78846e9f.0
Via: SIP/2.0/UDP 10.117.x.xxx:5060;rport=5060;branch=z9hG4bK-590628-5bc00f10-3253a7b8
Max-Forwards: 69
Supported: 100rel,replaces,timer
Contact: <sip:+911111111111@10.117.x.xxx:5060>
Expires: 3600
Authorization: Digest username="+911111111111",realm="xx.xxxx.xx.xxxx.xx",nonce="eqvZt86/W0Pt/0bN+SZKLw==",uri="sip:xx.xxxx.xx.xxxx.xx",response="3213963a37bd59cf4b16a5d4080da415",algorithm=MD5,cnonce="5bc00f1a",qop=auth,nc=00000001
Content-Length: 0
Path: <sip:127.0.1.1
-->
```
(paste your sip traffic here)
```
### Possible Solutions
<!--
reverted to pre "use snprintf() instead of sprintf()" merge for patch.c it works fine
-->
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.8.0-dev1 (x86_64/linux) 99eb1b
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_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: 99eb1b
compiled on 10:38:25 Sep 8 2023 with gcc 11.4.0
```
* **Operating System**:
<!--
Details about the operating system, the type: Linux (e.g.,: Debian 8.4, Ubuntu 16.04, CentOS 7.1, ...), MacOS, xBSD, Solaris, ...;
Kernel details (output of `lsb_release -a` and `uname -a`)
-->
```
Ubuntu 22.04
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3560
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3560(a)github.com>
Module: kamailio
Branch: master
Commit: 06b1a35c727155974eb3c10971eaac2606b430ea
URL: https://github.com/kamailio/kamailio/commit/06b1a35c727155974eb3c10971eaac2…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2023-09-11T14:46:24+02:00
modules: readme files regenerated - textops ... [skip ci]
---
Modified: src/modules/textops/README
---
Diff: https://github.com/kamailio/kamailio/commit/06b1a35c727155974eb3c10971eaac2…
Patch: https://github.com/kamailio/kamailio/commit/06b1a35c727155974eb3c10971eaac2…
---
diff --git a/src/modules/textops/README b/src/modules/textops/README
index fa73e19ea32..3979dfdf185 100644
--- a/src/modules/textops/README
+++ b/src/modules/textops/README
@@ -104,6 +104,7 @@ Ovidiu Sas
4.59. get_body_part_raw(content_type, opv)
4.60. remove_body_part(content_type)
4.61. regex_substring(itext, regexp, mindex, mcount, dpv)
+ 4.62. via_param_rm(name, idx)
2. Developer Guide
@@ -175,6 +176,7 @@ Ovidiu Sas
1.60. get_body_part_raw usage
1.61. remove_body_part usage
1.62. _regex_substring usage
+ 1.63. via_param_rm usage
Chapter 1. Admin Guide
@@ -252,6 +254,7 @@ Chapter 1. Admin Guide
4.59. get_body_part_raw(content_type, opv)
4.60. remove_body_part(content_type)
4.61. regex_substring(itext, regexp, mindex, mcount, dpv)
+ 4.62. via_param_rm(name, idx)
1. Overview
@@ -355,6 +358,7 @@ From: medabeda
4.59. get_body_part_raw(content_type, opv)
4.60. remove_body_part(content_type)
4.61. regex_substring(itext, regexp, mindex, mcount, dpv)
+ 4.62. via_param_rm(name, idx)
4.1. search(re)
@@ -1621,6 +1625,25 @@ msg_apply_changes();
----
...
+4.62. via_param_rm(name, idx)
+
+ Remove parameter matching by name from Via body at the specified index.
+ The index starts from 0 (first Via body). Negative index counts from
+ the end of the list for Via header bodies (-1 is the last Via body).
+
+ Meaning of the parameters is as follows:
+ * name - name of the parameter
+ * idx - index of the Via body.
+
+ The parameters can also be variables.
+
+ This function can be used from ANY_ROUTE.
+
+ Example 1.63. via_param_rm usage
+...
+via_param_rm("p1", "0");
+...
+
Chapter 2. Developer Guide
Table of Contents
Module: kamailio
Branch: master
Commit: fb2be3855c5971d312bad52ffc3c4cb99aeb63d8
URL: https://github.com/kamailio/kamailio/commit/fb2be3855c5971d312bad52ffc3c4cb…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-09-11T14:32:58+02:00
textops: docs for via_param_rm(...)
---
Modified: src/modules/textops/doc/textops_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/fb2be3855c5971d312bad52ffc3c4cb…
Patch: https://github.com/kamailio/kamailio/commit/fb2be3855c5971d312bad52ffc3c4cb…
---
diff --git a/src/modules/textops/doc/textops_admin.xml b/src/modules/textops/doc/textops_admin.xml
index 4af7a8bb805..9c121e51c70 100644
--- a/src/modules/textops/doc/textops_admin.xml
+++ b/src/modules/textops/doc/textops_admin.xml
@@ -2303,5 +2303,40 @@ msg_apply_changes();
</example>
</section>
+ <section id="textops.f.via_param_rm">
+ <title>
+ <function moreinfo="none">via_param_rm(name, idx)</function>
+ </title>
+ <para>
+ Remove parameter matching by name from Via body at the specified index.
+ The index starts from 0 (first Via body). Negative index counts from the
+ end of the list for Via header bodies (-1 is the last Via body).
+ </para>
+ <para>Meaning of the parameters is as follows:</para>
+ <itemizedlist>
+ <listitem>
+ <para><emphasis>name</emphasis> - name of the parameter
+ </para>
+ </listitem>
+ <listitem>
+ <para><emphasis>idx</emphasis> - index of the Via body.
+ </para>
+ </listitem>
+ </itemizedlist>
+ <para>The parameters can also be variables.</para>
+ <para>
+ This function can be used from ANY_ROUTE.
+ </para>
+
+ <example>
+ <title><function>via_param_rm</function> usage</title>
+ <programlisting format="linespecific">
+...
+via_param_rm("p1", "0");
+...
+</programlisting>
+ </example>
+ </section>
+
</section>
</chapter>
I have been trying to replace the SDP connection IP with some other IP using module `mangler` (`Kamailio v5.7.0`). But It seem like its malfunctioning
```
if ($rs == 200) {
xlog("L_INFO", "200 $rm Detected");
sdp_mangle_ip("192.168.0.132/24","8.8.8.8");
}
```
and results I received are like this below
```
v=0 o=- 107808851201429815 2 IN IP4 192.168.0.132
s=- t=0 0
m=audio 24794 RTP/AVP 0 8 101
c=IN IP4 192.168.0.1328.8.8.8
a=mid:audio
a=sendrecv
a=rtpmap:0 PCMU/8000
a=rtpmap:8 PCMA/8000
a=rtpmap:101 telephone-event/8000
a=ssrc:822878909 cname:0ZfUBhl8cMeHaeVl
a=nortpproxy:yes
```
I first ask the same question in mailing list `sr-users(a)lists.kamailio.org` but after waiting for some time (>24hrs) i cant see my question on on forum, that why I open a ticket here.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3554
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3554(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.
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
I have some buggy devices that get upset by SRTP payloads in SDP that are marked inactive. As a hack, I was attempting to delete them when going to certain endpoints that are known to be problematic:
```
if (is_method("INVITE"))
sdp_remove_transport("RTP/SAVP");
```
### Troubleshooting
Per below, there is some garbage that looks like a port number inserted before `m=audio` that seems to have come from rtpproxy rewriting for NAT traversal. Removing the NAT traversal rewrite logic entirely fixes the problem, but also makes NAT traversal not happen. Changing the order of `sdp_remove_transport()` with respect to `rtpproxy_manage()` does not change anything.
#### Reproduction
This was seen with a Grandstream DP752 phone talking to SEMS, but applying the code snippet above and getting a packet trace will be enough. The original SDP had the RTP/SAVP transport first, followed by an RTP/AVP one. For the problem to occur, NAT SDP rewriting has to occur using rtpproxy.
#### Debugging Data
Here is the payload after sfp_remove_transport() removed the RTP/SAVP section:
```
v=0
o=15176190619 8000 8000 IN IP4 X.X.X.X
s=SIP Call
c=IN IP4 X.X.X.X
t=0 0
46108m=audio 55082 RTP/AVP 125 9 0 8 2 18 97 101
a=sendrecv
a=rtpmap:125 opus/48000/2
a=fmtp:125 maxplaybackrate=16000
a=ptime:20
a=rtpmap:9 G722/8000
a=rtpmap:0 PCMU/8000
a=rtpmap:8 PCMA/8000
a=rtpmap:2 G726-32/8000
a=rtpmap:18 G729/8000
a=fmtp:18 annexb=no
a=rtpmap:97 iLBC/8000
a=fmtp:97 mode=20
a=rtpmap:101 telephone-event/8000
a=fmtp:101 0-16,32-36,54
a=nortpproxy:yes
```
There is a garbage 46108 (I think what would have been the port number for SAVP data) at the beginning of the m=audio line. It seems that the rewrite by rtpproxy is still inserting this port number even though the transport in question has been removed.
#### Log Messages
None relevant.
#### SIP Traffic
(See above)
### Possible Solutions
None.
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
$ kamailio -v
version: kamailio 5.6.2 (x86_64/freebsd) 54a9c1
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, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, select, kqueue.
id: 54a9c1
compiled on 10:44:33 Jul 20 2023 with cc FreeBSD clang version 13.0.0 (git@github.com:llvm/llvm-project.git llvmorg-13.0.0-0-gd7b669b3a303)
```
* **Operating System**:
FreeBSD 13.2
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3555
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3555(a)github.com>
Module: kamailio
Branch: master
Commit: b6979b503ffe65314526a8a2824f50b3f46b7a22
URL: https://github.com/kamailio/kamailio/commit/b6979b503ffe65314526a8a2824f50b…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: The Root <root(a)kamailio.org>
Date: 2023-09-05T12:46:08+02:00
modules: readme files regenerated - tm ... [skip ci]
---
Modified: src/modules/tm/README
---
Diff: https://github.com/kamailio/kamailio/commit/b6979b503ffe65314526a8a2824f50b…
Patch: https://github.com/kamailio/kamailio/commit/b6979b503ffe65314526a8a2824f50b…
---
diff --git a/src/modules/tm/README b/src/modules/tm/README
index 0b44b81813c..19f4a9a3781 100644
--- a/src/modules/tm/README
+++ b/src/modules/tm/README
@@ -1864,11 +1864,12 @@ modparam("tm", "failover_reply_codes", "code=403;code=488;class=5")
4.1. t_relay([host, port])
- Relay a message statefully either to the destination indicated in the
- current URI (if called without any parameters) or to the specified host
- and port. In the later case (host and port specified) the protocol used
- is the same protocol on which the message was received. It creates a
- transaction.
+ Relay a SIP request statefully either to the destination indicated in
+ the current URI (if called without any parameters) or to the specified
+ host and port. In the later case (host and port specified) the protocol
+ used is the same protocol on which the message was received. It creates
+ the SIP transaction, if none was created before (e.g., with
+ t_newtran()).
t_relay() is the stateful version for forward() while t_relay(host,
port) is similar to forward(host, port).
#### 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
- [ ] 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:
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
I am not very familiar with the build/packaging process but I'm hoping we can get `jwt` module packaged for debian since the dependencies (`libjwt`) already ship with most recent debian distributions. Any feedback would be appreciated here, thanks.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3556
-- Commit Summary --
* jwt: build jwt module for debian distributions
-- File Changes --
M pkg/kamailio/deb/bionic/rules (1)
M pkg/kamailio/deb/bookworm/rules (1)
M pkg/kamailio/deb/bullseye/rules (1)
M pkg/kamailio/deb/buster/rules (1)
M pkg/kamailio/deb/debian/rules (1)
M pkg/kamailio/deb/focal/rules (1)
M pkg/kamailio/deb/jammy/rules (1)
M pkg/kamailio/deb/jessie/rules (1)
M pkg/kamailio/deb/precise/rules (1)
M pkg/kamailio/deb/sid/rules (1)
M pkg/kamailio/deb/stretch/rules (1)
M pkg/kamailio/deb/trusty/rules (1)
M pkg/kamailio/deb/wheezy/rules (1)
M pkg/kamailio/deb/xenial/rules (1)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3556.patchhttps://github.com/kamailio/kamailio/pull/3556.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3556
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3556(a)github.com>
Halo Experts,
So sorry to disturb you for the kamailio start failure issue I faced recently when doing patching from version 5.5.4 to 5.5.6. Really appreciated if you could give me some assistance on it ! And attached is the start log. Thanks.
Best regards,
Louis LIU
Senior Software Engineer (Cloud Platform && Audio/Video Intercom) M +86 18575580920 8F-A CES Building,No.3099
Global Offer Innovation E bin.liu2(a)se.com<mailto:bin.liu2@se.com> Keyuan South Road, Nanshan District
Home & Distribution Division, Energy Management Shenzhen, China
[cid:image001.png@01D9CAE5.A472D190]
Download mySchneider app<https://www.se.com/ww/en/work/support/myschneider-app/> 24/7 support Mobile catalog. Acces to expert help.
Internal
<!-- 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)
- [ ] 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 -->
This PR fixes the conversion from UCS2 to UTF-8 and viceversa. Previously the code was not handling emojis and certain characters.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3546
-- Commit Summary --
* smsops: Fix conversion from UCS-2 to UTF-8 and viceversa
-- File Changes --
M src/modules/smsops/smsops_impl.c (219)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3546.patchhttps://github.com/kamailio/kamailio/pull/3546.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3546
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3546(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:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list:
* http://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot 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
I loaded module "db_postgres.so". I expected kamilio to start. It didn't
### Troubleshooting
#### Reproduction
<!--
If the issue can be reproduced, describe how it can be done.
-->
#### config file
```
#!KAMAILIO
#!define WITH_POSTGRESSQL
#!define WITH_AUTH
#!define WITH_USRLOCDB
#!define WITH_PRESENCE
#!define WITH_MSGREBUILD
#!define WITH_NAT
#!define WITH_NATSIPPING
..
.
..
#!ifdef WITH_POSTGRESSQL
loadmodule "db_postgres.so"
#!endif
```
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
Jul 22 10:31:16 opensips /usr/sbin/kamailio[8975]: ERROR: <core> [db.c:204]: db_bind_mod(): Module db_postgresql not found. Missing loadmodule?
Jul 22 10:31:16 opensips /usr/sbin/kamailio[8975]: ERROR: usrloc [usrloc_mod.c:361]: mod_init(): failed to bind database module
Jul 22 10:31:16 opensips /usr/sbin/kamailio[8975]: ERROR: <core> [core/sr_module.c:942]: init_mod(): Error while initializing module usrloc (/usr/lib/x86_64-linux-gnu/kamailio/modules/usrloc.so)
```
output from ls of modules directory:
```
acc.so call_obj.so dialog.so ipops.so msrp.so presence_conference.so qos.so sipcapture.so statsd.so uid_avp_db.so
acc_diameter.so carrierroute.so dialplan.so jsonrpcs.so mtree.so presence_dialoginfo.so ratelimit.so sipdump.so stun.so uid_domain.so
alias_db.so cfg_db.so dispatcher.so keepalive.so nat_traversal.so presence_mwi.so regex.so sipjson.so tcpops.so uid_gflags.so
app_jsdt.so cfg_rpc.so diversion.so kemix.so nathelper.so presence_profile.so registrar.so sipt.so textops.so uid_uri_db.so
app_sqlang.so cfgt.so dmq.so kex.so nosip.so presence_reginfo.so rls.so siptrace.so textopsx.so uri_db.so
async.so cfgutils.so dmq_usrloc.so lcr.so outbound.so presence_xml.so rr.so siputils.so timer.so userblocklist.so
auth.so corex.so domain.so log_custom.so p_usrloc.so pua.so rtimer.so sl.so tm.so usrloc.so
auth_db.so counters.so domainpolicy.so lrkproxy.so path.so pua_bla.so rtjson.so sms.so tmrec.so xcap_client.so
auth_diameter.so ctl.so drouting.so mangler.so pdb.so pua_dialoginfo.so rtpengine.so smsops.so tmx.so xcap_server.so
auth_xkeys.so db2_ops.so enum.so matrix.so pdt.so pua_reginfo.so rtpproxy.so speeddial.so topoh.so xhttp.so
avp.so db_cluster.so evrexec.so maxfwd.so permissions.so pua_rpc.so sanity.so sqlops.so topos.so xhttp_prom.so
avpops.so db_flatstore.so exec.so mediaproxy.so pike.so pua_usrloc.so sca.so ss7ops.so tsilo.so xhttp_rpc.so
benchmark.so db_postgres.so group.so mohqueue.so pipelimit.so pua_xmpp.so sdpops.so sst.so uac.so xlog.so
blst.so db_text.so htable.so mqueue.so prefix_route.so pv.so seas.so statistics.so uac_redirect.so xprint.so
call_control.so debugger.so imc.so msilo.so presence.so pv_headers.so secfilter.so statsc.so uid_auth_db.so
```
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix.
-->
tried renaming db_postgres.so to db_postgresql.so, but that didn't work, so I reversed it
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.5.4 (x86_64/linux)
flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED
ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: unknown
compiled with gcc 10.2.1
```
* **Operating System**:
<!--
Details about the operating system, the type: Linux (e.g.,: Debian 8.4, Ubuntu 16.04, CentOS 7.1, ...), MacOS, xBSD, Solaris, ...;
Kernel details (output of `lsb_release -a` and `uname -a`)
-->
```
Distributor ID: Debian
Description: Debian GNU/Linux 11 (bullseye)
Release: 11
Codename: bullseye
Linux opensips 5.10.0-16-amd64 #1 SMP Debian 5.10.127-1 (2022-06-30) x86_64 GNU/Linux
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3196
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3196(a)github.com>
I noticed that sometimes the trunk registration just fails after TTL expiry and the kamailio tries to resolve an A record for next_hop=<sip:reg.sip-trunk.telekom.de:5060;transport=tcp> where actually no A record exists resulting in an unregistered trunk for one register period. After the one failed period the SRV will result in correct CNAME resolving again and the trunk will be registered for one full hour (baes on TTL expiry of SRV record).
```
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: uac [uac_reg.c:1138]: uac_reg_send(): using custom socket tcp:10.124.2.2:5060 to send request
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/socket_info.c:702]: grep_sock_info(): checking if host==us: 10==10 && [10.124.2.2] == [10.124.2.2]
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/socket_info.c:709]: grep_sock_info(): checking if port 5060 (advertise 5060) matches port 5060
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: tm [h_table.c:580]: tm_xdata_replace(): replace existing list in backup xd from new xd
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: tm [h_table.c:413]: build_cell(): created new cell 0x7fc7766aa400
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: tm [h_table.c:568]: tm_xdata_replace(): restore X/AVP msg context from backup data
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: tm [uac.c:468]: t_uac_prepare(): next_hop=<sip:reg.sip-trunk.telekom.de:5060;transport=tcp>
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/dns_cache.c:534]: _dns_hash_find(): (_sip._tcp.reg.sip-trunk.telekom.de(34), 33), h=666
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/resolve.c:999]: get_record(): skipping 0 NS (p=0x560cf3e135dc, end=0x560cf3e135dc)
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/resolve.c:1012]: get_record(): parsing 0 ARs (p=0x560cf3e135dc, end=0x560cf3e135dc)
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/dns_cache.c:1640]: dns_get_related(): (0x7fc7766f9ab8 (_sip._tcp.reg.sip-trunk.telekom.de, 33), 33, *(nil)) (0)
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/dns_cache.c:746]: dns_cache_add_unsafe(): adding _sip._tcp.reg.sip-trunk.telekom.de(34) 33 (flags=0) at 666
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/dns_cache.c:534]: _dns_hash_find(): (reg.sip-trunk.telekom.de(24), 1), h=400
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/resolve.c:784]: get_record(): lookup(reg.sip-trunk.telekom.de, 1) failed
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/dns_cache.c:768]: dns_cache_mk_bad_entry(): (reg.sip-trunk.telekom.de, 1, 60, 1)
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: <core> [core/dns_cache.c:708]: dns_cache_add(): adding reg.sip-trunk.telekom.de(24) 1 (flags=1) at 400
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: ERROR: <core> [core/resolve.c:1730]: sip_hostport2su(): could not resolve hostname: "reg.sip-trunk.telekom.de"
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: ERROR: tm [ut.h:309]: uri2dst2(): failed to resolve "reg.sip-trunk.telekom.de"
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: ERROR: tm [uac.c:491]: t_uac_prepare(): no socket found
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: DEBUG: tm [h_table.c:132]: free_cell_helper(): freeing transaction 0x7fc7766aa400 from uac.c:632
Aug 21 16:55:57 seelze-iptel1 kamailio[4116711]: ERROR: uac [uac_reg.c:1178]: uac_reg_send(): failed to send request for [siptrunk]
```
At this point it should not try to resolve reg.sip-trunk.telekom.de but instead one of the SRV CNAMES.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3547
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3547(a)github.com>
### Description
I want debug memory usage by TLS module.
To do this used command
```
make -C src/modules/tls CFLAGS="-DTLS_MALLOC_DBG=1"
```
And this output
```
[root@sbc-stage-a0 kamailio]# make -C src/modules/tls CFLAGS="-DTLS_MALLOC_DBG=1"
make: Entering directory '/root/bak/kamailio/src/modules/tls'
CC (gcc) [M tls.so] tls_domain.o
CC (gcc) [M tls.so] tls_rand.o
CC (gcc) [M tls.so] tls_map.o
CC (gcc) [M tls.so] tls_cfg.o
CC (gcc) [M tls.so] tls_mod.o
CC (gcc) [M tls.so] tls_util.o
CC (gcc) [M tls.so] tls_config.o
CC (gcc) [M tls.so] tls_server.o
CC (gcc) [M tls.so] tls_select.o
CC (gcc) [M tls.so] tls_dump_vf.o
CC (gcc) [M tls.so] tls_init.o
tls_init.c:137:2: warning: #warning "tls module compiled with malloc debugging info (extra overhead)" [-Wcpp]
#warning "tls module compiled with malloc debugging info (extra overhead)"
^~~~~~~
tls_init.c: In function ‘ser_malloc’:
tls_init.c:218:5: warning: implicit declaration of function ‘_shm_malloc’; did you mean ‘shm_malloc’? [-Wimplicit-function-declaration]
p=_shm_malloc(size+s, file, "via ser_malloc", line);
^~~~~~~~~~~
shm_malloc
tls_init.c:218:4: warning: assignment to ‘void *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
p=_shm_malloc(size+s, file, "via ser_malloc", line);
^
tls_init.c:224:39: error: invalid application of ‘sizeof’ to incomplete type ‘struct qm_frag’
((struct qm_frag*)((char*)p-sizeof(struct qm_frag)))->func=
^~~~~~
tls_init.c: In function ‘ser_realloc’:
tls_init.c:253:5: warning: implicit declaration of function ‘_shm_realloc’; did you mean ‘shm_realloc’? [-Wimplicit-function-declaration]
p=_shm_realloc(ptr, size+s, file, "via ser_realloc", line);
^~~~~~~~~~~~
shm_realloc
tls_init.c:253:4: warning: assignment to ‘void *’ from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
p=_shm_realloc(ptr, size+s, file, "via ser_realloc", line);
^
tls_init.c:259:39: error: invalid application of ‘sizeof’ to incomplete type ‘struct qm_frag’
((struct qm_frag*)((char*)p-sizeof(struct qm_frag)))->func=
^~~~~~
tls_init.c: In function ‘tls_pre_init’:
tls_init.c:684:7: warning: implicit declaration of function ‘CRYPTO_set_mem_ex_functions’; did you mean ‘CRYPTO_set_mem_functions’? [-Wimplicit-function-declaration]
if (!CRYPTO_set_mem_ex_functions(ser_malloc, ser_realloc, ser_free)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~
CRYPTO_set_mem_functions
tls_init.c:684:60: error: ‘ser_free’ undeclared (first use in this function); did you mean ‘sk_free’?
if (!CRYPTO_set_mem_ex_functions(ser_malloc, ser_realloc, ser_free)) {
^~~~~~~~
sk_free
tls_init.c:684:60: note: each undeclared identifier is reported only once for each function it appears in
make: *** [../../Makefile.rules:100: tls_init.o] Error 1
make: Leaving directory '/root/bak/kamailio/src/modules/tls'
```
### Additional Information
kamailio 5.6.4 with some customisation
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3532
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3532(a)github.com>
Hi All,
I need to test ISUP to SIP-I conversion at Kamailio Softswitch .this SIP-I conversion need to verify SIpp-SIP end point.
**my architecure flow as below as:-**
isup sipi-i/sip-t
MTG 200 Trunk Gatway -----------> Kamailio Softswitch ----------------> SIPP.
How to make it happend in via Kamailio Softswitch .and what are the configurations need to make in Kamailio Softswitch
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3542
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3542(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
- [ ] 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
- [x] Related to issue #2946
#### Description
migrate modules to pcre2
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3535
-- Commit Summary --
* regex: migration to pcre2
-- File Changes --
M src/modules/regex/Makefile (11)
M src/modules/regex/regex_mod.c (121)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3535.patchhttps://github.com/kamailio/kamailio/pull/3535.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3535
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3535(a)github.com>
I removed the word atomic before transaction, since it implies that there are in addition non-atomic ways to create a transaction.
Per https://lists.kamailio.org/mailman3/hyperkitty/list/sr-users@lists.kamailio… t_relay() creates a transaction.
- typos
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3538
-- Commit Summary --
* tm: clarify which functions can create a transaction
-- File Changes --
M src/modules/tm/config.h (2)
M src/modules/tm/dlg.c (2)
M src/modules/tm/doc/event_routes.xml (2)
M src/modules/tm/doc/functions.xml (24)
M src/modules/tm/doc/params.xml (21)
M src/modules/tm/doc/tm.xml (4)
M src/modules/tm/h_table.c (4)
M src/modules/tm/h_table.h (2)
M src/modules/tm/select.c (4)
M src/modules/tm/sip_msg.c (2)
M src/modules/tm/sip_msg.h (2)
M src/modules/tm/t_fifo.c (4)
M src/modules/tm/t_funcs.c (2)
M src/modules/tm/t_funcs.h (2)
M src/modules/tm/t_fwd.c (16)
M src/modules/tm/t_hooks.c (4)
M src/modules/tm/t_hooks.h (6)
M src/modules/tm/t_lookup.c (10)
M src/modules/tm/t_msgbuilder.c (8)
M src/modules/tm/t_msgbuilder.h (2)
M src/modules/tm/t_reply.c (24)
M src/modules/tm/t_stats.h (10)
M src/modules/tm/t_suspend.c (2)
M src/modules/tm/timer.c (2)
M src/modules/tm/timer.h (2)
M src/modules/tm/tm.c (8)
M src/modules/tm/uac.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3538.patchhttps://github.com/kamailio/kamailio/pull/3538.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3538
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3538(a)github.com>
目前进行一同呼叫后,在acc表中可以看到Invite、ack及bye的记录详情并带有时间,acc_cdr表中只有start_time及end_time,是对应invite和bye的。
现在我想在acc_cdr中增加ack的时间我该如何去做呢?有没有大佬能解答一下?万分感谢!!!
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3541
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3541(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
- [ ] 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 -->
Added new module param to enable retention of call stats for existing destinations when reloading the list from file or database.
Also updated to retain stats for other destinations when adding/removing individual ones via RPC.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3534
-- Commit Summary --
* dispatcher: add option to retain existing latency stats when reloading destinations
-- File Changes --
M src/modules/dispatcher/dispatch.c (63)
M src/modules/dispatcher/dispatch.h (1)
M src/modules/dispatcher/dispatcher.c (2)
M src/modules/dispatcher/doc/dispatcher_admin.xml (32)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3534.patchhttps://github.com/kamailio/kamailio/pull/3534.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3534
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3534(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
- [ ] 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 -->
Primary goal is to update doc, for Rx_AAR() function, to mention is used only for replies.
I've also changed some LM_DBG to LM_WARN but I can revert that on request (don't want to overflow noone with warnings)
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3521
-- Commit Summary --
* ims_qos: doc update and warn logs
-- File Changes --
M src/modules/ims_qos/doc/ims_qos_admin.xml (6)
M src/modules/ims_qos/ims_qos_mod.c (16)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3521.patchhttps://github.com/kamailio/kamailio/pull/3521.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3521
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3521(a)github.com>
In the XAVPVIAFIELDS block there is the string written to the wrong variable.
The string is stored in _ksr_xavp_via_params.s but the lenght is stored in _ksr_xavp_via_fields.len. This breaks XAVPVIAPARAMS and XAVPVIAFIELDS.
<!-- 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
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
`via_use_xavp_fields("1");` and probably `via_add_xavp_params("1");` are broken because of a typo in parsing the xavp names. The values of xavp_via_params global parameter and xavp_via_fields global parameter are not stored correctly.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3540
-- Commit Summary --
* core: cfg.y typo in parsing XAVPVIAFIELDS
-- File Changes --
M src/core/cfg.y (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3540.patchhttps://github.com/kamailio/kamailio/pull/3540.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3540
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3540(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.
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
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
Migrating from Kamailio 5.6.4 IMS to Kamailio 5.7.1 IMS; using Open5GS CN, Amarisoft CallBox eNB, and Huawei UE to implement VoLTE infrastructure for lab testing. VoLTE works in Kamailio 5.6.4 IMS with IPSEC and non-IPSEC UE's; but Kamailio 5.7.1 IMS not working yet with IPSEC UE.
Client and server port settings in P-CSCF kamailio.cfg:
...
modparam("ims_ipsec_pcscf", "ipsec_client_port", 5100)
modparam("ims_ipsec_pcscf", "ipsec_server_port", 6100)
...
Debug log shows ports are exchanged (server port used as client port and vice-versa):
...
Jul 18 12:15:30 corsa03 p-cscf[58287]: DEBUG: ims_ipsec_pcscf [cmd.c:460]: create_ipsec_tunnel(): Creating security associations: Local IP: 10.169.138.17 **port_pc: 6100 port_ps: 5100**; UE IP: 10.46.0.6; port_uc 31409 port_us 31894; spi_pc 4096, spi_ps 4097, spi_uc 48734730, spi_us 96335982, alg hmac-md5-96, ealg null
...
### Troubleshooting
spi_add function definition and declaration differ:
From src/modules/ims_ipsec_pcscf/spi_list.h:
int spi_add(spi_list_t *list, uint32_t spi_cid, uint32_t spi_sid,
**uint16_t cport, uint16_t sport**);
But in spi_list.c:
int spi_add(spi_list_t *list, uint32_t spi_cid, uint32_t spi_sid,
**uint16_t sport, uint16_t cport**)
#### Reproduction
<!--
If the issue can be reproduced, describe how it can be done.
-->
#### Debugging Data
<!--
If you got a core dump, use gdb to extract troubleshooting data - full backtrace,
local variables and the list of the code at the issue location.
gdb /path/to/kamailio /path/to/corefile
bt full
info locals
list
If you are familiar with gdb, feel free to attach more of what you consider to
be relevant.
-->
```
(paste your debugging data here)
```
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your log messages here)
```
#### SIP Traffic
<!--
If the issue is exposed by processing specific SIP messages, grab them with ngrep or save in a pcap file, then add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your sip traffic here)
```
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix.
-->
Applied following patch:
--- a/src/modules/ims_ipsec_pcscf/spi_list.c 2023-06-28 09:40:45.000000000 +0200
+++ b/src/modules/ims_ipsec_pcscf/spi_list.c 2023-07-18 16:12:11.166476076 +0200
@@ -51,7 +51,7 @@
}
int spi_add(spi_list_t *list, uint32_t spi_cid, uint32_t spi_sid,
- uint16_t sport, uint16_t cport)
+ uint16_t cport, uint16_t sport)
{
if(!list) {
return 1;
Log now shows expected client and server port:
...
Jul 18 17:45:22 corsa03 p-cscf[82076]: DEBUG: ims_ipsec_pcscf [cmd.c:460]: create_ipsec_tunnel(): Creating security associations: Local IP: 10.169.138.17 **port_pc: 5100 port_ps: 6100**; UE IP: 10.46.0.6; port_uc 31411 port_us 31896; spi_pc 4096, spi_ps 4097, spi_uc 48734732, spi_us 96335984, alg hmac-md5-96, ealg null
...
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
version: kamailio 5.7.1 (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_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 16:22:26 Jul 18 2023 with x86_64-pc-linux-gnu-gcc 13.1.1
```
* **Operating System**:
<!--
Details about the operating system, the type: Linux (e.g.,: Debian 8.4, Ubuntu 16.04, CentOS 7.1, ...), MacOS, xBSD, Solaris, ...;
Kernel details (output of `lsb_release -a` and `uname -a`)
-->
```
Linux corsa03 6.4.2-gentoo-x86_64 #2 SMP PREEMPT_DYNAMIC Tue Jul 11 14:35:45 CEST 2023 x86_64 AMD EPYC 7513 32-Core Processor AuthenticAMD GNU/Linux
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3513
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3513(a)github.com>
Module: kamailio
Branch: master
Commit: 20634fccc59c82a35b45a3da0b8e6a0e64acb37e
URL: https://github.com/kamailio/kamailio/commit/20634fccc59c82a35b45a3da0b8e6a0…
Author: Stefan Mititelu <stefan-cristian.mititelu(a)1and1.ro>
Committer: Henning Westerholt <hw(a)gilawa.com>
Date: 2023-08-14T17:08:35+02:00
ims_qos: doc and logs update
---
Modified: src/modules/ims_qos/doc/ims_qos_admin.xml
Modified: src/modules/ims_qos/ims_qos_mod.c
---
Diff: https://github.com/kamailio/kamailio/commit/20634fccc59c82a35b45a3da0b8e6a0…
Patch: https://github.com/kamailio/kamailio/commit/20634fccc59c82a35b45a3da0b8e6a0…
---
diff --git a/src/modules/ims_qos/doc/ims_qos_admin.xml b/src/modules/ims_qos/doc/ims_qos_admin.xml
index b8a36c7b136..7755a963abd 100644
--- a/src/modules/ims_qos/doc/ims_qos_admin.xml
+++ b/src/modules/ims_qos/doc/ims_qos_admin.xml
@@ -29,7 +29,7 @@
<itemizedlist>
<listitem>
- <para>Dialog</para>
+ <para>IMS Dialog</para>
</listitem>
<listitem>
@@ -625,8 +625,8 @@ route[REG_AAR_REPLY]
</listitem>
</itemizedlist>
- <para>This function can be used from REQUEST_ROUTE or
- ONREPLY_ROUTE.</para>
+ <para>This function can be used only from ONREPLY_ROUTE, only for replies to INVITE, UPDATE, PRACK.
+ The reply and request must have SDP, for this function to work properly.</para>
<para>p.s. this is executed asynchronously. See example on how to
retrieve return value</para>
diff --git a/src/modules/ims_qos/ims_qos_mod.c b/src/modules/ims_qos/ims_qos_mod.c
index d2502020450..9ff174f03eb 100644
--- a/src/modules/ims_qos/ims_qos_mod.c
+++ b/src/modules/ims_qos/ims_qos_mod.c
@@ -205,8 +205,8 @@ static int pv_t_copy_msg(struct sip_msg *src, struct sip_msg *dst)
}
-static cmd_export_t cmds[] = {{"Rx_AAR", (cmd_function)w_rx_aar, 4, fixup_aar,
- 0, REQUEST_ROUTE | ONREPLY_ROUTE},
+static cmd_export_t cmds[] = {
+ {"Rx_AAR", (cmd_function)w_rx_aar, 4, fixup_aar, 0, ONREPLY_ROUTE},
{"Rx_AAR_Register", (cmd_function)w_rx_aar_register, 2,
fixup_aar_register, 0, REQUEST_ROUTE},
{0, 0, 0, 0, 0, 0}};
@@ -801,7 +801,7 @@ static int w_rx_aar(
}
if(t->uas.status >= 200) {
- LM_DBG("transaction sent out a final response already - %d\n",
+ LM_WARN("transaction sent out a final response already - %d\n",
t->uas.status);
return result;
}
@@ -864,13 +864,13 @@ static int w_rx_aar(
|| memcmp(t->method.s, "UPDATE", 6) == 0))) {
if(cscf_get_content_length(msg) == 0
|| cscf_get_content_length(orig_sip_request_msg) == 0) {
- LM_DBG("No SDP offer answer -> therefore we can not do Rx AAR");
+ LM_WARN("No SDP offer answer -> therefore we can not do Rx AAR");
//goto aarna; //AAR na if we don't have offer/answer pair
return result;
}
} else {
- LM_DBG("Message is not response to INVITE, PRACK or UPDATE -> "
- "therefore we do not Rx AAR");
+ LM_WARN("Message is not response to INVITE, PRACK or UPDATE -> "
+ "therefore we do not Rx AAR");
return result;
}
@@ -1162,7 +1162,7 @@ static int w_rx_aar(
int ret = create_new_callsessiondata(&callid, &ftag, &ttag, &identifier,
identifier_type, &ip, ip_version, &rx_authdata_p);
if(!ret) {
- LM_DBG("Unable to create new media session data parcel\n");
+ LM_ERR("Unable to create new media session data parcel\n");
goto error;
}
Module: kamailio
Branch: master
Commit: 2fd5d881563126f7425730f798afcbfc2feeac65
URL: https://github.com/kamailio/kamailio/commit/2fd5d881563126f7425730f798afcbf…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2023-08-12T13:46:24+02:00
modules: readme files regenerated - rabbitmq ... [skip ci]
---
Modified: src/modules/rabbitmq/README
Modified: src/modules/xhttp/README
---
Diff: https://github.com/kamailio/kamailio/commit/2fd5d881563126f7425730f798afcbf…
Patch: https://github.com/kamailio/kamailio/commit/2fd5d881563126f7425730f798afcbf…
---
diff --git a/src/modules/rabbitmq/README b/src/modules/rabbitmq/README
index 942afffa83f..889512b6a48 100644
--- a/src/modules/rabbitmq/README
+++ b/src/modules/rabbitmq/README
@@ -176,7 +176,7 @@ modparam("rabbitmq", "direct_reply_to", 1)
* content_type - the content_type of the messagebody.
* messagebody - the messagebody to be published.
- This function can be used from REQUEST_ROUTE.
+ This function can be used from any route.
Example 1.5. rabbitmq_publish usage
rabbitmq_publish("exchange", "routing_key", "application/json", "$avp(json_reque
diff --git a/src/modules/xhttp/README b/src/modules/xhttp/README
index 9a59c839aa1..ec009bacafb 100644
--- a/src/modules/xhttp/README
+++ b/src/modules/xhttp/README
@@ -83,8 +83,8 @@ Chapter 1. Admin Guide
SIP parser can easily handle HTTP requests just by adding a fake Via
header.
- The <module>xmlrpc</module> module uses the same concept. The xHTTP
- module offers a generic way of handling the HTTP protocol, by calling
+ The xmlrpc module uses the same concept. The xHTTP module offers a
+ generic way of handling the HTTP protocol, by calling
event_route[xhttp:request] in your config. You can check the HTTP URL
via the config variable $hu. Note that use of $ru will raise errors
since the structure of an HTTP URL is not compatible with that of a SIP
@@ -222,7 +222,10 @@ event_route[xhttp:request] {
6.1. xhttp:request
- The event route is executed when a new HTTP request is received.
+ The event route is executed when a new HTTP request is received. Most
+ of the variables related to a SIP request can be used inside this event
+ route to get HTTP request attributes (e.g., $si - source IP, $hdr(X) -
+ body of header X, $rm - request method, $rb - request body, ...).
...
tcp_accept_no_cl=yes
...
Enabling rabbitmq_publish function on any route so we can send events on reply reception.
Also disabling peer verification on the TLS handshake if no CA file is configured.
<!-- 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, ...)
- [ ] 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)
- [ ] 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/3537
-- Commit Summary --
* rabbitmq: Enabling rabbitmq_publish for any route and disabling peer
-- File Changes --
M src/modules/rabbitmq/doc/rabbitmq_admin.xml (2)
M src/modules/rabbitmq/rabbitmq.c (8)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3537.patchhttps://github.com/kamailio/kamailio/pull/3537.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3537
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3537(a)github.com>
Module: kamailio
Branch: master
Commit: 170067e633ef32ea7b17adad46fdfe2c85fadda0
URL: https://github.com/kamailio/kamailio/commit/170067e633ef32ea7b17adad46fdfe2…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-08-12T13:31:15+02:00
xhttp: note about available variables during http request processing
---
Modified: src/modules/xhttp/doc/xhttp_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/170067e633ef32ea7b17adad46fdfe2…
Patch: https://github.com/kamailio/kamailio/commit/170067e633ef32ea7b17adad46fdfe2…
---
diff --git a/src/modules/xhttp/doc/xhttp_admin.xml b/src/modules/xhttp/doc/xhttp_admin.xml
index f0e6d3fdeb8..1c7d29a051a 100644
--- a/src/modules/xhttp/doc/xhttp_admin.xml
+++ b/src/modules/xhttp/doc/xhttp_admin.xml
@@ -22,7 +22,7 @@
Via header.
</para>
<para>
- The <module>xmlrpc</module> module uses the same concept.
+ The xmlrpc module uses the same concept.
The xHTTP module offers a generic way of handling the <acronym>HTTP</acronym>
protocol, by calling <emphasis>event_route[xhttp:request]</emphasis>
in your config. You can check the HTTP URL via the config variable
@@ -228,7 +228,10 @@ event_route[xhttp:request] {
<function moreinfo="none">xhttp:request</function>
</title>
<para>
- The event route is executed when a new HTTP request is received.
+ The event route is executed when a new HTTP request is received. Most
+ of the variables related to a SIP request can be used inside this event
+ route to get HTTP request attributes (e.g., $si - source IP, $hdr(X)
+ - body of header X, $rm - request method, $rb - request body, ...).
</para>
<programlisting format="linespecific">
...
#### 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
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3497
-- Commit Summary --
* sipcapture: use memcpy instead of strncpy for coherence
* rtpproxy: use memcpy instead of strcpy for coherence
-- File Changes --
M src/modules/rtpproxy/rtpproxy.c (40)
M src/modules/sipcapture/sipcapture.c (16)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3497.patchhttps://github.com/kamailio/kamailio/pull/3497.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3497
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3497(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.
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
<!--
Explain what you did, what you expected to happen, and what actually happened.
-->
### Troubleshooting
#### Reproduction
<!--
If the issue can be reproduced, describe how it can be done.
-->
#### Debugging Data
<!--
If you got a core dump, use gdb to extract troubleshooting data - full backtrace,
local variables and the list of the code at the issue location.
gdb /path/to/kamailio /path/to/corefile
bt full
info locals
list
If you are familiar with gdb, feel free to attach more of what you consider to
be relevant.
-->
```
(paste your debugging data here)
```
#### Log Messages
<!--
Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your log messages here)
```
#### SIP Traffic
<!--
If the issue is exposed by processing specific SIP messages, grab them with ngrep or save in a pcap file, then add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site).
-->
```
(paste your sip traffic here)
```
### Possible Solutions
<!--
If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix.
-->
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
```
(paste your output here)
```
* **Operating System**:
<!--
Details about the operating system, the type: Linux (e.g.,: Debian 8.4, Ubuntu 16.04, CentOS 7.1, ...), MacOS, xBSD, Solaris, ...;
Kernel details (output of `lsb_release -a` and `uname -a`)
-->
```
(paste your output here)
```
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3533
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3533(a)github.com>
Hello, I have a question for [1] config function.
Can I extend it to work with INTs too? e.g. if pv not NULL and INT flag, return 1. This would help, because one won't need to know the type of the pvar beforehand, for using this function.
Or is it meant to be used only with string pv only?
[1] https://www.kamailio.org/docs/modules/devel/modules/pv.html#pv.f.not_empty
Thank you,
Stefan
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3520
-- Commit Summary --
* docs/ : typos
* etc/ typos
-- File Changes --
M INSTALL (2)
M doc/docbook/catalog.xml (2)
M doc/docbook/dep.xsl (4)
M doc/docbook/entities.xml (10)
M doc/docbook/html.chunked.xsl (2)
M doc/doxygen/main.dox (2)
M doc/man/kamailio.cfg.5 (2)
M doc/misc/HISTORY (10)
M doc/misc/NEWS (58)
M doc/misc/TODO (10)
M doc/misc/cvs-commit-rules.txt (6)
M doc/stylesheets/dbschema_k/catalog.xml (2)
M doc/stylesheets/dbschema_k/xsl/db_berkeley.xsl (4)
M doc/stylesheets/dbschema_k/xsl/db_redis.xsl (4)
M doc/stylesheets/dbschema_k/xsl/dbtext.xsl (4)
M doc/stylesheets/dbschema_k/xsl/mysql.xsl (2)
M doc/stylesheets/dbschema_k/xsl/pi_framework_mod.xsl (2)
M doc/tutorials/presence/cfg/ps.cfg (2)
M doc/tutorials/presence/install.xml (2)
M doc/tutorials/presence/intro.xml (4)
M doc/tutorials/serdev/routing_engine.xml (2)
M doc/tutorials/seruser/otherapps.xml (2)
M etc/kamailio.cfg (4)
M etc/sip-router.cfg (16)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3520.patchhttps://github.com/kamailio/kamailio/pull/3520.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3520
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3520(a)github.com>
Module: kamailio
Branch: master
Commit: 6e9335ff2245596b74bc1c749d1e7805bf186f02
URL: https://github.com/kamailio/kamailio/commit/6e9335ff2245596b74bc1c749d1e780…
Author: �������������������� �������������������������������� <git-dpa(a)aegee.org>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-08-07T15:32:56+02:00
etc/ typos
---
Modified: etc/kamailio.cfg
Modified: etc/sip-router.cfg
---
Diff: https://github.com/kamailio/kamailio/commit/6e9335ff2245596b74bc1c749d1e780…
Patch: https://github.com/kamailio/kamailio/commit/6e9335ff2245596b74bc1c749d1e780…
---
diff --git a/etc/kamailio.cfg b/etc/kamailio.cfg
index 68493736d41..715c289af1e 100644
--- a/etc/kamailio.cfg
+++ b/etc/kamailio.cfg
@@ -227,7 +227,7 @@ enable_sctp=no
####### Custom Parameters #########
-/* These parameters can be modified runtime via RPC interface
+/* These parameters can be modified at runtime via RPC interface
* - see the documentation of 'cfg_rpc' module.
*
* Format: group.id = value 'desc' description
@@ -389,7 +389,7 @@ modparam("registrar", "path_mode", 0)
modparam("acc", "early_media", 0)
modparam("acc", "report_ack", 0)
modparam("acc", "report_cancels", 0)
-/* by default ww do not adjust the direct of the sequential requests.
+/* by default we do not adjust the direct of the sequential requests.
* if you enable this parameter, be sure the enable "append_fromtag"
* in "rr" module */
modparam("acc", "detect_direction", 0)
diff --git a/etc/sip-router.cfg b/etc/sip-router.cfg
index 0390ef7ec57..8669ea63d79 100644
--- a/etc/sip-router.cfg
+++ b/etc/sip-router.cfg
@@ -1,7 +1,7 @@
#
# $Id$
#
-# Example configuration file (simpler then ser-oob.cfg, but more
+# Example configuration file (simpler than ser-oob.cfg, but more
# complex then ser-basic.cfg).
#
# First start SER sample config script with:
@@ -73,7 +73,7 @@ rev_dns=no # (cmd. line: -R)
#group=ser
#disable_core=yes #disables core dumping
#open_fd_limit=1024 # sets the open file descriptors limit
-#mhomed=yes # usefull for multihomed hosts, small performance penalty
+#mhomed=yes # useful for multihomed hosts, small performance penalty
#disable_tcp=yes
#tcp_accept_aliases=yes # accepts the tcp alias via option (see NEWS)
sip_warning=yes
@@ -172,7 +172,7 @@ modparam("ctl", "binrpc", "tcp:127.0.0.1:2046")
# failed transactions (=negative responses) should be logged to
modparam("acc_db", "failed_transactions", 1)
-# comment the next line if you dont want to have accounting to DB
+# comment the next line if you don't want to have accounting to DB
modparam("acc_db", "log_flag", "FLAG_ACC")
# -- tm params --
@@ -197,7 +197,7 @@ modparam("tls", "private_key", "ser-selfsigned.key")
# -- xmlrpc params --
-# using a sub-route from the module is a lot safer then relying on the
+# using a sub-route from the module is a lot safer than relying on the
# request method to distinguish HTTP from SIP
modparam("xmlrpc", "route", "RPC");
@@ -358,8 +358,8 @@ route[RR]
# particularly good if upstream and downstream entities
# use different transport protocol
- # if the inital INVITE got the ACC flag store this in
- # an RR AVP cookie. this is more for demonstration purpose
+ # if the initial INVITE got the ACC flag store this in
+ # an RR AVP cookie. This is more for demonstration purpose
if (isflagset(FLAG_ACC)) {
$account = "yes";
setavpflag($account, "dialog_cookie");
@@ -377,7 +377,7 @@ route[DOMAIN]
# check if the callee is at a local domain
lookup_domain("$td", "@ruri.host");
- # we dont know the domain of the caller and also not
+ # we don't know the domain of the caller and also not
# the domain of the callee -> somone uses our proxy as
# a relay
if (strempty($t.did) && strempty($f.did)) {
@@ -528,7 +528,7 @@ route[INBOUND]
if (lookup_contacts("location")) {
append_hf("P-hint: usrloc applied\r\n");
- # we set the TM module timers according to the prefences
+ # we set the TM module timers according to the preferences
# of the callee (avoid too long ringing of his phones)
# Note1: timer values have to be in ms now!
# Note2: this makes even more sense if you switch to a voicemail
Hello,
(Adding sr-dev to CC)
This looks indeed a bit strange. Do you get any error messages in the log? In which process you are freeing the memory, one of the worker processes or the RPC process?
You could also try to use another memory manager to see if you get better performance. There is a command line parameter to choose one during startup.
Cheers,
Henning
--
Henning Westerholt - https://skalatan.de/blog/
Kamailio services - https://gilawa.com<https://gilawa.com/>
From: Chaigneau, Nicolas <nicolas.chaigneau(a)capgemini.com>
Sent: Wednesday, January 18, 2023 6:49 PM
To: Kamailio (SER) - Users Mailing List <sr-users(a)lists.kamailio.org>
Subject: [SR-Users] Performances issue when freeing shared memory in custom module (Kamailio 5.5.2)
Hello,
I'm encountering performance issues with Kamailio (5.5.2).
I'm using a custom Kamailio module that loads routing data in memory, using Kamailio shared memory.
This routing data is very large. It can be fully reloaded through a Kamailio RPC command (which is done once each day).
When reloading, two sets of data are maintained, one "loading" and another "current" (the latter being used to handle SIP requests).
When loading of the new data is finished, it is swapped to "current". Then, memory of the old (now unused) data is freed.
I've noticed that when Kamailio is freeing the old data, there is a very significant performance impact on SIP requests.
This is surprising to me, because the SIP requests do not use this old data.
This is not a CPU issue, idle CPU% is at about 99% at that moment.
I'm using the following functions :
- shm_mallocxz
- shm_free
From what I understand, shm_free is actually "qm_shm_free" defined in "src\core\mem\q_malloc.c" (the default shared memory manager being "qm").
I've noticed that there is also a variant shm_free_unsafe ("qm_free"), which does not perform locking.
I'm wondering if the lock could be the cause of my performances issues ?
(But I'm not sure how this could be possible, because although the SIP requests need to access the shared memory allocated, they do not use directly the functions from the share memory manager.)
If the performances issues are causes by the lock, could I use the unsafe version "safely" ? (considering that it is guaranteed that the old data cannot be used by anyone else)
Thanks for your help.
Regards,
Nicolas.
This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.
**Description**
We are using ims_charging module and Freediameter as diameter server for billing process....recently we faced one issue...For Initial **INVITEs** kamailio sending **CCR request (INITIAL_REQUEST)** to **diameter server** but if we receive Immediate **CANCLE** for the **INVITE**, kamailio sending **CCR request ( TERMINATE_REQUEST )** without waiting for the **CCR request (INITIAL_REQUEST) response** from the **Diameter server.**..
**Expected behavior**
Kamailio has to send **CCR request ( TERMINATE_REQUEST )** after getting the **response** from the **INITIAL_REQUEST.**
**Additional Info**
kamailio server : 4.2
platform : linux
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2143
#### 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
- [x] Related to issue #3502
#### Description
From https://www.openssl.org/docs/man3.0/man7/migration_guide.html
> Use of low-level digest functions such as SHA1_Init(3) have been informally
> discouraged from use for a long time. Applications should instead use the
> high level EVP APIs EVP_DigestInit_ex(3), EVP_DigestUpdate(3) and
> EVP_DigestFinal_ex(3), or the quick one-shot EVP_Q_digest(3).
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3518
-- Commit Summary --
* crypto: SHA1_Init deprecated at openssl 3.0
-- File Changes --
M src/modules/crypto/crypto_uuid.c (19)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3518.patchhttps://github.com/kamailio/kamailio/pull/3518.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3518
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3518(a)github.com>
Hello everyone,
I'm trying to authenticate kamailio users using freeradius but I get the following error message when I try to start kamailio:
ERROR: <core> [core/cfg.y:3762]: yyparse(): cfg. parser: failed to find command radius_www_authorize (params 1).
Thanks for your help.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3525
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/issues/3525(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
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
<!-- Describe your changes in detail -->
The module has been broken in d00ceda2c04. The code to replace strcpy() did not terminate the target sting with \0.
Discovered by our voiptests rig:
https://github.com/sippy/voiptests/actions/workflows/kamailio.yml
Results in a broken SDP (note garbage around o= attribute:
```
1681 00:00:08.271/GLOBAL/alice_ua: RECEIVED message from [::1]:5060:
1682 SIP/2.0 183 Session Progress
1683 Via: SIP/2.0/UDP [::1]:5061;received=::1;rport=5061;branch=z9hG4bK20a1169c6781e4bfe26554e84d3b4925
1684 Record-Route: <sip:[::1];ftag=l5v4RNY4iYqP.*!0LQUA.MLVq8RX'm!q;lr>
1685 From: "Alice Smith" <sip:alice_2_ipv6@[::1]>;tag=l5v4RNY4iYqP.*!0LQUA.MLVq8RX'm!q
1686 To: <sip:bob_2@[::1]>;tag=c77c4b2c8430cf6c18566bc3d9212ed3
1687 Call-ID: O%CciL!Yf0-SQn]G46v4J(SPddRr<MtH(a))<0.ev}(]q.\%X[>
1688 CSeq: 200 INVITE
1689 Content-Type: application/sdp
1690 Content-Length: 296
1691
1692 v=0
1693 o=- 577651195072 577651195072 IN IP6 ::1\x86zU
1694 s=BOBSDP Session
1695 c=IN IP6 ::1\x86zU
1696 t=0 0
1697 m=image 13424 udptl t38
1698 a=T38FaxVersion:0
1699 a=T38MaxBitRate:14400
1700 a=T38FaxRateManagement:transferredTCF
1701 a=T38FaxMaxBuffer:262
1702 a=T38FaxMaxDatagram:176
1703 a=T38FaxUdpEC:t38UDPRedundancy
1704 a=nortpproxy:yes
1740 00:00:08.322/GLOBAL/alice_ua: RECEIVED message from 127.0.0.1:5060:
1741 SIP/2.0 183 Session Progress
1742 v: SIP/2.0/UDP 127.0.0.1:5061;received=127.0.0.1;rport=5061;branch=z9hG4bK8e357573055f35a768af0c9be78c836d
1743 Record-Route: <sip:127.0.0.1;ftag=q3~s'YH'`.Krs%LQKdFWrWnWgwPHJHnf;lr>
1744 f: "Alice Smith" <sip:alice_7_ipv4@127.0.0.1>;tag=q3~s'YH'`.Krs%LQKdFWrWnWgwPHJHnf
1745 t: <sip:bob_7@127.0.0.1>;tag=8ee80e5205ef358c3cb868fb5e5614f8
1746 i: m6!(QAFQlv}vTlycDH[rpMjW}mVV1phG@z/c{KI:L*n'[`%15
1747 CSeq: 200 INVITE
1748 c: application/sdp
1749 l: 328
1750
1751 v=0
1752 o=- 577651195078 577651195078 IN IP4 127.0.0.1z\xcfpzG\xd1\xcb`\xef=\x86zU
1753 s=BOBSDP Session
1754 c=IN IP4 127.0.0.1z\xcfpzG\xd1\xcb`\xef=\x86zU
1755 t=0 0
1756 m=image 13696 udptl t38
1757 a=T38FaxVersion:0
1758 a=T38MaxBitRate:14400
1759 a=T38FaxRateManagement:transferredTCF
1760 a=T38FaxMaxBuffer:262
1761 a=T38FaxMaxDatagram:176
1762 a=T38FaxUdpEC:t38UDPRedundancy
1763 a=nortpproxy:yes
```
Looks like there were multiple changes of the same nature. All of them probably needs to be re-evaluated to make sure this pattern is not propagated.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3524
-- Commit Summary --
* rtpproxy: make sure we null terminate the copy of the string.
-- File Changes --
M src/modules/rtpproxy/rtpproxy.c (3)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3524.patchhttps://github.com/kamailio/kamailio/pull/3524.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3524
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3524(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
- [ ] 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 -->
CDR avp() mos values gets trimmed at the first larger mos value received.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3517
-- Commit Summary --
* rtpengine: increase buf size to print mos llint range
-- File Changes --
M src/modules/rtpengine/rtpengine.c (32)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3517.patchhttps://github.com/kamailio/kamailio/pull/3517.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3517
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3517(a)github.com>
Module: kamailio
Branch: master
Commit: 8b1139851d9f88add1fd7da0dbc0b2b56933cdd3
URL: https://github.com/kamailio/kamailio/commit/8b1139851d9f88add1fd7da0dbc0b2b…
Author: Stefan Mititelu <stefan-cristian.mititelu(a)1and1.ro>
Committer: Stefan Mititelu <stefan.mititelu(a)govoip.ro>
Date: 2023-07-20T15:25:37+03:00
rtpengine: increase buf size to print mos llint range
---
Modified: src/modules/rtpengine/rtpengine.c
---
Diff: https://github.com/kamailio/kamailio/commit/8b1139851d9f88add1fd7da0dbc0b2b…
Patch: https://github.com/kamailio/kamailio/commit/8b1139851d9f88add1fd7da0dbc0b2b…
---
diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c
index 59bcfb47795..68003908268 100644
--- a/src/modules/rtpengine/rtpengine.c
+++ b/src/modules/rtpengine/rtpengine.c
@@ -180,14 +180,15 @@ struct minmax_mos_label_stats
};
struct minmax_stats_vals
{
- long long mos;
- long long at;
- long long packetloss;
- long long jitter;
- long long roundtrip;
- long long roundtrip_leg;
- long long samples;
- long long avg_samples; /* our own running count to average the averages */
+ long long int mos;
+ long long int at;
+ long long int packetloss;
+ long long int jitter;
+ long long int roundtrip;
+ long long int roundtrip_leg;
+ long long int samples;
+ long long int
+ avg_samples; /* our own running count to average the averages */
};
#define RTPE_LIST_VERSION_DELAY 10
@@ -3879,12 +3880,13 @@ static void avp_print_decimal(pv_elem_t *pv, int num, struct sip_msg *msg)
len = snprintf(buf, sizeof(buf), "%i.%i", num / 10, abs(num % 10));
avp_print_s(pv, buf, len, msg);
}
-static void avp_print_int(pv_elem_t *pv, int num, struct sip_msg *msg)
+static void avp_print_llint(
+ pv_elem_t *pv, long long int num, struct sip_msg *msg)
{
int len;
- char buf[8];
+ char buf[20];
- len = snprintf(buf, sizeof(buf), "%i", num);
+ len = snprintf(buf, sizeof(buf), "%lld", num);
avp_print_s(pv, buf, len, msg);
}
static void avp_print_time(pv_elem_t *pv, int num, struct sip_msg *msg)
@@ -3904,12 +3906,13 @@ static void avp_print_mos(struct minmax_mos_stats *s,
avp_print_decimal(s->mos_pv, vals->mos / vals->avg_samples, msg);
avp_print_time(s->at_pv, vals->at - created, msg);
- avp_print_int(s->packetloss_pv, vals->packetloss / vals->avg_samples, msg);
- avp_print_int(s->jitter_pv, vals->jitter / vals->avg_samples, msg);
- avp_print_int(s->roundtrip_pv, vals->roundtrip / vals->avg_samples, msg);
- avp_print_int(
+ avp_print_llint(
+ s->packetloss_pv, vals->packetloss / vals->avg_samples, msg);
+ avp_print_llint(s->jitter_pv, vals->jitter / vals->avg_samples, msg);
+ avp_print_llint(s->roundtrip_pv, vals->roundtrip / vals->avg_samples, msg);
+ avp_print_llint(
s->roundtrip_leg_pv, vals->roundtrip_leg / vals->avg_samples, msg);
- avp_print_int(s->samples_pv, vals->samples / vals->avg_samples, msg);
+ avp_print_llint(s->samples_pv, vals->samples / vals->avg_samples, msg);
}
static int decode_mos_vals_dict(
<!-- 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 -->
- [X] 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 -->
Make sure dialog cbs list is NULL after is destroyed.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3514
-- Commit Summary --
* dialog: set cbs list to NULL after destroying
-- File Changes --
M src/modules/dialog/dlg_hash.c (5)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3514.patchhttps://github.com/kamailio/kamailio/pull/3514.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3514
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3514(a)github.com>
Hello,
Kamailio SIP Server v5.5.7 stable release is out.
This is a maintenance release of the stable branch 5.5 that includes
fixes since the release of v5.5.6. There is no change to
database schema or configuration language structure that you have to do
on previous installations of v5.5.x. Deployments running previous v5.5.x
versions are strongly recommended to be upgraded to v5.5.7.
Note that 5.5 is now the third last stable branch, v5.5.7 being the last
planned released in 5.5.x series. The latest two stable branch are 5.6
and 5.7, with v5.7.1 being released a while ago.
For more details about version 5.5.7 (including links and guidelines to
download the tarball or from GIT repository), visit:
* https://www.kamailio.org/w/2023/07/kamailio-v5-5-7-released/
RPM, Debian/Ubuntu packages will be available soon as well.
Many thanks to all contributing and using Kamailio!
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - www.kamailioworld.com
Hello,
I am planning to release v5.5.7 our of branch 5.5 this week, most likely
on Wednesday, July 19, 2023. It will mark the end of official packaging
for release series 5.5.x, currently the latest two stable branches are
5.6 and 5.7.
Cheers,
Daniel
--
Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio World Conference - www.kamailioworld.com
Module: kamailio
Branch: master
Commit: c35088bf31b3e3e828433bf03b6ce647a75ef8e2
URL: https://github.com/kamailio/kamailio/commit/c35088bf31b3e3e828433bf03b6ce64…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-18T16:38:20+02:00
geoip2: docs - section ids and formatting
---
Modified: src/modules/geoip2/doc/geoip2_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/c35088bf31b3e3e828433bf03b6ce64…
Patch: https://github.com/kamailio/kamailio/commit/c35088bf31b3e3e828433bf03b6ce64…
---
diff --git a/src/modules/geoip2/doc/geoip2_admin.xml b/src/modules/geoip2/doc/geoip2_admin.xml
index 50a7d855fa8..cbc6ccb1493 100644
--- a/src/modules/geoip2/doc/geoip2_admin.xml
+++ b/src/modules/geoip2/doc/geoip2_admin.xml
@@ -11,26 +11,25 @@
<!-- Module User's Guide -->
<chapter>
-
+
<title>&adminguide;</title>
-
+
<section>
<title>Overview</title>
<para>
- This module allows real-time queries against the Max Mind GeoIP2
- database to be performed from the config script.
+ This module allows real-time queries against the Max Mind GeoIP2
+ database to be performed from the config script.
</para>
<para>
- The Max Mind GeoIP2 database is a map of IP network address assignments
+ The Max Mind GeoIP2 database is a map of IP network address assignments
to geographical locales that can be useful -- though approximate --
in identifying the physical location with which an IP host address
is associated on a relatively granular level.
</para>
<para>
- This database itself can be obtained on a free or commercial basis
+ This database itself can be obtained on a free or commercial basis
from <ulink url="http://dev.maxmind.com/geoip/">
- http://dev.maxmind.com/geoip/</ulink>. The
- library libmaxminddb
+ http://dev.maxmind.com/geoip/</ulink>. The library libmaxminddb
that interfaces with the Max Mind API, as well as scripts to
automate downloading of the on-disk version are available at
<ulink url="http://dev.maxmind.com/geoip/geoip2/downloadable/">
@@ -126,10 +125,10 @@ if(geoip2_match("$si", "src")) {
</example>
</section>
</section>
-
+
<section>
<title>Functions</title>
- <section>
+ <section id="geoip.f.geoip2_match">
<title>
<function moreinfo="none">geoip2_match(ipaddr, pvc)</function>
</title>
@@ -149,7 +148,7 @@ if(geoip2_match("$si", "src"))
</programlisting>
</example>
</section>
- <section>
+ <section id="geoip.f.geoip2_distance">
<title>
<function moreinfo="none">geoip2_distance(ip_addr, latitude, longitude)</function>
</title>
@@ -172,7 +171,7 @@ if(geoip2_match("$si", "src"))
</programlisting>
</example>
</section>
-
+
</section>
<section>
@@ -195,13 +194,13 @@ if(geoip2_match("$si", "src"))
</example>
</section>
</section>
-
- <section>
+
+ <section id="geoip.pvs">
<title>Exported pseudo-variables</title>
<itemizedlist>
<listitem><para>
- <emphasis>$gip2(pvc=>key)</emphasis> - <emphasis>pvc</emphasis> is an
- identifier for this query result; it is designated by the second
+ <emphasis>$gip2(pvc=>key)</emphasis> - <emphasis>pvc</emphasis> is an
+ identifier for this query result; it is designated by the second
parameter of geoip2_match(). The <emphasis>key</emphasis> can be one of
the following:
</para>
kamctl: added new options to dispatcher interface
- added a new `rmip` option to delete a specific gateway `ip` in a predefined `setid`
- added a new `rmset` option to delete a specific `setid` completely
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3512
-- Commit Summary --
* Add support for deleting a gateway or full setid in dispatcher.
* Small style fix to help screen
-- File Changes --
M utils/kamctl/kamctl (34)
M utils/kamctl/kamctl.base (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3512.patchhttps://github.com/kamailio/kamailio/pull/3512.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3512
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3512(a)github.com>
Adding support for secure AMQP connections over TLS (amqps).
<!-- 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)
- [ ] 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/3511
-- Commit Summary --
* rabbitmq: Adding amqps support
-- File Changes --
M src/modules/rabbitmq/rabbitmq.c (33)
M src/modules/rabbitmq/rabbitmq.h (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3511.patchhttps://github.com/kamailio/kamailio/pull/3511.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3511
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3511(a)github.com>
Module: kamailio
Branch: 5.5
Commit: edfbecd0ed515dab637a03a793c6c2d565dfdaee
URL: https://github.com/kamailio/kamailio/commit/edfbecd0ed515dab637a03a793c6c2d…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-17T11:17:12+02:00
sst: fix condition on sst_min_se for sst_check_min()
(cherry picked from commit a059af04ac47fd31238ca4f22ad868529e035c43)
(cherry picked from commit febfc9722f83d992a3c1643fa373468b73cb3fa0)
(cherry picked from commit 5fc4cca395e057714376b681bbad4935a9570d0d)
---
Modified: src/modules/sst/sst_handlers.c
---
Diff: https://github.com/kamailio/kamailio/commit/edfbecd0ed515dab637a03a793c6c2d…
Patch: https://github.com/kamailio/kamailio/commit/edfbecd0ed515dab637a03a793c6c2d…
---
diff --git a/src/modules/sst/sst_handlers.c b/src/modules/sst/sst_handlers.c
index 55010baaee2..690c03ddfac 100644
--- a/src/modules/sst/sst_handlers.c
+++ b/src/modules/sst/sst_handlers.c
@@ -672,7 +672,7 @@ int ki_sst_check_min(struct sip_msg *msg, int flag)
* too small. We will take the smaller of the messages
* Session-expires and min-se if stated.
*/
- if (sst_min_se < MIN(minse, se.interval)) {
+ if(sst_min_se > MIN(minse, se.interval)) {
/*
* Too small. See if we need to send the 422 and are able
* to send it.
Module: kamailio
Branch: 5.5
Commit: 2555e1b1a02af8d96504c43953ab395097c86adb
URL: https://github.com/kamailio/kamailio/commit/2555e1b1a02af8d96504c43953ab395…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-17T11:16:44+02:00
tm: enhance documentation for on_sl_reply param
(cherry picked from commit a4970749b2be40cd598d4ce5cf2eaa6b38fc2307)
(cherry picked from commit 2f86215a704ff10a58cb3f5e57497918c381a048)
---
Modified: src/modules/tm/doc/params.xml
---
Diff: https://github.com/kamailio/kamailio/commit/2555e1b1a02af8d96504c43953ab395…
Patch: https://github.com/kamailio/kamailio/commit/2555e1b1a02af8d96504c43953ab395…
---
diff --git a/src/modules/tm/doc/params.xml b/src/modules/tm/doc/params.xml
index 247d6ad5e6d..d8f8251818d 100644
--- a/src/modules/tm/doc/params.xml
+++ b/src/modules/tm/doc/params.xml
@@ -715,7 +715,7 @@ modparam("tm", "reparse_on_dns_failover", 0)
</para>
<para>
If KEMI is used, the parameter has to be the KEMI function name, which
- receives a string parameter with the event name.
+ receives a string parameter with the event name, respectively "on_sl_reply".
</para>
<example>
<title>Set <varname>on_sl_reply</varname> parameter</title>
@@ -727,6 +727,20 @@ modparam("tm", "on_sl_reply", "stateless_replies")
onreply_route["stateless_replies"] {
# do not allow stateless replies to be forwarded
return 0;
+}
+ </programlisting>
+ </example>
+ <example>
+ <title>Set <varname>on_sl_reply</varname> parameter (KEMI javascript)</title>
+ <programlisting>
+...
+modparam("tm", "on_sl_reply", "ksr_slreply")
+...
+-- event callback function implemented in javascript
+function ksr_slreply(evname) {
+ # the evname is set to "on_sl_reply"
+ # do not allow stateless replies to be forwarded
+ KSR.x.drop();
}
</programlisting>
</example>
Module: kamailio
Branch: 5.5
Commit: b1953700ae03bde4c4b4d0c549d09c2f66e8ec09
URL: https://github.com/kamailio/kamailio/commit/b1953700ae03bde4c4b4d0c549d09c2…
Author: Dennis <dennis_y(a)mail.ru>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-17T11:16:23+02:00
auth_ephemeral: fix authentication with secrets added at runtime (#3390)
* auth_ephemeral: fix authentication with secrets added at runtime
- proper way to work with shm pointer to secret list
* fix: free secret_struct shm
Free shm secret_struct in case the allocation error of secret_list pointer.
(cherry picked from commit 6fa5a8fa8ddfe09bdab2fb445bee94e51d86b7fc)
(cherry picked from commit ca13a3382b9067a0dc14fc7ec995ec2d95c8c6cb)
---
Modified: src/modules/auth_ephemeral/auth_ephemeral_mod.c
Modified: src/modules/auth_ephemeral/auth_ephemeral_mod.h
Modified: src/modules/auth_ephemeral/authorize.c
---
Diff: https://github.com/kamailio/kamailio/commit/b1953700ae03bde4c4b4d0c549d09c2…
Patch: https://github.com/kamailio/kamailio/commit/b1953700ae03bde4c4b4d0c549d09c2…
Module: kamailio
Branch: 5.6
Commit: 5fc4cca395e057714376b681bbad4935a9570d0d
URL: https://github.com/kamailio/kamailio/commit/5fc4cca395e057714376b681bbad493…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-17T10:44:58+02:00
sst: fix condition on sst_min_se for sst_check_min()
(cherry picked from commit a059af04ac47fd31238ca4f22ad868529e035c43)
(cherry picked from commit febfc9722f83d992a3c1643fa373468b73cb3fa0)
---
Modified: src/modules/sst/sst_handlers.c
---
Diff: https://github.com/kamailio/kamailio/commit/5fc4cca395e057714376b681bbad493…
Patch: https://github.com/kamailio/kamailio/commit/5fc4cca395e057714376b681bbad493…
---
diff --git a/src/modules/sst/sst_handlers.c b/src/modules/sst/sst_handlers.c
index 55010baaee2..690c03ddfac 100644
--- a/src/modules/sst/sst_handlers.c
+++ b/src/modules/sst/sst_handlers.c
@@ -672,7 +672,7 @@ int ki_sst_check_min(struct sip_msg *msg, int flag)
* too small. We will take the smaller of the messages
* Session-expires and min-se if stated.
*/
- if (sst_min_se < MIN(minse, se.interval)) {
+ if(sst_min_se > MIN(minse, se.interval)) {
/*
* Too small. See if we need to send the 422 and are able
* to send it.
Module: kamailio
Branch: 5.6
Commit: 2f86215a704ff10a58cb3f5e57497918c381a048
URL: https://github.com/kamailio/kamailio/commit/2f86215a704ff10a58cb3f5e5749791…
Author: Ovidiu Sas <osas(a)voipembedded.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-17T10:38:18+02:00
tm: enhance documentation for on_sl_reply param
(cherry picked from commit a4970749b2be40cd598d4ce5cf2eaa6b38fc2307)
---
Modified: src/modules/tm/doc/params.xml
---
Diff: https://github.com/kamailio/kamailio/commit/2f86215a704ff10a58cb3f5e5749791…
Patch: https://github.com/kamailio/kamailio/commit/2f86215a704ff10a58cb3f5e5749791…
---
diff --git a/src/modules/tm/doc/params.xml b/src/modules/tm/doc/params.xml
index 3f67ae3f4d9..8c01ab78d0c 100644
--- a/src/modules/tm/doc/params.xml
+++ b/src/modules/tm/doc/params.xml
@@ -715,7 +715,7 @@ modparam("tm", "reparse_on_dns_failover", 0)
</para>
<para>
If KEMI is used, the parameter has to be the KEMI function name, which
- receives a string parameter with the event name.
+ receives a string parameter with the event name, respectively "on_sl_reply".
</para>
<example>
<title>Set <varname>on_sl_reply</varname> parameter</title>
@@ -727,6 +727,20 @@ modparam("tm", "on_sl_reply", "stateless_replies")
onreply_route["stateless_replies"] {
# do not allow stateless replies to be forwarded
return 0;
+}
+ </programlisting>
+ </example>
+ <example>
+ <title>Set <varname>on_sl_reply</varname> parameter (KEMI javascript)</title>
+ <programlisting>
+...
+modparam("tm", "on_sl_reply", "ksr_slreply")
+...
+-- event callback function implemented in javascript
+function ksr_slreply(evname) {
+ # the evname is set to "on_sl_reply"
+ # do not allow stateless replies to be forwarded
+ KSR.x.drop();
}
</programlisting>
</example>
Module: kamailio
Branch: 5.6
Commit: ca13a3382b9067a0dc14fc7ec995ec2d95c8c6cb
URL: https://github.com/kamailio/kamailio/commit/ca13a3382b9067a0dc14fc7ec995ec2…
Author: Dennis <dennis_y(a)mail.ru>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-17T10:32:33+02:00
auth_ephemeral: fix authentication with secrets added at runtime (#3390)
* auth_ephemeral: fix authentication with secrets added at runtime
- proper way to work with shm pointer to secret list
* fix: free secret_struct shm
Free shm secret_struct in case the allocation error of secret_list pointer.
(cherry picked from commit 6fa5a8fa8ddfe09bdab2fb445bee94e51d86b7fc)
---
Modified: src/modules/auth_ephemeral/auth_ephemeral_mod.c
Modified: src/modules/auth_ephemeral/auth_ephemeral_mod.h
Modified: src/modules/auth_ephemeral/authorize.c
---
Diff: https://github.com/kamailio/kamailio/commit/ca13a3382b9067a0dc14fc7ec995ec2…
Patch: https://github.com/kamailio/kamailio/commit/ca13a3382b9067a0dc14fc7ec995ec2…
#### 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
- [ ] 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:
- [ ] PR should be backported to stable branches
- [x] Tested changes locally
- [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description
Added distance function which allows to calculate the distance between the geocoordinates of the the IP-address passed as incoming parameter (coordinates for the IP are determined inside the function) and the geocoordinates passed as parameters for the function.
Functions can be used to evaluate the RTPengine server closest to a user.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3506
-- Commit Summary --
* geoip2: added distance function
-- File Changes --
M src/modules/geoip2/doc/geoip2_admin.xml (23)
M src/modules/geoip2/geoip2_mod.c (154)
M src/modules/geoip2/geoip2_pv.c (8)
M src/modules/geoip2/geoip2_pv.h (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3506.patchhttps://github.com/kamailio/kamailio/pull/3506.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3506
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3506(a)github.com>
Module: kamailio
Branch: master
Commit: 60abf767988dba052d7e08c386a589c1aeb711ef
URL: https://github.com/kamailio/kamailio/commit/60abf767988dba052d7e08c386a589c…
Author: Nikolay Ivanuschak <ivanuschak.nikolay(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-14T14:47:12+02:00
geoip2: added distance function
Added distance function which allows to calculate the distance
between the geocoordinates of the the IP-address passed as
incoming parameter (coordinates for the IP are determined inside
the function) and the geocoordinates passed as parameters for the function.
---
Modified: src/modules/geoip2/doc/geoip2_admin.xml
Modified: src/modules/geoip2/geoip2_mod.c
Modified: src/modules/geoip2/geoip2_pv.c
Modified: src/modules/geoip2/geoip2_pv.h
---
Diff: https://github.com/kamailio/kamailio/commit/60abf767988dba052d7e08c386a589c…
Patch: https://github.com/kamailio/kamailio/commit/60abf767988dba052d7e08c386a589c…
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/3491
-- Commit Summary --
* mohqueue: typo
* osp: typos
* p_usrloc: typo
* permissions: typos
* pike: typo
* presence: typo
* qos: typos
* ratelimit: typo
* regex: typos
* registrar: typos and formatting
* rls: typos
* rr: typo
* rtimer: typo
* rtp_media_server: typos
* ruxc: typos
* sca: typos
* seas: typo
* snmpstats: typos
* statsd: typo
* tcpops: typos
* textops: typos
-- File Changes --
M src/modules/mohqueue/doc/mohqueue_admin.xml (2)
M src/modules/osp/destination.c (2)
M src/modules/osp/doc/osp_admin.xml (4)
M src/modules/osp/orig_transaction.c (2)
M src/modules/osp/osp_mod.c (2)
M src/modules/osp/osptoolkit.h (2)
M src/modules/p_usrloc/udomain.c (2)
M src/modules/permissions/trusted.c (4)
M src/modules/permissions/trusted.h (2)
M src/modules/pike/doc/pike_devel.xml (2)
M src/modules/presence/doc/presence_admin.xml (2)
M src/modules/qos/doc/qos_devel.xml (6)
M src/modules/qos/qos.c (2)
M src/modules/qos/qos_cb.c (4)
M src/modules/qos/qos_handlers.c (2)
M src/modules/ratelimit/doc/ratelimit_admin.xml (2)
M src/modules/regex/regex_mod.c (6)
M src/modules/registrar/doc/registrar_admin.xml (21)
M src/modules/registrar/lookup.c (4)
M src/modules/registrar/save.c (2)
M src/modules/registrar/save.h (2)
M src/modules/rls/notify.c (2)
M src/modules/rls/resource_notify.c (6)
M src/modules/rls/rls.c (4)
M src/modules/rls/subscribe.c (2)
M src/modules/rr/doc/rr_devel.xml (2)
M src/modules/rtimer/doc/rtimer_admin.xml (2)
M src/modules/rtp_media_server/doc/rtp_media_server_admin.xml (4)
M src/modules/rtp_media_server/rms_sdp.c (2)
M src/modules/rtp_media_server/rtp_media_server.c (6)
M src/modules/ruxc/doc/ruxc_admin.xml (8)
M src/modules/sca/sca_call_info.c (2)
M src/modules/sca/sca_subscribe.c (4)
M src/modules/sca/sca_util.c (2)
M src/modules/seas/event_dispatcher.c (2)
M src/modules/snmpstats/snmpObjects.c (2)
M src/modules/snmpstats/snmpstats.c (2)
M src/modules/statsd/doc/statsd_admin.xml (2)
M src/modules/tcpops/doc/params.xml (2)
M src/modules/tcpops/doc/tcpops.xml (2)
M src/modules/textops/api.h (2)
M src/modules/textops/doc/textops_admin.xml (2)
M src/modules/textopsx/api.h (2)
M src/modules/textopsx/doc/functions.xml (12)
M src/modules/textopsx/textopsx.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3491.patchhttps://github.com/kamailio/kamailio/pull/3491.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3491
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3491(a)github.com>
Module: kamailio
Branch: master
Commit: 1335e93a76d051cb724bc696ac857aa7c3a3f43e
URL: https://github.com/kamailio/kamailio/commit/1335e93a76d051cb724bc696ac857aa…
Author: Kamailio Dev <kamailio.dev(a)kamailio.org>
Committer: Kamailio Dev <kamailio.dev(a)kamailio.org>
Date: 2023-07-13T17:46:27+02:00
modules: readme files regenerated - imc ... [skip ci]
---
Modified: src/modules/imc/README
---
Diff: https://github.com/kamailio/kamailio/commit/1335e93a76d051cb724bc696ac857aa…
Patch: https://github.com/kamailio/kamailio/commit/1335e93a76d051cb724bc696ac857aa…
---
diff --git a/src/modules/imc/README b/src/modules/imc/README
index 6fa83d72aa6..a62f56d429f 100644
--- a/src/modules/imc/README
+++ b/src/modules/imc/README
@@ -46,6 +46,7 @@ Joey Golan
4.1. imc_manager()
4.2. imc_room_active(room)
+ 4.3. imc_room_member(room, user)
5. RPC Commands
@@ -73,7 +74,8 @@ Joey Golan
1.10. Set check_on_create parameter
1.11. Usage of imc_manager() function
1.12. Usage of imc_room_active() function
- 1.13. List of commands
+ 1.13. Usage of imc_room_member() function
+ 1.14. List of commands
Chapter 1. Admin Guide
@@ -102,6 +104,7 @@ Chapter 1. Admin Guide
4.1. imc_manager()
4.2. imc_room_active(room)
+ 4.3. imc_room_member(room, user)
5. RPC Commands
@@ -289,6 +292,7 @@ modparam("imc", "check_on_create", 1)
4.1. imc_manager()
4.2. imc_room_active(room)
+ 4.3. imc_room_member(room, user)
4.1. imc_manager()
@@ -329,6 +333,21 @@ if(is_method("MESSAGE)
}
...
+4.3. imc_room_member(room, user)
+
+ Return 1 (true) if the user is member of the room, -1 (false) if the
+ user is not member of the room. The parameters are the SIP URIs to
+ identify the room and the user, they can contain variables.
+
+ This function can be used from ANY_ROUTE.
+
+ Example 1.13. Usage of imc_room_member() function
+...
+ if(imc_room_member("sip:chat-sip@$fd", "sip:$rU@$rd")) {
+ ...
+ }
+...
+
5. RPC Commands
5.1. imc.list_rooms
@@ -377,7 +396,7 @@ kamcmd imc_list_members _room_
Next picture presents the list of commands and their parameters.
- Example 1.13. List of commands
+ Example 1.14. List of commands
...
1.create
Module: kamailio
Branch: master
Commit: 27a4fc543cb60636308e587523236b1ac4d56754
URL: https://github.com/kamailio/kamailio/commit/27a4fc543cb60636308e587523236b1…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2023-07-13T17:41:16+02:00
imc: docs for imc_room_member()
---
Modified: src/modules/imc/doc/imc_admin.xml
---
Diff: https://github.com/kamailio/kamailio/commit/27a4fc543cb60636308e587523236b1…
Patch: https://github.com/kamailio/kamailio/commit/27a4fc543cb60636308e587523236b1…
---
diff --git a/src/modules/imc/doc/imc_admin.xml b/src/modules/imc/doc/imc_admin.xml
index 1b485ac77ca..d77c3028b9b 100644
--- a/src/modules/imc/doc/imc_admin.xml
+++ b/src/modules/imc/doc/imc_admin.xml
@@ -332,6 +332,29 @@ if(is_method("MESSAGE)
...
}
...
+</programlisting>
+ </example>
+ </section>
+ <section>
+ <title id="imc.p.imc_room_member">
+ <function moreinfo="none">imc_room_member(room, user)</function>
+ </title>
+ <para>
+ Return 1 (true) if the user is member of the room, -1 (false) if the
+ user is not member of the room. The parameters are the SIP URIs to
+ identify the room and the user, they can contain variables.
+ </para>
+ <para>
+ This function can be used from ANY_ROUTE.
+ </para>
+ <example>
+ <title>Usage of <varname>imc_room_member()</varname> function</title>
+ <programlisting format="linespecific">
+...
+ if(imc_room_member("sip:chat-sip@$fd", "sip:$rU@$rd")) {
+ ...
+ }
+...
</programlisting>
</example>
</section>
<!-- 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 -->
- [x] 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/3489
-- Commit Summary --
* siptrace: fix pseudo var direction attribute length
-- File Changes --
M src/modules/siptrace/siptrace.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/3489.patchhttps://github.com/kamailio/kamailio/pull/3489.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3489
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3489(a)github.com>