sr-dev August 2019

sr-dev@lists.kamailio.org

17 participants
339 discussions

[kamailio/kamailio] to tags generation (#1164)
by Dmitri Savolainen 02 Sep '19

02 Sep '19

### Description Not shure is this relevant issue. according to rfc3261: > When a tag is generated by a UA for insertion into a request or > response, it MUST be globally unique and cryptographically random > with at least 32 bits of randomness. Kamailio has 16 instead and distribution is not good. I look at tags for 419485 sip flows (different callid) and table below (top keys ordered by count): tag suffix; count for different flows with this suffix ed54 6258 d118 5988 56ce 5923 a7a5 5220 de51 2150 82d8 1143 cf52 890 af92 740 7b5f 534 85f7 307 e553 163 7f7e 138 1d55 130 1afe 62 5a69 43 7caa 18 8474 18 9db2 18 11d3 18 f9ff 18 5420 17 dae6 17 908d 17 ede4 17 09ee 17 96be 17 25dc 17 af6d 16 2506 16 3da0 16 f161 16 1c29 16 #### SIP Traffic To: <sip:039*101@host:5071>;tag=555d2c3eafafcba8ac92b676e91bb6ed.58eb -- 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/1164

3 4

5.3 add rr record_route_preset to KEMI?
by Anthony Alba 01 Sep '19

01 Sep '19

Hi Daniel and Kamailio developers, Could you consider adding record_route_preset to KEMI in 5.3? If you like I can do the PR. Hope it is not too late. Regards Anthony Alba

2 1

git:master:8aeca29b: pkg/docker: updated submodule [skip ci]
by Sergey Safarov 31 Aug '19

31 Aug '19

Module: kamailio Branch: master Commit: 8aeca29b0666a6c6fbca7e7488677aad1a45e44e URL: https://github.com/kamailio/kamailio/commit/8aeca29b0666a6c6fbca7e7488677aa… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Sergey Safarov <s.safarov(a)gmail.com> Date: 2019-08-31T13:05:32+03:00 pkg/docker: updated submodule [skip ci] --- Modified: pkg/docker --- Diff: https://github.com/kamailio/kamailio/commit/8aeca29b0666a6c6fbca7e7488677aa… Patch: https://github.com/kamailio/kamailio/commit/8aeca29b0666a6c6fbca7e7488677aa… --- diff --git a/pkg/docker b/pkg/docker index b92a1c899b..5d6f46a758 160000 --- a/pkg/docker +++ b/pkg/docker @@ -1 +1 @@ -Subproject commit b92a1c899b8c474afadd0cf29dd7cb4ea01a3466 +Subproject commit 5d6f46a758e0d98fa4258925d40aa1521b1e27d0

1 0

git:master:8f98a04c: pkg/kamailio/obs: packaged kemix, lost and xhttp_prom modules
by Sergey Safarov 31 Aug '19

31 Aug '19

Module: kamailio Branch: master Commit: 8f98a04c0f9392db6a74289a184d06da11545ab0 URL: https://github.com/kamailio/kamailio/commit/8f98a04c0f9392db6a74289a184d06d… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Sergey Safarov <s.safarov(a)gmail.com> Date: 2019-08-31T13:03:20+03:00 pkg/kamailio/obs: packaged kemix, lost and xhttp_prom modules --- Modified: pkg/kamailio/obs/kamailio.spec --- Diff: https://github.com/kamailio/kamailio/commit/8f98a04c0f9392db6a74289a184d06d… Patch: https://github.com/kamailio/kamailio/commit/8f98a04c0f9392db6a74289a184d06d… --- diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec index 8fd4254eb7..9dec32a876 100644 --- a/pkg/kamailio/obs/kamailio.spec +++ b/pkg/kamailio/obs/kamailio.spec @@ -1425,7 +1425,9 @@ fi %doc %{_docdir}/kamailio/modules/README.htable %doc %{_docdir}/kamailio/modules/README.imc %doc %{_docdir}/kamailio/modules/README.ipops +%doc %{_docdir}/kamailio/modules/README.kemix %doc %{_docdir}/kamailio/modules/README.kex +%doc %{_docdir}/kamailio/modules/README.lost %doc %{_docdir}/kamailio/modules/README.malloc_test %doc %{_docdir}/kamailio/modules/README.mangler %doc %{_docdir}/kamailio/modules/README.matrix @@ -1492,6 +1494,7 @@ fi %doc %{_docdir}/kamailio/modules/README.userblacklist %doc %{_docdir}/kamailio/modules/README.usrloc %doc %{_docdir}/kamailio/modules/README.xhttp +%doc %{_docdir}/kamailio/modules/README.xhttp_prom %doc %{_docdir}/kamailio/modules/README.xhttp_rpc %doc %{_docdir}/kamailio/modules/README.xlog %doc %{_docdir}/kamailio/modules/README.xprint @@ -1576,7 +1579,9 @@ fi %{_libdir}/kamailio/modules/htable.so %{_libdir}/kamailio/modules/imc.so %{_libdir}/kamailio/modules/ipops.so +%{_libdir}/kamailio/modules/kemix.so %{_libdir}/kamailio/modules/kex.so +%{_libdir}/kamailio/modules/lost.so %{_libdir}/kamailio/modules/malloc_test.so %{_libdir}/kamailio/modules/mangler.so %{_libdir}/kamailio/modules/matrix.so @@ -1643,6 +1648,7 @@ fi %{_libdir}/kamailio/modules/userblacklist.so %{_libdir}/kamailio/modules/usrloc.so %{_libdir}/kamailio/modules/xhttp.so +%{_libdir}/kamailio/modules/xhttp_prom.so %{_libdir}/kamailio/modules/xhttp_rpc.so %{_libdir}/kamailio/modules/xlog.so %{_libdir}/kamailio/modules/xprint.so @@ -1875,7 +1881,9 @@ fi %files lua %defattr(-,root,root) %doc %{_docdir}/kamailio/modules/README.app_lua +%doc %{_docdir}/kamailio/modules/README.app_lua_sr %{_libdir}/kamailio/modules/app_lua.so +%{_libdir}/kamailio/modules/app_lua_sr.so %endif @@ -2217,6 +2225,8 @@ fi %changelog +* Sat Aug 31 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-dev7 + - Packaged kemix, lost and xhttp_prom modules * Sat Mar 30 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-0 - Added support of openSUSE:Leap:15.0, openSUSE:Leap:15.1 and Fedora 30 dists * Thu Feb 21 2019 Sergey Safarov <s.safarov(a)gmail.com> 5.3.0-0

1 0

Info: Upgrading kamcli to work with Python3
by Daniel-Constantin Mierla 30 Aug '19

30 Aug '19

Hello, with the perspective that Python2 is going to be removed from the major Linux distributes starting with the beginning of next year, the work to upgrade kamcli to run with Python3 has been started. >From now on, master branch of kamcli is expected to work only with Python3: * https://github.com/kamailio/kamcli There is still work to be done to upgrade all the code, but any use of it should be done with Python3 and eventual issues to be filed on bug tracker. Of course, pull requests are welcome as always! Who wants to use the version for Python2 has to use the code from git branch v1.2-python2. That branch is probably not going to receive any new feature, it will be kept for use with older systems (as well as a running option during the upgrade period to Python3). Cheers, Daniel -- Daniel-Constantin Mierla -- www.asipto.com www.twitter.com/miconda -- www.linkedin.com/in/miconda

1 0

git:master:72a4d665: tm: void the return for sr_kemi_route() to prevent analyzer warnings
by Daniel-Constantin Mierla 30 Aug '19

30 Aug '19

Module: kamailio Branch: master Commit: 72a4d665607d3c38278a744d3af3588ed54d2a05 URL: https://github.com/kamailio/kamailio/commit/72a4d665607d3c38278a744d3af3588… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-08-30T09:09:33+02:00 tm: void the return for sr_kemi_route() to prevent analyzer warnings --- Modified: src/modules/tm/t_reply.c --- Diff: https://github.com/kamailio/kamailio/commit/72a4d665607d3c38278a744d3af3588… Patch: https://github.com/kamailio/kamailio/commit/72a4d665607d3c38278a744d3af3588… --- diff --git a/src/modules/tm/t_reply.c b/src/modules/tm/t_reply.c index 350be35bdd..331bdbdf71 100644 --- a/src/modules/tm/t_reply.c +++ b/src/modules/tm/t_reply.c @@ -593,7 +593,7 @@ static int _reply_light( struct cell *trans, char* buf, unsigned int len, } else { bctx = sr_kemi_act_ctx_get(); sr_kemi_act_ctx_set(&ctx); - sr_kemi_route(keng, &pmsg, EVENT_ROUTE, + (void)sr_kemi_route(keng, &pmsg, EVENT_ROUTE, &_tm_event_callback_lres_sent, &evname); sr_kemi_act_ctx_set(bctx); }

1 0

git:master:932bbdb1: sl: void the return for sr_kemi_route() to prevent analyzer warnings
by Daniel-Constantin Mierla 30 Aug '19

30 Aug '19

Module: kamailio Branch: master Commit: 932bbdb19f48808fe6f40849e4798124ef4da127 URL: https://github.com/kamailio/kamailio/commit/932bbdb19f48808fe6f40849e479812… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-08-30T09:04:43+02:00 sl: void the return for sr_kemi_route() to prevent analyzer warnings --- Modified: src/modules/sl/sl_funcs.c --- Diff: https://github.com/kamailio/kamailio/commit/932bbdb19f48808fe6f40849e479812… Patch: https://github.com/kamailio/kamailio/commit/932bbdb19f48808fe6f40849e479812… --- diff --git a/src/modules/sl/sl_funcs.c b/src/modules/sl/sl_funcs.c index 83cf15bb5d..5e36a5542e 100644 --- a/src/modules/sl/sl_funcs.c +++ b/src/modules/sl/sl_funcs.c @@ -293,7 +293,7 @@ int sl_reply_helper(struct sip_msg *msg, int code, char *reason, str *tag) } else if (keng!=NULL) { bctx = sr_kemi_act_ctx_get(); sr_kemi_act_ctx_set(&ctx); - sr_kemi_route(keng, msg, EVENT_ROUTE, + (void)sr_kemi_route(keng, msg, EVENT_ROUTE, &_sl_event_callback_lres_sent, &evname); sr_kemi_act_ctx_set(bctx); } @@ -448,7 +448,7 @@ int sl_filter_ACK(struct sip_msg *msg, unsigned int flags, void *bar ) init_run_actions_ctx(&ctx); bctx = sr_kemi_act_ctx_get(); sr_kemi_act_ctx_set(&ctx); - sr_kemi_route(keng, msg, EVENT_ROUTE, + (void)sr_kemi_route(keng, msg, EVENT_ROUTE, &_sl_event_callback_fl_ack, &evname); sr_kemi_act_ctx_set(bctx); }

1 0

[kamailio/kamailio] tls_domain subdomain matching broken (#2049)
by Laszlo 29 Aug '19

29 Aug '19

Concerns the following code: https://github.com/kamailio/kamailio/blob/master/src/modules/tls/tls_domain… Let's say `sname = sip.example.com` and `p->server_name = example.com`. ``` dotpos = sname->len - p->server_name.len; > dotpos = len("sip.example.com") - len("example.com") > dotpos = 15 - 11 = 4 if (sname->s[dotpos] == '.') > if ("sip.example.com"[4] == '.') ``` `sip.example.com"[4]` is "e", not "." so the if clause will always fail and subdomains will never be matched. -- 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/2049

2 2

git:master:16beec2b: tls: proper dot position for matching subdomains
by Daniel-Constantin Mierla 29 Aug '19

29 Aug '19

Module: kamailio Branch: master Commit: 16beec2b7660d416a05f069695b4ce18e8b4e795 URL: https://github.com/kamailio/kamailio/commit/16beec2b7660d416a05f069695b4ce1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-08-29T18:51:17+02:00 tls: proper dot position for matching subdomains - reported by GH #2049 --- Modified: src/modules/tls/tls_domain.c --- Diff: https://github.com/kamailio/kamailio/commit/16beec2b7660d416a05f069695b4ce1… Patch: https://github.com/kamailio/kamailio/commit/16beec2b7660d416a05f069695b4ce1… --- diff --git a/src/modules/tls/tls_domain.c b/src/modules/tls/tls_domain.c index fff4603eba..1ca62ff200 100644 --- a/src/modules/tls/tls_domain.c +++ b/src/modules/tls/tls_domain.c @@ -1563,7 +1563,7 @@ tls_domain_t* tls_lookup_cfg(tls_domains_cfg_t* cfg, int type, if ((p->server_name_mode==KSR_TLS_SNM_INCDOM || p->server_name_mode==KSR_TLS_SNM_SUBDOM) && (p->server_name.len<sname->len)) { - dotpos = sname->len - p->server_name.len; + dotpos = sname->len - p->server_name.len - 1; if(sname->s[dotpos] == '.' && strncasecmp(p->server_name.s, sname->s + dotpos + 1,

1 0

git:master:152801ea: sl: docs - set section ids in expected format
by Daniel-Constantin Mierla 29 Aug '19

29 Aug '19

Module: kamailio Branch: master Commit: 152801eaef0d95a6cee027391b809bfe2289ef3d URL: https://github.com/kamailio/kamailio/commit/152801eaef0d95a6cee027391b809bf… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-08-29T14:36:00+02:00 sl: docs - set section ids in expected format --- Modified: pkg/docker Modified: src/modules/sl/doc/sl_functions.xml Modified: src/modules/sl/doc/sl_params.xml --- Diff: https://github.com/kamailio/kamailio/commit/152801eaef0d95a6cee027391b809bf… Patch: https://github.com/kamailio/kamailio/commit/152801eaef0d95a6cee027391b809bf… --- diff --git a/pkg/docker b/pkg/docker index 424756ff7a..b92a1c899b 160000 --- a/pkg/docker +++ b/pkg/docker @@ -1 +1 @@ -Subproject commit 424756ff7aca572466a99059f76d44ce57e01646 +Subproject commit b92a1c899b8c474afadd0cf29dd7cb4ea01a3466 diff --git a/src/modules/sl/doc/sl_functions.xml b/src/modules/sl/doc/sl_functions.xml index bdef94866e..1bb12b76c4 100644 --- a/src/modules/sl/doc/sl_functions.xml +++ b/src/modules/sl/doc/sl_functions.xml @@ -1,12 +1,12 @@ <?xml version="1.0" encoding="UTF-8"?> -<!DOCTYPE section PUBLIC "-//OASIS//DTD DocBook XML V4.2//EN" +<!DOCTYPE section PUBLIC "-//OASIS//DTD DocBook XML V4.2//EN" "http://www.oasis-open.org/docbook/xml/4.2/docbookx.dtd"> <section id="sl.functions" xmlns:xi="http://www.w3.org/2001/XInclude"> <title>Functions</title> - <section id="sl_send_reply"> + <section id="sl.f.sl_send_reply"> <title> <function>sl_send_reply(code, reason)</function> </title> @@ -49,14 +49,14 @@ sl_send_reply("404", "Not found"); </programlisting> </example> </section> - - <section id="send_reply"> + + <section id="sl.f.send_reply"> <title> <function moreinfo="none">send_reply(code, reason)</function> </title> <para> - For the current request, a reply is sent back having the given code - and text reason. The reply is sent stateful or stateless, depending of + For the current request, a reply is sent back having the given code + and text reason. The reply is sent stateful or stateless, depending of the <acronym>TM</acronym> module: if a transaction exists for the current request, then the reply is sent statefully, otherwise stateless. </para> @@ -88,7 +88,7 @@ send_reply("403", "Invalid user - $fU"); </example> </section> - <section id="sl_reply_error"> + <section id="sl.f.sl_reply_error"> <title> <function>sl_reply_error()</function> </title> @@ -107,7 +107,7 @@ sl_reply_error(); </example> </section> - <section id="sl_forward_reply"> + <section id="sl.f.sl_forward_reply"> <title> <function moreinfo="none">sl_forward_reply([ code, [ reason ] ])</function> </title> diff --git a/src/modules/sl/doc/sl_params.xml b/src/modules/sl/doc/sl_params.xml index 6c22e26a90..a6db7657da 100644 --- a/src/modules/sl/doc/sl_params.xml +++ b/src/modules/sl/doc/sl_params.xml @@ -1,12 +1,12 @@ <?xml version="1.0" encoding="UTF-8"?> -<!DOCTYPE section PUBLIC "-//OASIS//DTD DocBook XML V4.2//EN" +<!DOCTYPE section PUBLIC "-//OASIS//DTD DocBook XML V4.2//EN" "http://www.oasis-open.org/docbook/xml/4.2/docbookx.dtd"> <section id="sl.parameters" xmlns:xi="http://www.w3.org/2001/XInclude"> <title>Parameters</title> - <section id="default_code"> + <section id="sl.p.default_code"> <title><varname>default_code</varname> (int)</title> <para> Default reply status code. @@ -24,7 +24,7 @@ modparam("sl", "default_code", 505) </example> </section> - <section id="default_reason"> + <section id="sl.p.default_reason"> <title><varname>default_reason</varname> (str)</title> <para> Default reply reason phrase. @@ -42,7 +42,7 @@ modparam("sl", "default_reason", "Server Error") </example> </section> - <section id="bind_tm"> + <section id="sl.p.bind_tm"> <title><varname>bind_tm</varname> (int)</title> <para> Controls if SL module should attempt to bind to TM module in order @@ -61,7 +61,7 @@ modparam("sl", "bind_tm", 0) # feature disabled </example> </section> - <section id="rich_redirect"> + <section id="sl.p.rich_redirect"> <title><varname>rich_redirect</varname> (int)</title> <para> When sending a 3xx class reply, include additional branch info

1 0

← Newer
1
...
7
8
9
10
11
12
13
...
34
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2019