sr-dev

sr-dev@lists.kamailio.org

20 participants
45178 discussions

git:master:6c40d950: modules: readme files regenerated - app_perl ... [skip ci]
by Kamailio Dev 14 Jul '22

14 Jul '22

Module: kamailio Branch: master Commit: 6c40d950e9060d91df6a2df60051d627a0c73ede URL: https://github.com/kamailio/kamailio/commit/6c40d950e9060d91df6a2df60051d62… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-07-14T08:46:21+02:00 modules: readme files regenerated - app_perl ... [skip ci] --- Modified: src/modules/app_perl/README --- Diff: https://github.com/kamailio/kamailio/commit/6c40d950e9060d91df6a2df60051d62… Patch: https://github.com/kamailio/kamailio/commit/6c40d950e9060d91df6a2df60051d62… --- diff --git a/src/modules/app_perl/README b/src/modules/app_perl/README index 780fd683fd..9b5ac35bfd 100644 --- a/src/modules/app_perl/README +++ b/src/modules/app_perl/README @@ -427,7 +427,7 @@ modparam("app_perl", "parse_mode", 1) Allows to enable warning for perl_parse(). If set to 1, "-w" is passed to perl_parse() and detected warnings in the Perl script should be - printed to the log. + printed to the log. If set to 1, then "-W" is passed to perl_parse(). Default value is 0.

1 0

git:master:5828d825: topos: parse the corresponding param for initial methods
by Daniel-Constantin Mierla 14 Jul '22

14 Jul '22

Module: kamailio Branch: master Commit: 5828d8252c18df13b4a37321168c88687f9fc3b3 URL: https://github.com/kamailio/kamailio/commit/5828d8252c18df13b4a37321168c886… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-07-14T08:42:34+02:00 topos: parse the corresponding param for initial methods --- Modified: src/modules/topos/topos_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/5828d8252c18df13b4a37321168c886… Patch: https://github.com/kamailio/kamailio/commit/5828d8252c18df13b4a37321168c886… --- diff --git a/src/modules/topos/topos_mod.c b/src/modules/topos/topos_mod.c index 4c096cf298..90c60e39e4 100644 --- a/src/modules/topos/topos_mod.c +++ b/src/modules/topos/topos_mod.c @@ -229,7 +229,7 @@ static int mod_init(void) } } if(_tps_methods_noinitial_list.len>0) { - if(parse_methods(&_tps_methods_nocontact_list, &_tps_methods_noinitial)<0) { + if(parse_methods(&_tps_methods_noinitial_list, &_tps_methods_noinitial)<0) { LM_ERR("failed to parse methods_noinitial parameter\n"); return -1; }

1 0

git:master:e53db293: app_perl: if warn_mode is 2, set -W for perl_parse()
by Daniel-Constantin Mierla 14 Jul '22

14 Jul '22

Module: kamailio Branch: master Commit: e53db29378b452c79654b220353fa794081597f7 URL: https://github.com/kamailio/kamailio/commit/e53db29378b452c79654b220353fa79… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-07-13T18:24:50+02:00 app_perl: if warn_mode is 2, set -W for perl_parse() --- Modified: src/modules/app_perl/app_perl_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/e53db29378b452c79654b220353fa79… Patch: https://github.com/kamailio/kamailio/commit/e53db29378b452c79654b220353fa79… --- diff --git a/src/modules/app_perl/app_perl_mod.c b/src/modules/app_perl/app_perl_mod.c index 67120d49e4..1c55c14c28 100644 --- a/src/modules/app_perl/app_perl_mod.c +++ b/src/modules/app_perl/app_perl_mod.c @@ -212,6 +212,8 @@ int parser_init(void) { if(_ap_warn_mode==1) { argv[argc] = "-w"; argc++; + } else if(_ap_warn_mode==2) { + argv[argc] = "-W"; argc++; } /* Possible Include path extension by modparam */

1 0

git:master:ded47735: app_perl: docs - updates for warn_mode
by Daniel-Constantin Mierla 14 Jul '22

14 Jul '22

Module: kamailio Branch: master Commit: ded477354676cd6238534001a0dfa6abe1198ba3 URL: https://github.com/kamailio/kamailio/commit/ded477354676cd6238534001a0dfa6a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-07-13T18:27:42+02:00 app_perl: docs - updates for warn_mode --- Modified: src/modules/app_perl/doc/app_perl_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/ded477354676cd6238534001a0dfa6a… Patch: https://github.com/kamailio/kamailio/commit/ded477354676cd6238534001a0dfa6a… --- diff --git a/src/modules/app_perl/doc/app_perl_admin.xml b/src/modules/app_perl/doc/app_perl_admin.xml index 22b8c10308..8793086d85 100644 --- a/src/modules/app_perl/doc/app_perl_admin.xml +++ b/src/modules/app_perl/doc/app_perl_admin.xml @@ -267,7 +267,8 @@ modparam("app_perl", "parse_mode", 1) <para> Allows to enable warning for perl_parse(). If set to 1, "-w" is passed to perl_parse() and detected warnings in the Perl script - should be printed to the log. + should be printed to the log. If set to 1, then "-W" is passed + to perl_parse(). </para> <para> Default value is <emphasis>0</emphasis>.

1 0

[kamailio/kamailio] m_dump doesnt use usrloc to deliver message (Issue #3185)
by admin-toneca 13 Jul '22

13 Jul '22

### Description When client register, m_dump sent message to host from header "TO" in stored message, instead to send to AOR. ### Troubleshooting modparam("msilo", "db_url", DBURL) modparam("msilo", "from_address", "sip:registrar@kamailio.org") modparam("msilo", "content_type_hdr", "Content-Type: text/plain\r\n") modparam("msilo", "offline_message", "*** User $rU is offline!") route[REGISTRAR] { xlog("(REGISTRAR) =============================================================================\n"); if (!is_method("REGISTER")) return; if(isflagset(FLT_NATS)) { setbflag(FLB_NATB); #!ifdef WITH_NATSIPPING # do SIP NAT pinging setbflag(FLB_NATSIPPING); #!endif } if (!save("location")) { sl_reply_error(); } xlog("L_NOTICE", "REGISTER received -> dumping messages with MSILO\n"); # MSILO - dumping user's offline messages if (m_dump()) { xlog("L_NOTICE", "MSILO: offline messages dumped - if they were\n"); } else { xlog("L_NOTICE", "MSILO: no offline messages dumped\n"); }; exit; } #### Log Messages Message which kamailio tried to sent: INSERT INTO `silo` (`id`, `src_addr`, `dst_addr`, `username`, `domain`, `inc_time`, `exp_time`, `snd_time`, `ctype`, `body`, `extra_hdrs`, `callid`, `status`) VALUES (4, 'sip:1867@sbc.test.org', 'sip:1404@sbc.test.org', '1404', 'sbc.test.org', 1657654135, 1657913335, 0, 'text/html', 0x3c5350414e205354594c453d22464f4e542d46414d494c593a417269616c3b20464f4e542d53495a453a3130707420223e7171717171717171717171713c2f5350414e3e0d0a, '', '', 0); ``` [kamailio.log](https://github.com/kamailio/kamailio/files/9096693/kamailio.l… ``` #### SIP Traffic 3.127.6.184 is public IP of EC2 172.31.3.158 is private IP of same instance So you can see that kamailio tried to sent SIP MESSAGE to self ``` 19:59:52.341870 IP 17.74.26.179.50038 > 172.31.3.158.5060: SIP: REGISTER sip:sbc.test.org SIP/2.0 19:59:52.381456 IP 172.31.3.158.5060 > 17.74.26.179.50038: SIP: SIP/2.0 200 OK 19:59:52.538450 IP 17.74.26.179.50038 > 172.31.3.158.5060: SIP: SUBSCRIBE sip:1404@sbc.test.org SIP/2.0 19:59:52.577632 IP 172.31.3.158.5060 > 17.74.26.179.50038: SIP: SIP/2.0 407 Proxy Authentication Required 19:59:52.724509 IP 172.31.3.158.5060 > 3.127.6.184.5060: SIP: MESSAGE sip:1404@sbc.test.org SIP/2.0 19:59:52.724597 IP 3.127.6.184.5060 > 172.31.3.158.5060: SIP: MESSAGE sip:1404@sbc.test.org SIP/2.0 19:59:52.754379 IP 17.74.26.179.50038 > 172.31.3.158.5060: SIP: SUBSCRIBE sip:1404@sbc.test.org SIP/2.0 19:59:52.813768 IP 172.31.3.158.5060 > 3.127.6.184.5060: SIP: SIP/2.0 407 Proxy Authentication Required 19:59:52.813865 IP 3.127.6.184.5060 > 172.31.3.158.5060: SIP: SIP/2.0 407 Proxy Authentication Required ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.7.0-dev0 (x86_64/linux) 0d9380 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: 0d9380 compiled on 14:06:32 May 26 2022 with gcc 11.2.0 ``` * **Operating System**:  ``` Linux ip-172-31-3-158 5.15.0-1013-aws #17-Ubuntu SMP Fri Jun 10 10:40:12 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3185 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3185(a)github.com>

2 2

Kamailio Random Crash Alert - 5.4.7
by Hardik Patel 13 Jul '22

13 Jul '22

Hello Team, We are using Kamailio 5.4.7 and we found there is a random crash. We have observed the core logs and we noticed it is due to some of the RPC events. We have noticed it has been crashed from the "ctl handler" process. Please find the attached crash logs and Kamailio version details are mentioned below. version: kamailio 5.4.7 (x86_64/linux) a19fe1 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, 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: a19fe1 compiled on 02:56:57 Feb 16 2022 with gcc 8.3.0 -- Hardik Patel

1 0

Kamailio Random Crash Alert - 5.4.7
by Hardik Patel 12 Jul '22

12 Jul '22

1 0

[kamailio/kamailio] presence: No autocommit+rollback if no active watchers (PR #3183)
by Alessio Garzi 12 Jul '22

12 Jul '22

#### Pre-Submission Checklist    - [x] 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 - [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:  - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description This is a small performance improvement, the goal is to avoid issuing rollbacks if not necessary. On my setup, Kamailio presence executes a lot of ROLLBACKS in MariaDB since function process_dialogs() runs repeatedly. What see inside MariaDB logs is the following ``` SET autocommit=0 select `presentity_uri`,`callid`,`to_tag`,`from_tag`,`event` from `active_watchers` where `updated`=3 AND `event`<>'presence.winfo'\G ROLLBACK ``` Maybe we can improve this behaviour telling presence to do autocommit to do not execute them if the select returns zero elements? You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3183 -- Commit Summary -- * presence: No autocommit+rollback if no active watchers -- File Changes -- M src/modules/presence/notify.c (20) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3183.patch https://github.com/kamailio/kamailio/pull/3183.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3183 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3183(a)github.com>

3 6

git:master:c80cccd9: presence: No autocommit+rollback if no active watchers
by Victor Seva 12 Jul '22

12 Jul '22

Module: kamailio Branch: master Commit: c80cccd996dcc3b91d4f0f68a2de51104a16d1a5 URL: https://github.com/kamailio/kamailio/commit/c80cccd996dcc3b91d4f0f68a2de511… Author: Alessio Garzi <agarzi(a)sipwise.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2022-07-12T15:39:33+02:00 presence: No autocommit+rollback if no active watchers - In case the active_watcher query returns no elements there is a weird situation where kamailio first runs the select for active_watcher then rollbacks. This can happen a lot of times for each second since function process_dialogs() runs repeatedly. For this reason trying to avoid the rollback can result in a quite good performance boost. --- Modified: src/modules/presence/notify.c --- Diff: https://github.com/kamailio/kamailio/commit/c80cccd996dcc3b91d4f0f68a2de511… Patch: https://github.com/kamailio/kamailio/commit/c80cccd996dcc3b91d4f0f68a2de511… --- diff --git a/src/modules/presence/notify.c b/src/modules/presence/notify.c index cf52c21c22..4980dc4b0a 100644 --- a/src/modules/presence/notify.c +++ b/src/modules/presence/notify.c @@ -2867,6 +2867,7 @@ int process_dialogs(int round, int presence_winfo) str ev_sname, winfo = str_init("presence.winfo"); int now = (int)time(NULL); int updated = 0; + int no_active_watchers = 0; db_query_f query_fn = pa_dbf.query_lock ? pa_dbf.query_lock : pa_dbf.query; query_cols[n_query_cols] = &str_updated_col; @@ -2900,13 +2901,6 @@ int process_dialogs(int round, int presence_winfo) goto error; } - if(pa_dbf.start_transaction) { - if(pa_dbf.start_transaction(pa_db, pres_db_table_lock) < 0) { - LM_ERR("in start_transaction\n"); - goto error; - } - } - /* Step 1: Find active_watchers that require notification */ if(query_fn(pa_db, query_cols, query_ops, query_vals, result_cols, n_query_cols, n_result_cols, 0, &dialog_list) @@ -2920,7 +2914,17 @@ int process_dialogs(int round, int presence_winfo) } if(dialog_list->n <= 0) + { + no_active_watchers = 1; goto done; + } + + if(pa_dbf.start_transaction) { + if(pa_dbf.start_transaction(pa_db, pres_db_table_lock) < 0) { + LM_ERR("in start_transaction\n"); + goto error; + } + } /* Step 2: Update the records so they are not notified again */ if(pa_dbf.update(pa_db, query_cols, query_ops, query_vals, update_cols, @@ -3164,7 +3168,7 @@ int process_dialogs(int round, int presence_winfo) if(dialog) pa_dbf.free_result(pa_db, dialog); - if(pa_dbf.abort_transaction) { + if(no_active_watchers == 0 && pa_dbf.abort_transaction) { if(pa_dbf.abort_transaction(pa_db) < 0) LM_ERR("in abort_transaction\n"); }

1 0

[kamailio/kamailio] modify 'rpc uac.reg_add' (Issue #3179)
by hance2010 12 Jul '22

12 Jul '22

### Description  rpc uac.reg_add Use a dot (.) if no value should be set for auth_password, auth_ha1, contact_addr, realm,or socket ### Expected behavior https://github.com/kamailio/kamailio/blob/master/src/modules/uac/uac_reg.c#… ``` if(reg.realm.len==1 && reg.realm.s[0] == '.') { reg.realm.s = NULL; reg.realm.len = 0; } if(reg.socket.len==1 && reg.socket.s[0] == '.') { reg.socket.s = NULL; reg.socket.len = 0; } ``` #### Actual observed behavior #### Debugging Data ``` (paste your debugging data here) ``` #### Log Messages  ``` (paste your log messages here) ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` Master ``` * **Operating System**:  ``` Debian11 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3179 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3179(a)github.com>

2 3

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev