sr-dev August 2016

sr-dev@lists.kamailio.org

25 participants
197 discussions

[kamailio/kamailio] permissions: fixed rpc array (#755)

by kelchy

- added array structure when returning multiple address and subnets You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/755 -- Commit Summary -- * permissions: fixed rpc array -- File Changes -- M modules/permissions/hash.c (24) M modules/permissions/permissions.c (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/755.patch https://github.com/kamailio/kamailio/pull/755.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/755

7 years, 8 months

T_replicate and tls

by Olle E. Johansson

Hi! I have this interesting scenario that I’m not sure whether it’s a bug or a hidden feature or just a misunderstanding… The incoming request arrives over TLS to Kamailio. It hits t_replicate(“192.168.40.40”, “5060”); Kamailio now tries to set up a TLS connection to port 5060 which we didn’t anticipate, but looking at the script it makes sense (knowing that Kamailio prefers to use the same transport for the outbound message). I will move that code to using a URI with a specific transport - but I wonder if this should be documented somewhere… At least a warning in the docs. I have succeeded rather well in convincing myself that this is not a bug. Am I right? /O

7 years, 8 months

[kamailio/kamailio] Async Routes and Route-Types (#750)

by ng-voice GmbH

Issue: ------------------------------------------ Asynchronous Route Blocks gets executed as a failure_route, even though there never ever was a failure. Version: ------------------------------------------ GIT Trunk (dev), not checked earlier versions. Example Config / how to reproduce ------------------------------------------ I've got the following main route: ``` route { [...] if (is_method("PUBLISH") && search("^(Event|o)([ \t]*):([ \t]*)reg")) { route(PUBLISH); break; } async_sleep("1"); #Set DLG flag to track dialogs using dialog2 if (!is_method("REGISTER|SUBSCRIBE")) setflag(FLT_DIALOG); [...] } ``` With debugger module enabled, it shows as follows in syslog: ``` Aug 17 15:41:09 ubuntu-2gb-fra1-01 [6968]: ERROR: *** cfgtrace:request_route=[DEFAULT_ROUTE] c=[/etc/kamailio/kamailio.cfg] l=397 a=25 n=is_method Aug 17 15:41:09 ubuntu-2gb-fra1-01 [6968]: ERROR: *** cfgtrace:request_route=[DEFAULT_ROUTE] c=[/etc/kamailio/kamailio.cfg] l=407 a=16 n=if Aug 17 15:41:09 ubuntu-2gb-fra1-01 [6968]: ERROR: *** cfgtrace:request_route=[DEFAULT_ROUTE] c=[/etc/kamailio/kamailio.cfg] l=402 a=25 n=is_method Aug 17 15:41:09 ubuntu-2gb-fra1-01 [6968]: ERROR: *** cfgtrace:request_route=[DEFAULT_ROUTE] c=[/etc/kamailio/kamailio.cfg] l=407 a=25 n=async_sleep Aug 17 15:41:09 ubuntu-2gb-fra1-01 [6975]: ERROR: *** cfgtrace:failure_route=[DEFAULT_ROUTE] c=[/etc/kamailio/kamailio.cfg] l=414 a=16 n=if Aug 17 15:41:09 ubuntu-2gb-fra1-01 [6975]: ERROR: *** cfgtrace:failure_route=[DEFAULT_ROUTE] c=[/etc/kamailio/kamailio.cfg] l=410 a=25 n=is_method ``` ------------------------------------------ Why is route_type suddenly a failure-route? At a different part of my logic, this leads to unexpected results. Looking at modules/tm/t_suspend.c (lines 277 ff.), I see the reason: ``` route_type_bk = get_route_type(); set_route_type(FAILURE_ROUTE); /* execute the pre/post -script callbacks based on original route block */ if (exec_pre_script_cb(&faked_req, cb_type)>0) { if (run_top_route(route, &faked_req, 0)<0) LM_ERR("failure inside run_top_route\n"); exec_post_script_cb(&faked_req, cb_type); } set_route_type(route_type_bk); ``` Any particular reason, why the route-type is explicitly set to FAILURE_ROUTE? Thanks, Carsten -- 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/750

7 years, 8 months

[kamailio/kamailio] RPC fix for arrays on permissions module (#751)

by kelchy

subnet dump and address dump You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/751 -- Commit Summary -- * fix rpc array for address * fix subnet rpc array -- File Changes -- M modules/permissions/hash.c (18) -- Patch Links -- https://github.com/kamailio/kamailio/pull/751.patch https://github.com/kamailio/kamailio/pull/751.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/751

7 years, 8 months

git:master:cfb806e4: outbound: use pkg-config for libcrypto in makefile

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: cfb806e41ecde8392105a12c4e8ace31f1345352 URL: https://github.com/kamailio/kamailio/commit/cfb806e41ecde8392105a12c4e8ace3… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-08-19T09:03:44+02:00 outbound: use pkg-config for libcrypto in makefile --- Modified: modules/outbound/Makefile --- Diff: https://github.com/kamailio/kamailio/commit/cfb806e41ecde8392105a12c4e8ace3… Patch: https://github.com/kamailio/kamailio/commit/cfb806e41ecde8392105a12c4e8ace3… --- diff --git a/modules/outbound/Makefile b/modules/outbound/Makefile index df43360..4f6f0f3 100644 --- a/modules/outbound/Makefile +++ b/modules/outbound/Makefile @@ -1,4 +1,4 @@ -# +# # WARNING: do not run this directly, it should be run by the master Makefile include ../../Makefile.defs @@ -6,15 +6,32 @@ auto_gen= NAME=outbound.so ifeq ($(CROSS_COMPILE),) + BUILDER = $(shell which pkg-config) +endif + +ifneq ($(BUILDER),) SSL_BUILDER=$(shell \ if pkg-config --exists libssl; then \ echo 'pkg-config libssl'; \ fi) + +CRYPTO_BUILDER=$(shell \ + if pkg-config --exists libcrypto; then \ + echo 'pkg-config libcrypto'; \ + fi) endif +ifneq ($(BUILDER),) + ifneq ($(SSL_BUILDER),) DEFS += $(shell $(SSL_BUILDER) --cflags) LIBS += $(shell $(SSL_BUILDER) --libs) +endif +ifneq ($(CRYPTO_BUILDER),) + DEFS += $(shell $(CRYPTO_BUILDER) --cflags) + LIBS += $(shell $(CRYPTO_BUILDER) --libs) +endif + else DEFS += -I$(LOCALBASE)/ssl/include LIBS += -L$(LOCALBASE)/lib -L$(LOCALBASE)/ssl/lib \

7 years, 8 months

git:master:d61ff2e4: core: fix some logging in tcp_read.c

by Dragos Oancea

Module: kamailio Branch: master Commit: d61ff2e4cf49479a811019c2f01ace46344cacc2 URL: https://github.com/kamailio/kamailio/commit/d61ff2e4cf49479a811019c2f01ace4… Author: Dragos Oancea <dragos.oancea(a)athonet.com> Committer: Dragos Oancea <dragos.oancea(a)athonet.com> Date: 2016-08-17T11:15:19+02:00 core: fix some logging in tcp_read.c --- Modified: tcp_read.c --- Diff: https://github.com/kamailio/kamailio/commit/d61ff2e4cf49479a811019c2f01ace4… Patch: https://github.com/kamailio/kamailio/commit/d61ff2e4cf49479a811019c2f01ace4… --- diff --git a/tcp_read.c b/tcp_read.c index 499db29..a31d3a2 100644 --- a/tcp_read.c +++ b/tcp_read.c @@ -287,11 +287,10 @@ int tcp_read_data(int fd, struct tcp_connection *c, } } LOG(cfg_get(core, core_cfg, corelog), - "error reading: %s (%d) ([%s]:%u -> [%s]:%u)\n", + "error reading: %s (%d) ([%s]:%u ->", strerror(errno), errno, - ip_addr2a(&c->rcv.src_ip), c->rcv.src_port, - ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); - + ip_addr2a(&c->rcv.src_ip), c->rcv.src_port); + LOG(cfg_get(core, core_cfg, corelog),"-> [%s]:%u)\n", ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); if((errno == ECONNRESET || errno == ETIMEDOUT) && likely(c->rcv.proto_reserved1 != 0)){ tcp_make_closed_event(&c->rcv, c); } @@ -305,9 +304,8 @@ int tcp_read_data(int fd, struct tcp_connection *c, if (likely(c->rcv.proto_reserved1 != 0)){ tcp_make_closed_event(&c->rcv, c); } - LM_DBG("EOF on %p, FD %d ([%s]:%u -> [%s]:%u)\n", c, fd, - ip_addr2a(&c->rcv.src_ip), c->rcv.src_port, - ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); + LM_DBG("EOF on %p, FD %d ([%s]:%u ->", c, fd, ip_addr2a(&c->rcv.src_ip), c->rcv.src_port); + LM_DBG("-> [%s]:%u)\n", ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); }else{ if (unlikely(c->state==S_CONN_CONNECT || c->state==S_CONN_ACCEPT)){ TCP_STATS_ESTABLISHED(c->state);

7 years, 8 months

git:master:ec25a62f: Merge pull request #749 from athonet-open/fix__log_tcp

by GitHub

Module: kamailio Branch: master Commit: ec25a62f059584b5e653e319169f03fc63b25848 URL: https://github.com/kamailio/kamailio/commit/ec25a62f059584b5e653e319169f03f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2016-08-17T11:49:22+02:00 Merge pull request #749 from athonet-open/fix__log_tcp core: fix some logging in tcp_read.c --- Modified: tcp_read.c --- Diff: https://github.com/kamailio/kamailio/commit/ec25a62f059584b5e653e319169f03f… Patch: https://github.com/kamailio/kamailio/commit/ec25a62f059584b5e653e319169f03f… --- diff --git a/tcp_read.c b/tcp_read.c index 499db29..a31d3a2 100644 --- a/tcp_read.c +++ b/tcp_read.c @@ -287,11 +287,10 @@ int tcp_read_data(int fd, struct tcp_connection *c, } } LOG(cfg_get(core, core_cfg, corelog), - "error reading: %s (%d) ([%s]:%u -> [%s]:%u)\n", + "error reading: %s (%d) ([%s]:%u ->", strerror(errno), errno, - ip_addr2a(&c->rcv.src_ip), c->rcv.src_port, - ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); - + ip_addr2a(&c->rcv.src_ip), c->rcv.src_port); + LOG(cfg_get(core, core_cfg, corelog),"-> [%s]:%u)\n", ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); if((errno == ECONNRESET || errno == ETIMEDOUT) && likely(c->rcv.proto_reserved1 != 0)){ tcp_make_closed_event(&c->rcv, c); } @@ -305,9 +304,8 @@ int tcp_read_data(int fd, struct tcp_connection *c, if (likely(c->rcv.proto_reserved1 != 0)){ tcp_make_closed_event(&c->rcv, c); } - LM_DBG("EOF on %p, FD %d ([%s]:%u -> [%s]:%u)\n", c, fd, - ip_addr2a(&c->rcv.src_ip), c->rcv.src_port, - ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); + LM_DBG("EOF on %p, FD %d ([%s]:%u ->", c, fd, ip_addr2a(&c->rcv.src_ip), c->rcv.src_port); + LM_DBG("-> [%s]:%u)\n", ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port); }else{ if (unlikely(c->state==S_CONN_CONNECT || c->state==S_CONN_ACCEPT)){ TCP_STATS_ESTABLISHED(c->state);

7 years, 8 months

[kamailio/kamailio] core: fix some logging in tcp_read.c (#749)

by Dragos Oancea

This patch to emphasize that ip_addr2a() uses a static buffer and because of this lots of log lines in tcp_read.c print the same IP as src and dst . It can be confusing when debugging TCP connections. Practically LM_DBG or LOG print the same buffer twice in these cases. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/749 -- Commit Summary -- * core: fix some logging in tcp_read.c -- File Changes -- M tcp_read.c (12) -- Patch Links -- https://github.com/kamailio/kamailio/pull/749.patch https://github.com/kamailio/kamailio/pull/749.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/749

7 years, 8 months

[kamailio/kamailio] modules/siptrace: fix parsing of unsecure ws uris (#746)

by Sebastian Denz

due to missing adaption of cutlen, the first decimal number of the ip address is cut off when using 'ws:' uris You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/746 -- Commit Summary -- * modules/siptrace: fix parsing of unsecure ws uris -- File Changes -- M modules/siptrace/siptrace.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/746.patch https://github.com/kamailio/kamailio/pull/746.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/746

7 years, 8 months

git:master:efefe2d0: Merge pull request #746 from denzs/master

by GitHub

Module: kamailio Branch: master Commit: efefe2d09367395685e7574dcd9dc12680646d99 URL: https://github.com/kamailio/kamailio/commit/efefe2d09367395685e7574dcd9dc12… Author: Alexandr Dubovikov <alexandr.dubovikov(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2016-08-15T13:20:55+02:00 Merge pull request #746 from denzs/master modules/siptrace: fix parsing of unsecure ws uris --- Modified: modules/siptrace/siptrace.c --- Diff: https://github.com/kamailio/kamailio/commit/efefe2d09367395685e7574dcd9dc12… Patch: https://github.com/kamailio/kamailio/commit/efefe2d09367395685e7574dcd9dc12… --- diff --git a/modules/siptrace/siptrace.c b/modules/siptrace/siptrace.c index 078f1a9..c7ae4e9 100644 --- a/modules/siptrace/siptrace.c +++ b/modules/siptrace/siptrace.c @@ -1854,7 +1854,7 @@ static int pipport2su (char *pipport, union sockaddr_union *tmp_su, unsigned int if(strncmp(pipport, "udp:",4) == 0) *proto = IPPROTO_UDP; else if(strncmp(pipport, "tcp:",4) == 0) *proto = IPPROTO_TCP; else if(strncmp(pipport, "tls:",4) == 0) *proto = IPPROTO_IDP; /* fake proto type */ - else if(strncmp(pipport, "ws:",3) == 0) *proto = IPPROTO_IDP; /* fake proto type */ + else if(strncmp(pipport, "ws:",3) == 0) cutlen = 3, *proto = IPPROTO_IDP; /* fake proto type */ else if(strncmp(pipport, "wss:",4) == 0) *proto = IPPROTO_IDP; /* fake proto type */ #ifdef USE_SCTP else if(strncmp(pipport, "sctp:",5) == 0) cutlen = 5, *proto = IPPROTO_SCTP;

7 years, 8 months

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

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2016