sr-dev

sr-dev@lists.kamailio.org

8 participants
45206 discussions

git:master:f6c68817: Merge pull request #2304 from kamailio/grumvalski/usrloc_stats
by GitHub 02 May '20

02 May '20

Module: kamailio Branch: master Commit: f6c688178902138dff8115cdabd72e8631a635dc URL: https://github.com/kamailio/kamailio/commit/f6c688178902138dff8115cdabd72e8… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2020-05-02T16:22:06+02:00 Merge pull request #2304 from kamailio/grumvalski/usrloc_stats usrloc: change "-" for "_" in stats name to be prometheus compliant --- Modified: src/core/cfg.lex Modified: src/core/cfg.y Modified: src/core/counters.c Modified: src/core/counters.h Modified: src/core/globals.h Modified: src/modules/ims_usrloc_pcscf/udomain.c Modified: src/modules/p_usrloc/udomain.c Modified: src/modules/usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/f6c688178902138dff8115cdabd72e8… Patch: https://github.com/kamailio/kamailio/commit/f6c688178902138dff8115cdabd72e8…

1 0

git:master:9948696f: core: remove stats_name_separator from cfg_group_core
by Federico Cabiddu 02 May '20

02 May '20

Module: kamailio Branch: master Commit: 9948696fd84605919e6ce6a7106099f1cdd0de98 URL: https://github.com/kamailio/kamailio/commit/9948696fd84605919e6ce6a7106099f… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Federico Cabiddu <federico.cabiddu(a)gmail.com> Date: 2020-05-02T10:14:23+02:00 core: remove stats_name_separator from cfg_group_core --- Modified: src/core/cfg.y Modified: src/core/cfg_core.c Modified: src/core/cfg_core.h Modified: src/core/config.h Modified: src/core/counters.c Modified: src/core/counters.h Modified: src/core/globals.h Modified: src/modules/ims_usrloc_pcscf/udomain.c Modified: src/modules/p_usrloc/udomain.c Modified: src/modules/usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/9948696fd84605919e6ce6a7106099f… Patch: https://github.com/kamailio/kamailio/commit/9948696fd84605919e6ce6a7106099f…

1 0

git:master:0595795c: ims_usrloc_pcscf, p_usrloc, usrloc: use KSR_STATS_NAMESEP for stats names
by Federico Cabiddu 02 May '20

02 May '20

Module: kamailio Branch: master Commit: 0595795c659f6af586e63837efe56b45808cd735 URL: https://github.com/kamailio/kamailio/commit/0595795c659f6af586e63837efe56b4… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Federico Cabiddu <federico.cabiddu(a)gmail.com> Date: 2020-04-28T16:24:24+02:00 ims_usrloc_pcscf, p_usrloc, usrloc: use KSR_STATS_NAMESEP for stats names --- Modified: src/modules/ims_usrloc_pcscf/udomain.c Modified: src/modules/p_usrloc/udomain.c Modified: src/modules/usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/0595795c659f6af586e63837efe56b4… Patch: https://github.com/kamailio/kamailio/commit/0595795c659f6af586e63837efe56b4… --- diff --git a/src/modules/ims_usrloc_pcscf/udomain.c b/src/modules/ims_usrloc_pcscf/udomain.c index 7f63932e6b..bd5fabdb60 100644 --- a/src/modules/ims_usrloc_pcscf/udomain.c +++ b/src/modules/ims_usrloc_pcscf/udomain.c @@ -82,7 +82,7 @@ static char *build_stat_name( str* domain, char *var_name) } memcpy( s, domain->s, domain->len); p = s + domain->len; - *(p++) = '_'; + *(p++) = KSR_STATS_NAMESEP; memcpy( p , var_name, strlen(var_name)); p += strlen(var_name); *(p++) = 0; diff --git a/src/modules/p_usrloc/udomain.c b/src/modules/p_usrloc/udomain.c index 88612e957b..91db944a0c 100644 --- a/src/modules/p_usrloc/udomain.c +++ b/src/modules/p_usrloc/udomain.c @@ -55,7 +55,7 @@ static char *build_stat_name( str* domain, char *var_name) } memcpy( s, domain->s, domain->len); p = s + domain->len; - *(p++) = '_'; + *(p++) = KSR_STATS_NAMESEP; memcpy( p , var_name, strlen(var_name)); p += strlen(var_name); *(p++) = 0; diff --git a/src/modules/usrloc/udomain.c b/src/modules/usrloc/udomain.c index 321b72b64a..9f5e78cfd6 100644 --- a/src/modules/usrloc/udomain.c +++ b/src/modules/usrloc/udomain.c @@ -61,7 +61,7 @@ static char *build_stat_name( str* domain, char *var_name) } memcpy( s, domain->s, domain->len); p = s + domain->len; - *(p++) = '_'; + *(p++) = KSR_STATS_NAMESEP; memcpy( p , var_name, strlen(var_name)); p += strlen(var_name); *(p++) = 0;

1 0

git:master:3203a657: core: new core parameter stats_name_separator
by Federico Cabiddu 02 May '20

02 May '20

Module: kamailio Branch: master Commit: 3203a6572b874d191c3be61519853005bd5ab056 URL: https://github.com/kamailio/kamailio/commit/3203a6572b874d191c3be6151985300… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Federico Cabiddu <federico.cabiddu(a)gmail.com> Date: 2020-05-01T13:02:39+02:00 core: new core parameter stats_name_separator --- Modified: src/core/cfg.lex Modified: src/core/cfg.y Modified: src/core/cfg_core.c Modified: src/core/cfg_core.h Modified: src/core/config.h Modified: src/modules/ims_usrloc_pcscf/udomain.c Modified: src/modules/p_usrloc/udomain.c Modified: src/modules/usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/3203a6572b874d191c3be6151985300… Patch: https://github.com/kamailio/kamailio/commit/3203a6572b874d191c3be6151985300…

1 0

git:master:4357b5e6: core: define KSR_STATS_NAMESEP for stats name separator
by Federico Cabiddu 02 May '20

02 May '20

Module: kamailio Branch: master Commit: 4357b5e6be2a71a401e9d53c43e787b9bfcc8ed0 URL: https://github.com/kamailio/kamailio/commit/4357b5e6be2a71a401e9d53c43e787b… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Federico Cabiddu <federico.cabiddu(a)gmail.com> Date: 2020-04-28T16:22:40+02:00 core: define KSR_STATS_NAMESEP for stats name separator --- Modified: src/core/config.h --- Diff: https://github.com/kamailio/kamailio/commit/4357b5e6be2a71a401e9d53c43e787b… Patch: https://github.com/kamailio/kamailio/commit/4357b5e6be2a71a401e9d53c43e787b… --- diff --git a/src/core/config.h b/src/core/config.h index 63ff8dc26c..6af3dfecfd 100644 --- a/src/core/config.h +++ b/src/core/config.h @@ -211,4 +211,6 @@ #define DEFAULT_MAX_WHILE_LOOPS 100 /*!< Maximum allowed iterations for a while (to catch runaways) */ +#define KSR_STATS_NAMESEP '_' + #endif

1 0

git:master:ddd27713: usrloc: change "-" for "_" in stats name to be prometheus compliant
by Federico Cabiddu 02 May '20

02 May '20

Module: kamailio Branch: master Commit: ddd277139165f39406832ddd557c2b56b75a7100 URL: https://github.com/kamailio/kamailio/commit/ddd277139165f39406832ddd557c2b5… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Federico Cabiddu <federico.cabiddu(a)gmail.com> Date: 2020-04-28T10:24:04+02:00 usrloc: change "-" for "_" in stats name to be prometheus compliant --- Modified: src/modules/usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/ddd277139165f39406832ddd557c2b5… Patch: https://github.com/kamailio/kamailio/commit/ddd277139165f39406832ddd557c2b5… --- diff --git a/src/modules/usrloc/udomain.c b/src/modules/usrloc/udomain.c index 59ddff92d4..321b72b64a 100644 --- a/src/modules/usrloc/udomain.c +++ b/src/modules/usrloc/udomain.c @@ -61,7 +61,7 @@ static char *build_stat_name( str* domain, char *var_name) } memcpy( s, domain->s, domain->len); p = s + domain->len; - *(p++) = '-'; + *(p++) = '_'; memcpy( p , var_name, strlen(var_name)); p += strlen(var_name); *(p++) = 0;

1 0

git:master:2fe0378c: p_usrloc, ims_usrloc_pcscf: change "-" for "_" in stats name to be prometheus compliant
by Federico Cabiddu 02 May '20

02 May '20

Module: kamailio Branch: master Commit: 2fe0378c3d0596e30d8ec9ea4cb7d828321d2030 URL: https://github.com/kamailio/kamailio/commit/2fe0378c3d0596e30d8ec9ea4cb7d82… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Federico Cabiddu <federico.cabiddu(a)gmail.com> Date: 2020-04-28T14:48:59+02:00 p_usrloc, ims_usrloc_pcscf: change "-" for "_" in stats name to be prometheus compliant --- Modified: src/modules/ims_usrloc_pcscf/udomain.c Modified: src/modules/p_usrloc/udomain.c --- Diff: https://github.com/kamailio/kamailio/commit/2fe0378c3d0596e30d8ec9ea4cb7d82… Patch: https://github.com/kamailio/kamailio/commit/2fe0378c3d0596e30d8ec9ea4cb7d82… --- diff --git a/src/modules/ims_usrloc_pcscf/udomain.c b/src/modules/ims_usrloc_pcscf/udomain.c index 1be609ad42..7f63932e6b 100644 --- a/src/modules/ims_usrloc_pcscf/udomain.c +++ b/src/modules/ims_usrloc_pcscf/udomain.c @@ -82,7 +82,7 @@ static char *build_stat_name( str* domain, char *var_name) } memcpy( s, domain->s, domain->len); p = s + domain->len; - *(p++) = '-'; + *(p++) = '_'; memcpy( p , var_name, strlen(var_name)); p += strlen(var_name); *(p++) = 0; diff --git a/src/modules/p_usrloc/udomain.c b/src/modules/p_usrloc/udomain.c index c25f1c6253..88612e957b 100644 --- a/src/modules/p_usrloc/udomain.c +++ b/src/modules/p_usrloc/udomain.c @@ -55,7 +55,7 @@ static char *build_stat_name( str* domain, char *var_name) } memcpy( s, domain->s, domain->len); p = s + domain->len; - *(p++) = '-'; + *(p++) = '_'; memcpy( p , var_name, strlen(var_name)); p += strlen(var_name); *(p++) = 0;

1 0

json request fails
by Juha Heinanen 02 May '20

02 May '20

I noticed that json rpc commands have started to fail, for example, May 2 15:41:15 char /usr/bin/sip-proxy[10440]: INFO: Executing JSON request <{"jsonrpc":"2.0","method":"siptrace.status","params":["on"],"id":1}> from <127.0.0.1> with host <127.0.0.1:6060> May 2 15:41:15 char /usr/bin/sip-proxy[10440]: ERROR: <core> [core/parser/parse_from.c:53]: parse_from_header(): bad msg or missing FROM header May 2 15:41:15 char /usr/bin/sip-proxy[10440]: ERROR: siptrace [siptrace_send.c:55]: sip_trace_prepare(): cannot parse FROM header Has something changed that I have missed? I didn't find anything related on the wiki upgrade page. -- Juha

2 4

git:master:c840fa6e: siptrace: skip non-sip traffic for db storage using core callbacks
by Daniel-Constantin Mierla 02 May '20

02 May '20

Module: kamailio Branch: master Commit: c840fa6eb19b4b5d316e36ce534f4d481805ee0b URL: https://github.com/kamailio/kamailio/commit/c840fa6eb19b4b5d316e36ce534f4d4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-02T15:02:55+02:00 siptrace: skip non-sip traffic for db storage using core callbacks --- Modified: src/modules/siptrace/siptrace.c --- Diff: https://github.com/kamailio/kamailio/commit/c840fa6eb19b4b5d316e36ce534f4d4… Patch: https://github.com/kamailio/kamailio/commit/c840fa6eb19b4b5d316e36ce534f4d4… --- diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c index 5b4c064bec..6cc3471fcf 100644 --- a/src/modules/siptrace/siptrace.c +++ b/src/modules/siptrace/siptrace.c @@ -2077,6 +2077,24 @@ int siptrace_net_data_recv(sr_event_param_t *evp) goto afterdb; } + if(tmsg.first_line.type==SIP_REQUEST) { + if(!IS_SIP(&tmsg)) { + LM_DBG("non sip request message\n"); + free_sip_msg(&tmsg); + goto afterdb; + } + } else if(tmsg.first_line.type==SIP_REPLY) { + if(!IS_SIP_REPLY(&tmsg)) { + LM_DBG("non sip reply message\n"); + free_sip_msg(&tmsg); + goto afterdb; + } + } else { + LM_DBG("unknown sip message type %d\n", tmsg.first_line.type); + free_sip_msg(&tmsg); + goto afterdb; + } + if(sip_trace_msg_attrs(&tmsg, &sto) < 0) { free_sip_msg(&tmsg); goto afterdb; @@ -2179,6 +2197,24 @@ int siptrace_net_data_send(sr_event_param_t *evp) goto afterdb; } + if(tmsg.first_line.type==SIP_REQUEST) { + if(!IS_SIP(&tmsg)) { + LM_DBG("non sip request message\n"); + free_sip_msg(&tmsg); + goto afterdb; + } + } else if(tmsg.first_line.type==SIP_REPLY) { + if(!IS_SIP_REPLY(&tmsg)) { + LM_DBG("non sip reply message\n"); + free_sip_msg(&tmsg); + goto afterdb; + } + } else { + LM_DBG("unknown sip message type %d\n", tmsg.first_line.type); + free_sip_msg(&tmsg); + goto afterdb; + } + if(sip_trace_msg_attrs(&tmsg, &sto) < 0) { free_sip_msg(&tmsg); goto afterdb;

1 0

undefined symbol: ul_cseq_delay
by Juha Heinanen 02 May '20

02 May '20

I build my sip proxy from today's master and it didn't start: May 2 15:12:51 char sip-proxy[22422]: 0(22474) ERROR: <core> [core/sr_module.c:512]: load_module(): could not open module </usr/lib/x86_64-linux-gnu/sip-proxy/modules/usrloc.so>: /usr/lib/x86_64-linux-gnu/sip-proxy/modules/usrloc.so: undefined symbol: ul_cseq_delay -- Juha

2 2

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev