sr-dev June 2022

sr-dev@lists.kamailio.org

20 participants
282 discussions

[kamailio/kamailio] detailed_ipv4_type func pkg memory leak. (Issue #3159)
by 杰哥 25 Jun '22

25 Jun '22

### Description when call detailed_ipv4_type function, pkg memory leak. ```c detailed_ipv4_type($si,"$var(iptype)"); ``` ### Troubleshooting #### Reproduction just call detailed_ipv4_type function. #### Debugging Data ``` ALERT: qm_sums: qm_sums(): count= 48161 size= 1789640 bytes from core: core/rvalue.c: rval_get_str(1293) ``` ### Possible Solutions ip address string not free in rval_get_str() function. [s->s=pkg_malloc(tmp.len+1/* 0 term */);](https://github.com/kamailio/kamailio/blob/1ddc27f199061025a6a43da3e8a1388fcaddfe19/src/core/rvalue.c#L1293) need call pkg_free() free memory. ### Additional Information test 3 version. ``` kamailio 5.5.4 kamailio 5.4.8 kamailio 5.3.9 ``` * **Operating System**: ``` buster 10 ubuntu 20.04 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3159 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3159(a)github.com>

2 7

[kamailio/kamailio] topos breaks early-dialog update even on 5.6.0~rc0 (Issue #3112)
by George Diamantopoulos 24 Jun '22

24 Jun '22

### Description Topos still seems to break early-dialog UPDATEs in my configuration. This is on 5.6.0~rc0+bpo11.20220506004754.1 from 56-nightly which I believe includes c309122 and ceb688b. I haven't tested rc1 yet (there are no debs at this time) but there doesn't seem to be any topos-related commits since rc0 either. ### Troubleshooting #### SIP Traffic Attaching pcaps from a lab environment to demonstrate the problem. One with topos on (failure), one with off (success). There were no other configuration changes between these tests. ### Possible Solutions Only turning topos off seems to resolve this for me. ### Additional Information ``` version: kamailio 5.6.0-rc0 (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**: ``` Linux sbcprov0-stage-tis0-cn1 5.10.0-10-amd64 #1 SMP Debian 5.10.84-1 (2021-12-08) x86_64 GNU/Linux ``` [pcaps.tar.gz](https://github.com/kamailio/kamailio/files/8704222/pcaps.tar.… -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3112 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3112(a)github.com>

3 6

git:master:87b698f7: modules: readme files regenerated - textops ... [skip ci]
by Kamailio Dev 24 Jun '22

24 Jun '22

Module: kamailio Branch: master Commit: 87b698f7deca1794019f98167ba49d3e4737ba71 URL: https://github.com/kamailio/kamailio/commit/87b698f7deca1794019f98167ba49d3… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-06-24T16:01:17+02:00 modules: readme files regenerated - textops ... [skip ci] --- Modified: src/modules/textops/README Modified: src/modules/tls/README --- Diff: https://github.com/kamailio/kamailio/commit/87b698f7deca1794019f98167ba49d3… Patch: https://github.com/kamailio/kamailio/commit/87b698f7deca1794019f98167ba49d3… --- diff --git a/src/modules/textops/README b/src/modules/textops/README index fabc8e3371..c187c4c8db 100644 --- a/src/modules/textops/README +++ b/src/modules/textops/README @@ -157,8 +157,8 @@ Ovidiu Sas 1.43. has_body usage 1.44. is_audio_on_hold usage 1.45. is_privacy usage - 1.46. in_list() usage - 1.47. in_list() usage + 1.46. in_list usage + 1.47. in_list_prefix usage 1.48. cmp_str usage 1.49. cmp_str usage 1.50. starts_with usage @@ -1239,7 +1239,7 @@ if(is_privacy("id")) Function can be used from all kinds of routes. - Example 1.46. in_list() usage + Example 1.46. in_list usage ... $var(subject) = "fi"; $var(list) = "dk,fi,no,se"; @@ -1257,7 +1257,7 @@ if (in_list("$var(subject)", "$var(list)", ",")) { Function can be used from all kinds of routes. - Example 1.47. in_list() usage + Example 1.47. in_list_prefix usage ... $var(subject) = "final"; $var(list) = "dk,fi,no,se"; diff --git a/src/modules/tls/README b/src/modules/tls/README index 94c8cfc069..c9bed56ec5 100644 --- a/src/modules/tls/README +++ b/src/modules/tls/README @@ -250,7 +250,7 @@ Chapter 1. Admin Guide The most important parameters to set the path to the public certificate and private key files. You can either have them in different file or in the same file in PEM format. The parameters for them are certificate - and private_key. They can be given as modparam or or provided in the + and private_key. They can be given as modparam or provided in the profiles of tls.cfg file. When installing tls module of kamailio, a sample 'tls.cfg' file is

1 0

[kamailio/kamailio] Fix typos (PR #3162)
by Akash Gupta 24 Jun '22

24 Jun '22

#### Pre-Submission Checklist    - [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:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3162 -- Commit Summary -- * textops: Fix typos * tls: Fix typo -- File Changes -- M src/modules/textops/doc/textops_admin.xml (4) M src/modules/tls/doc/tls.xml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3162.patch https://github.com/kamailio/kamailio/pull/3162.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3162 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3162(a)github.com>

2 2

git:master:4570b3eb: tls: Fix typo
by Daniel-Constantin Mierla 24 Jun '22

24 Jun '22

Module: kamailio Branch: master Commit: 4570b3ebc6da35efc3cc55b1f4418446b9071688 URL: https://github.com/kamailio/kamailio/commit/4570b3ebc6da35efc3cc55b1f441844… Author: Akash Gupta <akash(a)telnyx.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-06-24T15:49:50+02:00 tls: Fix typo --- Modified: src/modules/tls/doc/tls.xml --- Diff: https://github.com/kamailio/kamailio/commit/4570b3ebc6da35efc3cc55b1f441844… Patch: https://github.com/kamailio/kamailio/commit/4570b3ebc6da35efc3cc55b1f441844… --- diff --git a/src/modules/tls/doc/tls.xml b/src/modules/tls/doc/tls.xml index e356d5bff7c..f3ff2321c78 100644 --- a/src/modules/tls/doc/tls.xml +++ b/src/modules/tls/doc/tls.xml @@ -84,7 +84,7 @@ The most important parameters to set the path to the public certificate and private key files. You can either have them in different file or in the same file in PEM format. The parameters for them are <varname>certificate</varname> and <varname>private_key</varname>. - They can be given as modparam or or provided in the profiles of tls.cfg file. + They can be given as modparam or provided in the profiles of tls.cfg file. </para> <para> When installing tls module of kamailio, a sample 'tls.cfg' file is deployed in the same

1 0

git:master:c5cd4002: textops: Fix typos
by Daniel-Constantin Mierla 24 Jun '22

24 Jun '22

Module: kamailio Branch: master Commit: c5cd400287e27fce6fcf6272a4efe9a9fa8aa419 URL: https://github.com/kamailio/kamailio/commit/c5cd400287e27fce6fcf6272a4efe9a… Author: Akash Gupta <akash(a)telnyx.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-06-24T15:49:50+02:00 textops: Fix typos --- Modified: src/modules/textops/doc/textops_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/c5cd400287e27fce6fcf6272a4efe9a… Patch: https://github.com/kamailio/kamailio/commit/c5cd400287e27fce6fcf6272a4efe9a… --- diff --git a/src/modules/textops/doc/textops_admin.xml b/src/modules/textops/doc/textops_admin.xml index 9805aac6aab..dbca57e5f0a 100644 --- a/src/modules/textops/doc/textops_admin.xml +++ b/src/modules/textops/doc/textops_admin.xml @@ -1711,7 +1711,7 @@ if(is_privacy("id")) Function can be used from all kinds of routes. </para> <example> - <title><function>in_list()</function> usage</title> + <title><function>in_list</function> usage</title> <programlisting format="linespecific"> ... $var(subject) = "fi"; @@ -1735,7 +1735,7 @@ if (in_list("$var(subject)", "$var(list)", ",")) { Function can be used from all kinds of routes. </para> <example> - <title><function>in_list()</function> usage</title> + <title><function>in_list_prefix</function> usage</title> <programlisting format="linespecific"> ... $var(subject) = "final";

1 0

[kamailio/kamailio] Fix typos (PR #3161)
by Akash Gupta 24 Jun '22

24 Jun '22

#### Pre-Submission Checklist    - [ ] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - - [x] [ ] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - - [ ] [ ] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3161 -- Commit Summary -- * Fix typos -- File Changes -- M src/modules/textops/doc/textops_admin.xml (4) M src/modules/tls/doc/tls.xml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3161.patch https://github.com/kamailio/kamailio/pull/3161.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3161 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3161(a)github.com>

2 4

git:master:65a484df: modules: readme files regenerated - nats ... [skip ci]
by Kamailio Dev 24 Jun '22

24 Jun '22

Module: kamailio Branch: master Commit: 65a484dffa3c854bc17068175cee3dc37fd09d18 URL: https://github.com/kamailio/kamailio/commit/65a484dffa3c854bc17068175cee3dc… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-06-24T12:46:16+02:00 modules: readme files regenerated - nats ... [skip ci] --- Modified: src/modules/nats/README --- Diff: https://github.com/kamailio/kamailio/commit/65a484dffa3c854bc17068175cee3dc… Patch: https://github.com/kamailio/kamailio/commit/65a484dffa3c854bc17068175cee3dc… --- diff --git a/src/modules/nats/README b/src/modules/nats/README index f7f1ad7239..29cef171b7 100644 --- a/src/modules/nats/README +++ b/src/modules/nats/README @@ -35,6 +35,7 @@ Emmanuel Schmidbauer 4.1. nats_url (str) 4.2. num_publish_workers (int) 4.3. subject_queue_group (str) + 4.4. event_callback (str) 5. Functions @@ -48,9 +49,10 @@ Emmanuel Schmidbauer 1.1. Set nats_url parameter 1.2. Set num_publish_workers parameter 1.3. Set subject_queue_group parameter - 1.4. nats_publish usage - 1.5. Example usage of $natsData pseudo variable - 1.6. Define the event routes + 1.4. Set event_callback parameter + 1.5. nats_publish usage + 1.6. Example usage of $natsData pseudo variable + 1.7. Define the event routes Chapter 1. Admin Guide @@ -68,6 +70,7 @@ Chapter 1. Admin Guide 4.1. nats_url (str) 4.2. num_publish_workers (int) 4.3. subject_queue_group (str) + 4.4. event_callback (str) 5. Functions @@ -115,6 +118,7 @@ Chapter 1. Admin Guide 4.1. nats_url (str) 4.2. num_publish_workers (int) 4.3. subject_queue_group (str) + 4.4. event_callback (str) 4.1. nats_url (str) @@ -161,6 +165,24 @@ modparam("nats", "subject_queue_group", "MyQueue1:2021") modparam("nats", "subject_queue_group", "MyQueue2:2021") ... +4.4. event_callback (str) + + Name of the KEMI function to be executed instead of the event route. + + Default value is not set. + + Example 1.4. Set event_callback parameter +... +modparam("nats", "event_callback", "ksr_nats_event") + +-- event callback function implemented in Lua +function ksr_nats_event(evname) + KSR.info("===== nats module received event: " .. evname .. + ", data:" .. KSR.pv.gete('$natsData') .. "\n"); + return 1; +end +... + 5. Functions 5.1. nats_publish(subject, payload) @@ -169,7 +191,7 @@ modparam("nats", "subject_queue_group", "MyQueue2:2021") Publishes the payload to subject. - Example 1.4. nats_publish usage + Example 1.5. nats_publish usage ... $var(my_info) = "$ci=" + $ci + " $fU=" + $fU; nats_publish("mysubject", "$var(my_info)"); # publish $var(my_info) to "mysubjec @@ -178,7 +200,7 @@ t" 6. Pseudo Variables - Example 1.5. Example usage of $natsData pseudo variable + Example 1.6. Example usage of $natsData pseudo variable ... xlog("L_INFO", "received payload $natsData"); } @@ -191,7 +213,7 @@ t" received payload. The name of the event-route name must match the subject of the message. - Example 1.6. Define the event routes + Example 1.7. Define the event routes ... modparam("nats", "subject_queue_group", "Kamailio-World:2021") modparam("nats", "subject_queue_group", "MyQueue1:2021")

1 0

[kamailio/kamailio] nats: add KEMI publish function and event_callback param (PR #3154)
by Seven Du 24 Jun '22

24 Jun '22

#### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [ ] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] 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  - add `event_callback` KEMI param like in other modules - expose `KSR.nats.publish` I can fix module doc if it looks good. How to generate docs locally btw? You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3154 -- Commit Summary -- * nats: add KEMI publish function and event_callback param -- File Changes -- M src/modules/nats/defs.h (3) M src/modules/nats/nats_mod.c (88) M src/modules/nats/nats_mod.h (3) M src/modules/nats/nats_pub.c (5) M src/modules/nats/nats_pub.h (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3154.patch https://github.com/kamailio/kamailio/pull/3154.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3154 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3154(a)github.com>

4 11

git:master:25d42b7e: nats: add KEMI publish function and event_callback param (#3154)
by GitHub 24 Jun '22

24 Jun '22

Module: kamailio Branch: master Commit: 25d42b7e69cc6242c66887a2816499d841e3ba89 URL: https://github.com/kamailio/kamailio/commit/25d42b7e69cc6242c66887a2816499d… Author: Seven Du <dujinfang(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2022-06-24T12:44:28+02:00 nats: add KEMI publish function and event_callback param (#3154) --- Modified: src/modules/nats/defs.h Modified: src/modules/nats/doc/nats_admin.xml Modified: src/modules/nats/nats_mod.c Modified: src/modules/nats/nats_mod.h Modified: src/modules/nats/nats_pub.c Modified: src/modules/nats/nats_pub.h --- Diff: https://github.com/kamailio/kamailio/commit/25d42b7e69cc6242c66887a2816499d… Patch: https://github.com/kamailio/kamailio/commit/25d42b7e69cc6242c66887a2816499d…

1 0

← Newer
1
...
11
12
13
14
15
16
17
...
29
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev June 2022