sr-dev March 2021

sr-dev@lists.kamailio.org

28 participants
489 discussions

git:master:0eb57ecf: modules: readme files regenerated - uac ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 0eb57ecfa56e0bf49c2f80291d2287080094340a URL: https://github.com/kamailio/kamailio/commit/0eb57ecfa56e0bf49c2f80291d22870… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2021-03-11T18:46:13+01:00 modules: readme files regenerated - uac ... [skip ci] --- Modified: src/modules/uac/README --- Diff: https://github.com/kamailio/kamailio/commit/0eb57ecfa56e0bf49c2f80291d22870… Patch: https://github.com/kamailio/kamailio/commit/0eb57ecfa56e0bf49c2f80291d22870…

3 years, 8 months

[kamailio/kamailio] Failed to find command uac_auth (params 1) (#2672)

by Ben Kaufman

### Description According to the documentation for the UAC module, the `uac_reg` command can accept an optional argument to use the pre-calculated HA1 format: ``` 4.7. uac_auth([mode]) This function can be called only from failure route and will build the authentication response header and insert it into the request without sending anything. If mode is set to 1, then the password has to be provided in HA1 format. The parameter can be a static integer or a variable holding an integer value ``` However, when I attempt to use this in my configuration file, Kamailio fails to start. ### Troubleshooting #### Reproduction The problem is reproducible in the 5.4, 5.4-nightly, and dev-nightly debian packages with the following config file: ``` #!KAMAILIO loadmodule "tm" loadmodule "sl" loadmodule "pv" loadmodule "uac" loadmodule "rr" modparam("uac","auth_username_avp","$avp(auser)") modparam("uac","auth_password_avp","$avp(apass)") modparam("uac","auth_realm_avp","$avp(arealm)") request_route { if( $rm == "INVITE" ) { t_on_failure("TRUNKAUTH"); } t_relay(); } failure_route[TRUNKAUTH] { $avp(auser) = "user"; $avp(arealm) = "realm"; $avp(apass) = "password"; if (uac_auth(1)) { t_relay(); } exit; } ``` A check on syntax file syntax gives an error of: ``` $ sudo kamailio -eEc 0(18427) ERROR: <core> [core/cfg.y:3451]: yyparse(): cfg. parser: failed to find command uac_auth (params 1) 0(18427) CRITICAL: <core> [core/cfg.y:3592]: yyerror_at(): parse error in config file /etc/kamailio/kamailio.cfg, line 30, column 19: unknown command, missing loadmodule? ERROR: bad config file (1 errors) ``` No failures are observed if the parameter is removed from line 30. Changed from `if (uac_auth(1)) {` to `if (uac_auth()) {`. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.4.4 (x86_64/linux) 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: unknown compiled with gcc 8.3.0 ``` * **Operating System**:  ``` Linux aarenet-sipdir2 4.19.0-14-cloud-amd64 #1 SMP Debian 4.19.171-2 (2021-01-30) x86_64 GNU/Linux ``` -- 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/2672

3 years, 8 months

git:master:bb1fe2fc: uac: exposed uac_auth_mode() also as uac_mode([mode]) to match docs

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: bb1fe2fc12cb0499c5242c220dd8065e83fe6e61 URL: https://github.com/kamailio/kamailio/commit/bb1fe2fc12cb0499c5242c220dd8065… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-03-11T18:39:46+01:00 uac: exposed uac_auth_mode() also as uac_mode([mode]) to match docs --- Modified: src/modules/uac/uac.c --- Diff: https://github.com/kamailio/kamailio/commit/bb1fe2fc12cb0499c5242c220dd8065… Patch: https://github.com/kamailio/kamailio/commit/bb1fe2fc12cb0499c5242c220dd8065… --- diff --git a/src/modules/uac/uac.c b/src/modules/uac/uac.c index 0988669f67..486fed2894 100644 --- a/src/modules/uac/uac.c +++ b/src/modules/uac/uac.c @@ -135,6 +135,8 @@ static cmd_export_t cmds[]={ REQUEST_ROUTE | BRANCH_ROUTE }, {"uac_restore_to", (cmd_function)w_restore_to, 0, 0, 0, REQUEST_ROUTE }, {"uac_auth", (cmd_function)w_uac_auth, 0, 0, 0, FAILURE_ROUTE }, + {"uac_auth", (cmd_function)w_uac_auth_mode, 1, + fixup_igp_null, fixup_free_igp_null, FAILURE_ROUTE }, {"uac_auth_mode", (cmd_function)w_uac_auth_mode, 1, fixup_igp_null, fixup_free_igp_null, FAILURE_ROUTE }, {"uac_req_send", (cmd_function)w_uac_req_send, 0, 0, 0, ANY_ROUTE},

3 years, 8 months

git:master:be4e1843: uac: docs - listed uac_auth_mode() function

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: be4e1843c54e70206688c1f2d1f5483165baef8f URL: https://github.com/kamailio/kamailio/commit/be4e1843c54e70206688c1f2d1f5483… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-03-11T18:39:16+01:00 uac: docs - listed uac_auth_mode() function --- Modified: src/modules/uac/doc/uac_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/be4e1843c54e70206688c1f2d1f5483… Patch: https://github.com/kamailio/kamailio/commit/be4e1843c54e70206688c1f2d1f5483… --- diff --git a/src/modules/uac/doc/uac_admin.xml b/src/modules/uac/doc/uac_admin.xml index a738c84b81..c8271974a3 100644 --- a/src/modules/uac/doc/uac_admin.xml +++ b/src/modules/uac/doc/uac_admin.xml @@ -884,6 +884,57 @@ failure_route[TRUNKAUTH] { exit; } } +... + </programlisting> + </example> + </section> + <section id="uac.f.uac_auth_mode"> + <title> + <function moreinfo="none">uac_auth_mode(vmode)</function> + </title> + <para> + This function can be called only from failure route and will + build the authentication response header and insert it into the + request without sending anything. + </para> + <para> + If mode is set to 1, then the password has to be provided in HA1 format. + The parameter can be a static integer or a variable holding an integer value. + </para> + <para> + This function can be used from FAILURE_ROUTE. + </para> + <example> + <title><function>uac_auth_mode</function> usage</title> + <programlisting format="linespecific"> +... +modparam("uac","auth_username_avp","$avp(auser)") +modparam("uac","auth_password_avp","$avp(apass)") +modparam("uac","auth_realm_avp","$avp(arealm)") + +request_route { + ... + if(is_method("INVITE")) { + t_on_failure("TRUNKAUTH"); + } + ... +} + +failure_route[TRUNKAUTH] { + + if (t_is_canceled()) { + exit; + } + if(t_check_status("401|407")) { + $avp(auser) = "test"; + $avp(apass) = "test"; + # $avp(apass) = "36d0a02793542b4961e8348347236dbf"; + if (uac_auth_mode("1")) { + t_relay(); + } + exit; + } +} ... </programlisting> </example>

3 years, 8 months

git:master:afe84def: tm: log message if starting cancel retransmission fails

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: afe84defa4ab391064c266e15f557415c5249f7c URL: https://github.com/kamailio/kamailio/commit/afe84defa4ab391064c266e15f55741… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-03-11T17:28:09+01:00 tm: log message if starting cancel retransmission fails --- Modified: src/modules/tm/t_cancel.c --- Diff: https://github.com/kamailio/kamailio/commit/afe84defa4ab391064c266e15f55741… Patch: https://github.com/kamailio/kamailio/commit/afe84defa4ab391064c266e15f55741… --- diff --git a/src/modules/tm/t_cancel.c b/src/modules/tm/t_cancel.c index 69fdb9f77e..170330395c 100644 --- a/src/modules/tm/t_cancel.c +++ b/src/modules/tm/t_cancel.c @@ -527,7 +527,9 @@ unsigned int t_uac_cancel( str *headers, str *body, LM_ERR("send failed\n"); goto error1; } - start_retr(cancel); + if(start_retr(cancel)!=0) { + LM_CRIT("failed to start retransmission for cancel %p\n", cancel); + } /* </start_sending> */ return ret;

3 years, 8 months

Errored: kamailio/kamailio#9953 (master - 9917deb)

by Travis CI

Build Update for kamailio/kamailio ------------------------------------- Build: #9953 Status: Errored Duration: 5 mins and 32 secs Commit: 9917deb (master) Author: Daniel-Constantin Mierla Message: dialplan: renamed global variables and set them static - use module specific prefix View the changeset: https://github.com/kamailio/kamailio/compare/c05c7133a565...9917deb664fa View the full build log and details: https://travis-ci.org/github/kamailio/kamailio/builds/762478964?utm_medium=… -- You can unsubscribe from build emails from the kamailio/kamailio repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=3596148&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

3 years, 8 months

git:master:9917deb6: dialplan: renamed global variables and set them static

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 9917deb664fa5df6795cbf59f9f156d2e10d1a95 URL: https://github.com/kamailio/kamailio/commit/9917deb664fa5df6795cbf59f9f156d… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-03-11T16:55:23+01:00 dialplan: renamed global variables and set them static - use module specific prefix --- Modified: src/modules/dialplan/dp_db.c --- Diff: https://github.com/kamailio/kamailio/commit/9917deb664fa5df6795cbf59f9f156d… Patch: https://github.com/kamailio/kamailio/commit/9917deb664fa5df6795cbf59f9f156d…

3 years, 8 months

git:master:0838c3e4: dispatcher: renamed global variables and set them static

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 0838c3e4bc5f9f37d408bc01ba48e4abd6d56ce3 URL: https://github.com/kamailio/kamailio/commit/0838c3e4bc5f9f37d408bc01ba48e4a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-03-11T16:55:23+01:00 dispatcher: renamed global variables and set them static - use module specific prefix --- Modified: src/modules/dispatcher/dispatch.c Modified: src/modules/dispatcher/dispatcher.c --- Diff: https://github.com/kamailio/kamailio/commit/0838c3e4bc5f9f37d408bc01ba48e4a… Patch: https://github.com/kamailio/kamailio/commit/0838c3e4bc5f9f37d408bc01ba48e4a…

3 years, 8 months

git:5.4:0ddb019f: pv_headers: fix detection of split marker

by Victor Seva

Module: kamailio Branch: 5.4 Commit: 0ddb019f3d80e4bef2d00a504dbe1baa6b982bba URL: https://github.com/kamailio/kamailio/commit/0ddb019f3d80e4bef2d00a504dbe1ba… Author: Victor Seva <vseva(a)sipwise.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2021-03-11T13:44:07+01:00 pv_headers: fix detection of split marker If we set Diversion in split_headers and we get a header like > "RULTEST, normalaa" <sip:0046341355354340@whatever.net;user=phone>;reason=unconditional There was false detection of two Diversion headers. Skip split marker between double quotes to avoid this (cherry picked from commit c05c7133a5657a547a5b0ced0b0a05707b44bc06) --- Modified: src/modules/pv_headers/pvh_func.c Modified: src/modules/pv_headers/pvh_hash.c Modified: src/modules/pv_headers/pvh_str.c Modified: src/modules/pv_headers/pvh_str.h --- Diff: https://github.com/kamailio/kamailio/commit/0ddb019f3d80e4bef2d00a504dbe1ba… Patch: https://github.com/kamailio/kamailio/commit/0ddb019f3d80e4bef2d00a504dbe1ba…

3 years, 8 months

git:master:c05c7133: pv_headers: fix detection of split marker

by Victor Seva

Module: kamailio Branch: master Commit: c05c7133a5657a547a5b0ced0b0a05707b44bc06 URL: https://github.com/kamailio/kamailio/commit/c05c7133a5657a547a5b0ced0b0a057… Author: Victor Seva <vseva(a)sipwise.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2021-03-11T13:41:41+01:00 pv_headers: fix detection of split marker If we set Diversion in split_headers and we get a header like > "RULTEST, normalaa" <sip:0046341355354340@whatever.net;user=phone>;reason=unconditional There was false detection of two Diversion headers. Skip split marker between double quotes to avoid this --- Modified: src/modules/pv_headers/pvh_func.c Modified: src/modules/pv_headers/pvh_hash.c Modified: src/modules/pv_headers/pvh_str.c Modified: src/modules/pv_headers/pvh_str.h --- Diff: https://github.com/kamailio/kamailio/commit/c05c7133a5657a547a5b0ced0b0a057… Patch: https://github.com/kamailio/kamailio/commit/c05c7133a5657a547a5b0ced0b0a057…

3 years, 8 months

← Newer
1
...
35
36
37
38
39
40
41
...
49
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev March 2021