sr-dev October 2015

sr-dev@lists.kamailio.org

28 participants
236 discussions

git:master:84d75d43: evapi fix some minor typos in debug/error messages

by Olle E. Johansson

Module: kamailio Branch: master Commit: 84d75d43c17b4773bb15a80c1a1f33a47424d858 URL: https://github.com/kamailio/kamailio/commit/84d75d43c17b4773bb15a80c1a1f33a… Author: Olle E. Johansson <oej(a)edvina.net> Committer: Olle E. Johansson <oej(a)edvina.net> Date: 2015-10-20T20:52:10+02:00 evapi fix some minor typos in debug/error messages --- Modified: modules/evapi/evapi_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/84d75d43c17b4773bb15a80c1a1f33a… Patch: https://github.com/kamailio/kamailio/commit/84d75d43c17b4773bb15a80c1a1f33a… --- diff --git a/modules/evapi/evapi_mod.c b/modules/evapi/evapi_mod.c index eeafc21..146c9d2 100644 --- a/modules/evapi/evapi_mod.c +++ b/modules/evapi/evapi_mod.c @@ -114,12 +114,12 @@ static int mod_init(void) /* init faked sip msg */ if(faked_msg_init()<0) { - LM_ERR("failed to init faked sip msg\n"); + LM_ERR("failed to init faked sip message\n"); return -1; } if(load_tm_api( &tmb ) < 0) { - LM_INFO("cannot load the TM-functions - async relay disabled\n"); + LM_INFO("cannot load the TM module functions - async relay disabled\n"); memset(&tmb, 0, sizeof(tm_api_t)); } @@ -180,7 +180,7 @@ static int child_init(int rank) /* main function for dispatcher */ evapi_close_notify_sockets_child(); if(evapi_run_dispatcher(_evapi_bind_addr, _evapi_bind_port)<0) { - LM_ERR("failed to initialize disptacher process\n"); + LM_ERR("failed to initialize evapi dispatcher process\n"); return -1; } } @@ -276,7 +276,7 @@ static int w_evapi_async_relay(sip_msg_t *msg, char *evdata, char *p2) } if(tmb.t_suspend(msg, &tindex, &tlabel)<0) { - LM_ERR("failed to suppend request processing\n"); + LM_ERR("failed to suspend request processing\n"); return -1; }

9 years, 1 month

git:master:b6d83953: curl Fix connection ID handling, content-type for post data and more

by Olle E. Johansson

Module: kamailio Branch: master Commit: b6d83953571c17001e96859606a18c91c6ee5c9a URL: https://github.com/kamailio/kamailio/commit/b6d83953571c17001e96859606a18c9… Author: Olle E. Johansson <oej(a)edvina.net> Committer: Olle E. Johansson <oej(a)edvina.net> Date: 2015-10-20T20:48:47+02:00 curl Fix connection ID handling, content-type for post data and more --- Modified: modules/curl/curl.c Modified: modules/curl/curlcon.c Modified: modules/curl/functions.c --- Diff: https://github.com/kamailio/kamailio/commit/b6d83953571c17001e96859606a18c9… Patch: https://github.com/kamailio/kamailio/commit/b6d83953571c17001e96859606a18c9…

9 years, 1 month

git:4.3:f699719f: tls: init tls hooks to {0}, avoiding fields enumeration

by Daniel-Constantin Mierla

Module: kamailio Branch: 4.3 Commit: f699719fed8b1fffc653c272c01941730dab6d3e URL: https://github.com/kamailio/kamailio/commit/f699719fed8b1fffc653c272c019417… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-10-20T20:27:30+02:00 tls: init tls hooks to {0}, avoiding fields enumeration - if a new field is added, then should be no longer needed to update initialization (cherry picked from commit 92c7656a37549de0eed3621ec243022400f6c6c4) --- Modified: tls_hooks.c --- Diff: https://github.com/kamailio/kamailio/commit/f699719fed8b1fffc653c272c019417… Patch: https://github.com/kamailio/kamailio/commit/f699719fed8b1fffc653c272c019417… --- diff --git a/tls_hooks.c b/tls_hooks.c index 92e709f..b46a429 100644 --- a/tls_hooks.c +++ b/tls_hooks.c @@ -28,7 +28,7 @@ #ifdef TLS_HOOKS -struct tls_hooks tls_hook= {0,0,0,0,0,0,0,0}; +struct tls_hooks tls_hook= {0}; static int tls_hooks_loaded=0;

9 years, 1 month

git:master:92c7656a: tls: init tls hooks to {0}, avoiding fields enumeration

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 92c7656a37549de0eed3621ec243022400f6c6c4 URL: https://github.com/kamailio/kamailio/commit/92c7656a37549de0eed3621ec243022… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-10-20T19:44:47+02:00 tls: init tls hooks to {0}, avoiding fields enumeration - if a new field is added, then should be no longer needed to update initialization --- Modified: tls_hooks.c --- Diff: https://github.com/kamailio/kamailio/commit/92c7656a37549de0eed3621ec243022… Patch: https://github.com/kamailio/kamailio/commit/92c7656a37549de0eed3621ec243022… --- diff --git a/tls_hooks.c b/tls_hooks.c index 92e709f..b46a429 100644 --- a/tls_hooks.c +++ b/tls_hooks.c @@ -28,7 +28,7 @@ #ifdef TLS_HOOKS -struct tls_hooks tls_hook= {0,0,0,0,0,0,0,0}; +struct tls_hooks tls_hook= {0}; static int tls_hooks_loaded=0;

9 years, 1 month

Two active kamailio instance with single HA Proxy

by kaushik parmar

Hello All, I have One HA proxy server which have public ip. HA proxy server can load balance between two kamailio instances installed in Server1 and Server2. How to configure kamailio server such that If user registered on Server2 then server1 will redirect INVITE to server2. user1 --> server1 --> server2 --> user2 Can anyone please help please? -- Thanks & Regards Kaushik Parmar

9 years, 1 month

git:master:7db9bfea: Makefile.groups: added systemd group

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 7db9bfea7d03b50ba94ccc58cffa6d8d049e9515 URL: https://github.com/kamailio/kamailio/commit/7db9bfea7d03b50ba94ccc58cffa6d8… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-10-20T11:54:11+02:00 Makefile.groups: added systemd group - it includes the log_systemd module --- Modified: Makefile.groups --- Diff: https://github.com/kamailio/kamailio/commit/7db9bfea7d03b50ba94ccc58cffa6d8… Patch: https://github.com/kamailio/kamailio/commit/7db9bfea7d03b50ba94ccc58cffa6d8… --- diff --git a/Makefile.groups b/Makefile.groups index c0a8a4e..880753d 100644 --- a/Makefile.groups +++ b/Makefile.groups @@ -183,6 +183,9 @@ mod_list_cnxcc=cnxcc # - modules depending on erlang library mod_list_erlang=erlang +# - modules depending on systemd library +mod_list_systemd=log_systemd + # - all modules mod_list_all=$(sort $(mod_list_basic) $(mod_list_extra) \ $(mod_list_db) $(mod_list_dbuid) \ @@ -209,7 +212,7 @@ mod_list_all=$(sort $(mod_list_basic) $(mod_list_extra) \ $(mod_list_ev) $(mod_list_kazoo) \ $(mod_list_mongodb) $(mod_list_cnxcc) \ $(mod_list_jansson) $(mod_list_geoip2) \ - $(mod_list_erlang)) + $(mod_list_erlang) $(mod_list_systemd)) @@ -394,6 +397,9 @@ module_group_kcnxcc=$(mod_list_cnxcc) # pkg erlang module module_group_kerlang=$(mod_list_erlang) +# pkg systemd module +module_group_ksystemd=$(mod_list_systemd) + # list of static modules # static_modules:=

9 years, 1 month

git:master:14989708: log_systemd: module to provide logging to systemd journal from config file

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 14989708b551081380a96c25ece225e3db4d99a1 URL: https://github.com/kamailio/kamailio/commit/14989708b551081380a96c25ece225e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-10-20T11:51:29+02:00 log_systemd: module to provide logging to systemd journal from config file --- Added: modules/log_systemd/Makefile Added: modules/log_systemd/README Added: modules/log_systemd/doc/Makefile Added: modules/log_systemd/doc/log_systemd.xml Added: modules/log_systemd/doc/log_systemd_admin.xml Added: modules/log_systemd/log_systemd_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/14989708b551081380a96c25ece225e… Patch: https://github.com/kamailio/kamailio/commit/14989708b551081380a96c25ece225e…

9 years, 1 month

[kamailio] nathelper sending keep alives on nodes where contact replicated using dmq_usrloc (#310)

by 00Asgaroth00

I have 3 registrars being replicated to each other using dmq_usrloc. These registrars sit behind proxy servers. I use the path header to specify the received address. When the nathelper module is enabled, the nathelper then sends keep alive options messages from all three registrars for the same contact instead of the node that processed the original registration. I am using in-memory replication (db_mode = 0), so the server_id core parameter does not seem to be taking affect for in-memory replication, i believe this is only for database mode only. I currently have server_id set differently for each registrar, to no effect. nathelper module configuration parameters are as follows: ``` modparam("nathelper", "natping_interval", 20) modparam("nathelper", "natping_processes", 4) modparam("nathelper", "ping_nated_only", 1) modparam("nathelper", "sipping_from", "sip:keepalive@domain.com") modparam("nathelper", "sipping_method", "OPTIONS") modparam("nathelper", "force_socket", NAT_PING_SOCKET) ``` NAT_PING_SOCKET is set to the local interface of each registrar where I want the message to originate from: ``` # Registrar01 #!define NAT_PING_SOCKET "10.7.0.170:5060" # Registrar02 #!define NAT_PING_SOCKET "10.7.0.171:5060" # Registrar03 #!define NAT_PING_SOCKET "10.7.0.172:5060" ``` server_id is set for each registrar ``` # Registrar01 server_id=001 # Registrar02 server_id=002 # Registrar03 server_id=003 ``` I noticed that when the force_socket is set and a registration is sent from a node that did not process the request, then an options messages is sent via the forced_socket but the originating IP is that of the primary interface of the node, in this case, the kamailio listening address on the 10.6.0.0/24 segment, and example message follows: ``` 12:42:13.320519 00:50:56:91:5e:77 > 00:50:56:91:64:fe, ethertype IPv4 (0x0800), length 455: (tos 0x60, ttl 64, id 62009, offset 0, flags [none], proto UDP (17), length 441) 10.6.0.170.sip > 10.7.0.109.sip: [udp sum ok] SIP, length: 413 OPTIONS sip:subscriber01@172.16.0.138:5060;alias=78.143.152.30~49233~1 SIP/2.0 Via: SIP/2.0/UDP 10.6.0.170:5060;branch=0 Route: <sip:10.7.0.109;lr;received=sip:78.143.152.30:49233> From: sip:keepalive@domain.com;tag=uloc-3-55e0473b-1503-af38-6cf5fbae-864c1976 To: sip:subscriber01@172.16.0.138:5060;alias=78.143.152.30~49233~1 Call-ID: 9f855711-a24ef866-112a3d5(a)10.6.0.170 CSeq: 1 OPTIONS Content-Length: 0 ``` The MAC address above confirms that the sending interface is that of force_socket for nathelper, but the source IP address is that of the primary interface of the registrar. This results in martian source messages on the target proxy server: ``` Aug 31 12:42:13 proxy01 kernel: martian source 10.7.0.109 from 10.6.0.170, on dev eth1 Aug 31 12:42:13 proxy01 kernel: ll header: 00:50:56:91:64:fe:00:50:56:91:5e:77:08:00 ``` I suspect the latter issue (martian source) will resolve itself if the registrar not processing the registration stops sending nathelper keep alive messages for contacts it did not itself process. Please let me know if you require any additional information from me to debug this issue further. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/310

9 years, 1 month

git:master:974abf6a: dialog: setting dlg_flag parameter is no longer mandatory

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 974abf6ac02785af5781b67e135e25a2b1f47949 URL: https://github.com/kamailio/kamailio/commit/974abf6ac02785af5781b67e135e25a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-10-19T16:43:09+02:00 dialog: setting dlg_flag parameter is no longer mandatory - dlg_manage() can be used to start tracking the dialog - using flags can still be used, the parameter has to be set as well as the appropriate transaction flag --- Modified: modules/dialog/dialog.c Modified: modules/dialog/dlg_handlers.c --- Diff: https://github.com/kamailio/kamailio/commit/974abf6ac02785af5781b67e135e25a… Patch: https://github.com/kamailio/kamailio/commit/974abf6ac02785af5781b67e135e25a… --- diff --git a/modules/dialog/dialog.c b/modules/dialog/dialog.c index b6c4d5c..553715d 100644 --- a/modules/dialog/dialog.c +++ b/modules/dialog/dialog.c @@ -496,10 +496,7 @@ static int mod_init(void) return -1; /* param checkings */ - if (dlg_flag==-1) { - LM_ERR("no dlg flag set!!\n"); - return -1; - } else if (dlg_flag>MAX_FLAG) { + if (dlg_flag>MAX_FLAG) { LM_ERR("invalid dlg flag %d!!\n",dlg_flag); return -1; } diff --git a/modules/dialog/dlg_handlers.c b/modules/dialog/dlg_handlers.c index ef7a336..f9a16ce 100644 --- a/modules/dialog/dlg_handlers.c +++ b/modules/dialog/dlg_handlers.c @@ -55,7 +55,7 @@ #include "dlg_var.h" static str rr_param; /*!< record-route parameter for matching */ -static int dlg_flag; /*!< flag for dialog tracking */ +static int dlg_flag_mask=0; /*!< flag for dialog tracking */ static pv_spec_t *timeout_avp; /*!< AVP for timeout setting */ static int default_timeout; /*!< default dialog timeout */ static int seq_match_mode; /*!< dlg_match mode */ @@ -107,7 +107,7 @@ void init_dlg_handlers(char *rr_param_p, int dlg_flag_p, rr_param.s = rr_param_p; rr_param.len = strlen(rr_param.s); - dlg_flag = 1<<dlg_flag_p; + if(dlg_flag_p>=0) dlg_flag_mask = 1<<dlg_flag_p; timeout_avp = timeout_avp_p; default_timeout = default_timeout_p; @@ -711,7 +711,7 @@ void dlg_onreq(struct cell* t, int type, struct tmcb_params *param) } } if (dlg==NULL) { - if((req->flags&dlg_flag)!=dlg_flag) + if((req->flags&dlg_flag_mask)!=dlg_flag_mask) return; LM_DBG("dialog creation on config flag\n"); dlg_new_dialog(req, t, 1);

9 years, 1 month

[kamailio] Issue in acc module in 4.3? (#350)

by mtryfoss

I've found some strange cdr records produced by kamailio. Haven't experienced this before in earlier versions. One example (last field is kamailios call duration): 1442923048|1442923184|136298900000.000000 The duration of this call is in reality ~136 seconds. This matches the first three digits of the duration-field. Is this just a coincidence? Another example: 1443011985|1443012286|301148.400000 This call has a duration of 301 seconds. Same as the first three digits. I haven't found anything special with these calls, and it doesn't seem possible to reproduce. Just happens sometimes. Settings for acc module: modparam("acc", "db_url", "flatstore:/accounting") modparam("acc", "log_level", 3) modparam("acc", "cdr_enable", 1) modparam("acc", "cdr_start_on_confirmed", 1) modparam("acc", "cdr_extra", "anum=$dlg_var(caller);bnum=$dlg_var(callee);dlg_start_time=$dlg_var(inv_start);cancel_time=$dlg_var(cancel);account=$dlg_var(account);calltype=$dlg_var(calltype);callid=$dlg_var(callid);callid_system=$dlg_var(callid_system);trunk_id=$dlg_var(trunk_id)") modparam("dialog", "dlg_flag", 4) modparam("acc", "cdrs_table", "cdr") modparam("acc", "cdr_on_failed", 0) --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/350

9 years, 1 month

← Newer
1
...
8
9
10
11
12
13
14
...
24
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev October 2015