sr-dev November 2020

sr-dev@lists.kamailio.org

22 participants
311 discussions

[kamailio/kamailio] sl: Fix 3xx stats RPC marshalling (#2542)

by Guillem Jover

#### 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) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist: - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description The struct_add() method was getting 4 values but only 3 "d". You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2542 -- Commit Summary -- * sl: Fix 3xx stats RPC marshalling -- File Changes -- M src/modules/sl/sl_stats.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2542.patch https://github.com/kamailio/kamailio/pull/2542.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2542

3 years, 5 months

git:master:ac55cc5c: sl: Fix 3xx stats RPC marshalling

by Guillem Jover

Module: kamailio Branch: master Commit: ac55cc5ce5fcbb50d3a2a6b228585dce8fc1150d URL: https://github.com/kamailio/kamailio/commit/ac55cc5ce5fcbb50d3a2a6b228585dc… Author: Guillem Jover <gjover(a)sipwise.com> Committer: Guillem Jover <gjover(a)sipwise.com> Date: 2020-11-05T18:37:30+01:00 sl: Fix 3xx stats RPC marshalling The struct_add() method was getting 4 values but only 3 "d". --- Modified: src/modules/sl/sl_stats.c --- Diff: https://github.com/kamailio/kamailio/commit/ac55cc5ce5fcbb50d3a2a6b228585dc… Patch: https://github.com/kamailio/kamailio/commit/ac55cc5ce5fcbb50d3a2a6b228585dc… --- diff --git a/src/modules/sl/sl_stats.c b/src/modules/sl/sl_stats.c index dd683eda5b..079786aade 100644 --- a/src/modules/sl/sl_stats.c +++ b/src/modules/sl/sl_stats.c @@ -73,7 +73,7 @@ static void rpc_stats(rpc_t* rpc, void* c) "202", total.err[RT_202], "2xx", total.err[RT_2xx]); - rpc->struct_add(st, "ddd", + rpc->struct_add(st, "dddd", "300", total.err[RT_300], "301", total.err[RT_301], "302", total.err[RT_302],

3 years, 5 months

git:master:e80fcf42: Merge pull request #2542 from guillemj/pu/rpc-marshalling

by GitHub

Module: kamailio Branch: master Commit: e80fcf42ebe52fc095016277a4b44d4726d4e9c5 URL: https://github.com/kamailio/kamailio/commit/e80fcf42ebe52fc095016277a4b44d4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2020-11-06T07:50:18+01:00 Merge pull request #2542 from guillemj/pu/rpc-marshalling sl: Fix 3xx stats RPC marshalling --- Modified: src/modules/sl/sl_stats.c --- Diff: https://github.com/kamailio/kamailio/commit/e80fcf42ebe52fc095016277a4b44d4… Patch: https://github.com/kamailio/kamailio/commit/e80fcf42ebe52fc095016277a4b44d4… --- diff --git a/src/modules/sl/sl_stats.c b/src/modules/sl/sl_stats.c index dd683eda5b..079786aade 100644 --- a/src/modules/sl/sl_stats.c +++ b/src/modules/sl/sl_stats.c @@ -73,7 +73,7 @@ static void rpc_stats(rpc_t* rpc, void* c) "202", total.err[RT_202], "2xx", total.err[RT_2xx]); - rpc->struct_add(st, "ddd", + rpc->struct_add(st, "dddd", "300", total.err[RT_300], "301", total.err[RT_301], "302", total.err[RT_302],

3 years, 5 months

[kamailio/kamailio] Kamailio freezes with DMQ and dialog (#2535)

by pwakano

### Description When running a load test Kamailio eventually becames unresponsive and stops processing calls. Kamilio is configured to use the DMQ replication for dialog and usrloc. Also the dialog keepalived is enabled. ### Troubleshooting >From investigation, the problem happens faster and easier when there is some network degradation causing packet loss and/or retransmissions but even without any noticeable network issue the freeze eventually happens. #### Reproduction Run a simple load test making calls at a rate of ~5 cps and keep around ~2000 calls connected all the time. A higher cps seems to make it easier to reproduce the problem. Adding network degradation to the environment makes the problem happens, but when running a tool such as SIPp for the load test, the retransmission can be forced by simply killing the SIPp instance receiving calls which will then force Kamailio to retransmit. #### Debugging Data  Output of kamct trap: [gdb_kamailio_20201028_213030.txt](https://github.com/kamailio/kamailio/file… #### Log Messages Local generated requests shows up in the log, but are not sent in the network ### Possible Solutions Not found so far ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` # kamailio -v version: kamailio 5.4.2 (x86_64/linux) c3b91f 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_BLACKLIST, HAVE_RESOLV_RES 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: c3b91f compiled on 13:50:37 Oct 27 2020 with gcc 4.8.5 ``` * **Operating System**:  ``` # cat /etc/centos-release CentOS Linux release 7.8.2003 (Core) # uname -a Linux hostname 3.10.0-1127.19.1.el7.x86_64 #1 SMP Tue Aug 25 17:23:54 UTC 2020 x86_64 x86_64 x86_64 GNU/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/2535

3 years, 5 months

[kamailio/kamailio] add rtpproxy in kamailio/kamailio-ci:master-alpine (#2109)

by Gaurav Bmotra

hi can any one help me to add rtpproxy in kamailio/kamailio-ci:master-alpine thank you -- 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/2109

3 years, 5 months

[kamailio/kamailio] ACK not forwarded in kamailio 5.4.x (#2539)

by Julien Chavanton

### Description I notice a test failure when validating Kamailio 5.4.x, and after looking at the SIP traces, Kamailio is does not forwarding the first ACK. However, it does it after receiving a 200 OK re-transmission . ``` 192.0.2.12:5062 192.0.2.24:5060 ──────────┬───────── ──────────┬───────── │ │ │ │ │ │ 200 coco (SDP │ │ <<<──────────────── 200 coco (SDP) │ <───────────────────────────────────────────── │ ACK │ ─────────────────────────────────────────────> │ ACK │ ───────────────────────────────────────────>>> │ │ │ 200 coco (SDP │ │ <<<──────────────── 200 coco (SDP) │ <───────────────────────────────────────────── │ │ │ ACK │ ─────────────────────────────────────────────> │ │ │ ACK │ │ ──────────────────> ``` ### Troubleshooting I did proceed by isolation and ended up concluding that this commit was triggering this behavior. Simply, reverting this commit is fixing the problem. ``` commit 28049aafc8dd06c160ce5e7b8d5e4fc728441b0c Author: Semen Darienko <semen.darienko(a)wildix.com> Date: Sun May 3 12:26:45 2020 +0200 core: dns - use all NAPTR records - enable using of all NAPTR records instead of the first one ordered by priority - GH #2290 ``` I did not look at the code modification and/or anything config specific yet, I will try to find some time to debug. -- 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/2539

3 years, 5 months

git:5.4:a9720603: core: DNS stop resolving NAPTR for IP addresses (#2541)

by Julien Chavanton

Module: kamailio Branch: 5.4 Commit: a97206031ff0c28f791a5789f6dfc37707a4b59d URL: https://github.com/kamailio/kamailio/commit/a97206031ff0c28f791a5789f6dfc37… Author: Julien Chavanton <jchavanton(a)gmail.com> Committer: Julien Chavanton <jchavanton(a)gmail.com> Date: 2020-11-05T07:14:44-08:00 core: DNS stop resolving NAPTR for IP addresses (#2541) (cherry picked from commit 37f06f031c4ce93bcf4494744282c17f7a2d157c) --- Modified: src/core/dns_cache.c --- Diff: https://github.com/kamailio/kamailio/commit/a97206031ff0c28f791a5789f6dfc37… Patch: https://github.com/kamailio/kamailio/commit/a97206031ff0c28f791a5789f6dfc37… --- diff --git a/src/core/dns_cache.c b/src/core/dns_cache.c index 923c65049d..e383786de2 100644 --- a/src/core/dns_cache.c +++ b/src/core/dns_cache.c @@ -3292,6 +3292,9 @@ inline static int dns_naptr_sip_resolve(struct dns_srv_handle* h, str* name, } try_lookup_naptr = 1; } + /* check if it's an ip address, dns_srv_sip_resolve will return the right failure */ + if (str2ip(name) || str2ip6(name)) + goto naptr_not_found; /* do naptr lookup */ if ((e=dns_get_entry(name, T_NAPTR))==0) goto naptr_not_found;

3 years, 5 months

[kamailio/kamailio] core: DNS stop resolving NAPTR for IP addresses (#2541)

by Julien Chavanton

#### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) #### Checklist:  - [x] PR should be backported to stable branches - [ ] Tested changes locally - [x] Related to issue #2539 #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2541 -- Commit Summary -- * core: DNS stop resolving NAPTR for IP addresses -- File Changes -- M src/core/dns_cache.c (3) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2541.patch https://github.com/kamailio/kamailio/pull/2541.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2541

3 years, 5 months

git:master:37f06f03: core: DNS stop resolving NAPTR for IP addresses (#2541)

by GitHub

Module: kamailio Branch: master Commit: 37f06f031c4ce93bcf4494744282c17f7a2d157c URL: https://github.com/kamailio/kamailio/commit/37f06f031c4ce93bcf4494744282c17… Author: Julien Chavanton <jchavanton(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2020-11-05T07:09:38-08:00 core: DNS stop resolving NAPTR for IP addresses (#2541) --- Modified: src/core/dns_cache.c --- Diff: https://github.com/kamailio/kamailio/commit/37f06f031c4ce93bcf4494744282c17… Patch: https://github.com/kamailio/kamailio/commit/37f06f031c4ce93bcf4494744282c17… --- diff --git a/src/core/dns_cache.c b/src/core/dns_cache.c index 926dbfef76..9a4b90fba3 100644 --- a/src/core/dns_cache.c +++ b/src/core/dns_cache.c @@ -3292,6 +3292,9 @@ inline static int dns_naptr_sip_resolve(struct dns_srv_handle* h, str* name, } try_lookup_naptr = 1; } + /* check if it's an ip address, dns_srv_sip_resolve will return the right failure */ + if (str2ip(name) || str2ip6(name)) + goto naptr_not_found; /* do naptr lookup */ if ((e=dns_get_entry(name, T_NAPTR))==0) goto naptr_not_found;

3 years, 5 months

[kamailio/kamailio] kex: fix and cleanup stats rpc (#2537)

by Alex Hermann

#1762 ## Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [x] 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  Fix invalid JSON returned by stats.fetch with multiple modules as parameter. Cleanup by reducing code duplication. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2537 -- Commit Summary -- * kex: Make stats.fetch(n) reply with valid JSON * kex: Remove a lot of duplicate code from rpc_stats_fetch* * kex: Let the RPC server do printf as it may have optimizations -- File Changes -- M src/modules/kex/core_stats.c (222) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2537.patch https://github.com/kamailio/kamailio/pull/2537.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2537

3 years, 5 months

← Newer
1
...
24
25
26
27
28
29
30
31
32
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev November 2020