sr-dev July 2021

sr-dev@lists.kamailio.org

13 participants
227 discussions

[kamailio/kamailio] Wrong return code used by sl_reply_error() (#2802)

by Norbixin

### Description I wanted to change 478 response to 503 in case of t_relay() failure, but leave default behavior in rest error cases. I used part of code from "Example 1.1." inside https://www.kamailio.org/docs/modules/devel/modules/outbound.html. When I get -478 from t_relay() it works fine, but when this code is different sl_reply_erorr() don't use it as it should be. This is my part of kamailio.cfg: ``` route[RELAY] { if (!t_relay()) { switch($rc) { case -478: xlog("L_WARN","T_Relay returned 478, sending back 503.\n"); sl_send_reply("503", "Service Unavailable"); exit; default: sl_reply_error(); exit; } } exit; } ``` When I log $rc value before sl_reply_error() it is valid, but not used by sl_reply_error() as SIP error value, it sends 500 instead. ### Troubleshooting #### Reproduction I've added incorrect hostnames with flag 16 to set where the message should be sent and t_relay() returns -478. To check the default case inside switch I changed case -478 to -479. #### Log Messages  ``` 1(8) ERROR: <core> [core/resolve.c:1731]: sip_hostport2su(): could not resolve hostname: "va4brhino" 1(8) ERROR: tm [ut.h:309]: uri2dst2(): failed to resolve "va4brhino" 1(8) ERROR: tm [t_fwd.c:1759]: t_forward_nonack(): failure to add branches 1(8) ERROR: sl [sl_funcs.c:414]: sl_reply_error(): stateless error reply used: I'm terribly sorry, server error occurred (1/SL) ``` Logs with different route relay ``` route[RELAY] { if (!t_relay()) { switch($rc) { case -479: xlog("L_WARN","T_Relay returned 479, sending back 503.\n"); sl_send_reply("503", "Service Unavailable"); exit; default: xlog("L_WARN","Return code: $rc\n"); sl_reply_error(); exit; } } exit; } ``` ``` 1(8) ERROR: <core> [core/resolve.c:1731]: sip_hostport2su(): could not resolve hostname: "va4brhino" 1(8) ERROR: tm [ut.h:309]: uri2dst2(): failed to resolve "va4brhino" 1(8) ERROR: tm [t_fwd.c:1759]: t_forward_nonack(): failure to add branches 1(8) WARNING: <script>: Return code: -478 1(8) ERROR: sl [sl_funcs.c:414]: sl_reply_error(): stateless error reply used: I'm terribly sorry, server error occurred (1/SL) ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.5.0 (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_BLOCKLIST, HAVE_RESOLV_RES 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 4.8.4 ``` * **Operating System**:  ``` Linux v4bkamailio 5.10.25-linuxkit #1 SMP Tue Mar 23 09:27:39 UTC 2021 x86_64 x86_64 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/2802

3 years, 4 months

git:master:40b1d627: modules: readme files regenerated - textopsx ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 40b1d627d8f81235814ccee64c562a425f382648 URL: https://github.com/kamailio/kamailio/commit/40b1d627d8f81235814ccee64c562a4… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2021-07-26T17:31:24+02:00 modules: readme files regenerated - textopsx ... [skip ci] --- Modified: src/modules/textopsx/README --- Diff: https://github.com/kamailio/kamailio/commit/40b1d627d8f81235814ccee64c562a4… Patch: https://github.com/kamailio/kamailio/commit/40b1d627d8f81235814ccee64c562a4… --- diff --git a/src/modules/textopsx/README b/src/modules/textopsx/README index a72249f1be..3824577860 100644 --- a/src/modules/textopsx/README +++ b/src/modules/textopsx/README @@ -38,6 +38,7 @@ Daniel-Constantin Mierla 2.17. hf_iterator_end(iname) 2.18. textopsx.f.hf_iterator_next(iname) 2.19. hf_iterator_rm(iname) + 2.20. hf_iterator_append(iname, htext) 3. Selects @@ -66,7 +67,8 @@ Daniel-Constantin Mierla 1.17. hf_iterator_end usage 1.18. hf_iterator_next usage 1.19. hf_iterator_rm usage - 1.20. @hf_value select usage + 1.20. hf_iterator_append usage + 1.21. @hf_value select usage Chapter 1. Admin Guide @@ -94,6 +96,7 @@ Chapter 1. Admin Guide 2.17. hf_iterator_end(iname) 2.18. textopsx.f.hf_iterator_next(iname) 2.19. hf_iterator_rm(iname) + 2.20. hf_iterator_append(iname, htext) 3. Selects @@ -132,6 +135,7 @@ Chapter 1. Admin Guide 2.17. hf_iterator_end(iname) 2.18. textopsx.f.hf_iterator_next(iname) 2.19. hf_iterator_rm(iname) + 2.20. hf_iterator_append(iname, htext) 2.1. msg_apply_changes() @@ -493,6 +497,26 @@ hf_iterator_end("i1"); hf_iterator_end("i1"); ... +2.20. hf_iterator_append(iname, htext) + + Add headers after the one at the current iterator possition. + + The parameters can be dynamic strings with variables. + + This function can be used from ANY_ROUTE. + + Example 1.20. hf_iterator_append usage +... + hf_iterator_start("i1"); + while(hf_iterator_next("i1")) { + if($hfitname(i1)=="My-Header") { + hf_iterator_append("i1", "My-New-Header: abc\r\n"); + break; + } + } + hf_iterator_end("i1"); +... + 3. Selects 3.1. @hf_value @@ -528,7 +552,7 @@ hf_iterator_end("i1"); * IDX - Value index, negative value counts from bottom * PARAM_NAME - name of parameter - Example 1.20. @hf_value select usage + Example 1.21. @hf_value select usage ... $a = @hf_value.my_header[1].my_param; xplog("L_ERR", "$sel((a)hf_value.via[-1]), $sel((a)hf_value.from.tag)\n");

3 years, 4 months

[kamailio/kamailio] RTPEngine: Async connection/communication between Kamailio and RTPEngine (#2807)

by Miteshkumar Thakkar

### Description Can we have async connection/communication between Kamailio and RTPEngine #### Actual observed behavior Currently, if there is a network latency or unstable network between Kamailio and RTPEngine, the Kamailio is keep waiting/polling on RTPEngine. That can be overcome if we have async connection. -Miteshkumar -- 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/2807

3 years, 4 months

git:master:001e07dd: textopsx: docs for hf_iterator_append()

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 001e07ddaff30153ee9bc9beb5cb2430007173e5 URL: https://github.com/kamailio/kamailio/commit/001e07ddaff30153ee9bc9beb5cb243… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-07-26T17:15:38+02:00 textopsx: docs for hf_iterator_append() --- Modified: src/modules/textopsx/doc/functions.xml --- Diff: https://github.com/kamailio/kamailio/commit/001e07ddaff30153ee9bc9beb5cb243… Patch: https://github.com/kamailio/kamailio/commit/001e07ddaff30153ee9bc9beb5cb243… --- diff --git a/src/modules/textopsx/doc/functions.xml b/src/modules/textopsx/doc/functions.xml index c1c5fd5f64..b43a0d4bb8 100644 --- a/src/modules/textopsx/doc/functions.xml +++ b/src/modules/textopsx/doc/functions.xml @@ -589,6 +589,35 @@ hf_iterator_end("i1"); } hf_iterator_end("i1"); ... +</programlisting> + </example> + </section> + <section id="textopsx.f.hf_iterator_append"> + <title> + <function moreinfo="none">hf_iterator_append(iname, htext)</function> + </title> + <para> + Add headers after the one at the current iterator possition. + </para> + <para> + The parameters can be dynamic strings with variables. + </para> + <para> + This function can be used from ANY_ROUTE. + </para> + <example> + <title><function>hf_iterator_append</function> usage</title> + <programlisting format="linespecific"> +... + hf_iterator_start("i1"); + while(hf_iterator_next("i1")) { + if($hfitname(i1)=="My-Header") { + hf_iterator_append("i1", "My-New-Header: abc\r\n"); + break; + } + } + hf_iterator_end("i1"); +... </programlisting> </example> </section>

3 years, 4 months

git:master:3f3fb6ad: textopsx: added hf_iterator_append(iname, htext) function

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 3f3fb6ad94805407b316eb0ec4883631ac1df604 URL: https://github.com/kamailio/kamailio/commit/3f3fb6ad94805407b316eb0ec488363… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-07-26T17:02:01+02:00 textopsx: added hf_iterator_append(iname, htext) function --- Modified: src/modules/textopsx/textopsx.c --- Diff: https://github.com/kamailio/kamailio/commit/3f3fb6ad94805407b316eb0ec488363… Patch: https://github.com/kamailio/kamailio/commit/3f3fb6ad94805407b316eb0ec488363… --- diff --git a/src/modules/textopsx/textopsx.c b/src/modules/textopsx/textopsx.c index ec3e1afb75..297ff1098e 100644 --- a/src/modules/textopsx/textopsx.c +++ b/src/modules/textopsx/textopsx.c @@ -77,6 +77,7 @@ static int w_hf_iterator_start(sip_msg_t *msg, char *piname, char *p2); static int w_hf_iterator_next(sip_msg_t *msg, char *piname, char *p2); static int w_hf_iterator_end(sip_msg_t *msg, char *piname, char *p2); static int w_hf_iterator_rm(sip_msg_t *msg, char *piname, char *p2); +static int w_hf_iterator_append(sip_msg_t *msg, char *piname, char *phtext); static int bind_textopsx(textopsx_api_t *tob); @@ -139,6 +140,8 @@ static cmd_export_t cmds[] = { fixup_free_spve_null, ANY_ROUTE}, {"hf_iterator_rm", w_hf_iterator_rm, 1, fixup_spve_null, fixup_free_spve_null, ANY_ROUTE}, + {"hf_iterator_append", w_hf_iterator_append, 2, fixup_spve_spve, + fixup_free_spve_spve, ANY_ROUTE}, {"bind_textopsx", (cmd_function)bind_textopsx, 1, 0, 0, ANY_ROUTE}, @@ -2057,6 +2060,63 @@ static int w_hf_iterator_rm(sip_msg_t *msg, char *piname, char *p2) return ki_hf_iterator_rm(msg, &iname); } +/** + * + */ +static int ki_hf_iterator_append(sip_msg_t *msg, str *iname, str *htext) +{ + int k; + sr_lump_t *anchor; + str sval = STR_NULL; + + k = ki_hf_iterator_index(msg, iname); + if(k<0 || _hf_iterators[k].it==NULL) { + return -1; + } + anchor = anchor_lump(msg, _hf_iterators[k].it->name.s + + _hf_iterators[k].it->len - msg->buf, 0, 0); + if (anchor==0) { + LM_ERR("cannot append hdr after %.*s\n", _hf_iterators[k].it->name.len, + _hf_iterators[k].it->name.s); + return -1; + } + sval.s = (char*)pkg_malloc(htext->len + 1); + if(sval.s==NULL) { + LM_ERR("failed append hdr after %.*s\n", _hf_iterators[k].it->name.len, + _hf_iterators[k].it->name.s); + return -1; + } + memcpy(sval.s, htext->s, htext->len); + sval.len = htext->len; + sval.s[sval.len] = '\0'; + + if (insert_new_lump_before(anchor, sval.s, sval.len, 0) == 0) { + LM_ERR("cannot insert lump\n"); + pkg_free(sval.s); + return -1; + } + return 1; +} + +/** + * + */ +static int w_hf_iterator_append(sip_msg_t *msg, char *piname, char *phtext) +{ + str iname = STR_NULL; + str htext = STR_NULL; + if(fixup_get_svalue(msg, (gparam_t*)piname, &iname)<0) { + LM_ERR("failed to get iterator name\n"); + return -1; + } + if(fixup_get_svalue(msg, (gparam_t*)phtext, &htext)<0) { + LM_ERR("failed to get header text\n"); + return -1; + } + + return ki_hf_iterator_append(msg, &iname, &htext); +} + /** * */

3 years, 4 months

Info: Kamailio World Online - Sep 1-2, 2021 - Celebrating 20 Years Of Development

by Daniel-Constantin Mierla

Hello, given the current situation across the world, no physical Kamailio World Conference is planned in 2021. However, this year the project celebrates 20 years of development -- according to commit history, the first one was done on the 3rd of September, 2021: > ❯ git log --reverse > commit 512dcd9838f5aeb0bfccd02cfc156067b7f02b71 > Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> > Date: Mon Sep 3 21:27:11 2001 +0000 > Initial revision To mark this notable milestone, we are going to organize an online event during Sep 1-2, 2021, similar to the one from 2020 -- the website for the event is available at: * https://www.kamailioworld.com/k09-online/ Mark the date in your calendar, more details about the content of the event will be published in the near future! Cheers, Daniel -- Daniel-Constantin Mierla -- www.asipto.com www.twitter.com/miconda -- www.linkedin.com/in/miconda

3 years, 4 months

kamailio trouble

by 崔群

Dear sir: I have some problems。the kamailio version is 5.3.0. My linux version is centos 7. the kamailio log: ERROR: <core> [core/tcp_main.c:3508]: send_fd_queue_run(): send_fd failed on socket 63 , queue entry 1, retries 90, connection 0x7ff46e6b1b60, tcp socket 218, errno=11 (Resource temporarily unavailable) ERROR: {1 12529 MESSAGE 359bf0410a7e471c950623e3f7a3b17b} <core> [core/kemi.c:94]: sr_kemi_core_err(): 202->408 CRITICAL: <core> [core/tcp_main.c:4218]: send2child(): tcp child 6, socket 65: queue full, 291 requests queued (total handled 10665748) This problem does not happen every time。When the number of user registrations increases significantly, errors will appear。I tried to set the parameter tcp_children is adjusted to 8, it will not appear frequently, but when the number of user registrations increases to about 5000, errors will still appear. My kamailio.cfg global parameters are set as follows：tls_max_connections=65536,tcp_max_connections=65536,open_max_files=65536。 I hope you can reply to me，thanks.

3 years, 4 months

git:master:6ec588a7: modules: readme files regenerated - ruxc ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 6ec588a71e49c88ac0b62f3385cb6e7744fc13f9 URL: https://github.com/kamailio/kamailio/commit/6ec588a71e49c88ac0b62f3385cb6e7… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2021-07-24T08:31:20+02:00 modules: readme files regenerated - ruxc ... [skip ci] --- Modified: src/modules/ruxc/README --- Diff: https://github.com/kamailio/kamailio/commit/6ec588a71e49c88ac0b62f3385cb6e7… Patch: https://github.com/kamailio/kamailio/commit/6ec588a71e49c88ac0b62f3385cb6e7… --- diff --git a/src/modules/ruxc/README b/src/modules/ruxc/README index 6fb238e3b7..e437234139 100644 --- a/src/modules/ruxc/README +++ b/src/modules/ruxc/README @@ -28,6 +28,7 @@ Daniel-Constantin Mierla 3.1. http_timeout (int) 3.2. http_tlsmode (int) + 3.3. http_reuse (int) 4. Functions @@ -40,9 +41,10 @@ Daniel-Constantin Mierla 1.1. Set http_timeout parameter 1.2. Set http_tlsmode parameter - 1.3. ruxc_http_get() usage - 1.4. ruxc_http_post() usage - 1.5. Libruxc Usage + 1.3. Set http_reuse parameter + 1.4. ruxc_http_get() usage + 1.5. ruxc_http_post() usage + 1.6. Libruxc Usage Chapter 1. Admin Guide @@ -58,6 +60,7 @@ Chapter 1. Admin Guide 3.1. http_timeout (int) 3.2. http_tlsmode (int) + 3.3. http_reuse (int) 4. Functions @@ -95,6 +98,7 @@ Chapter 1. Admin Guide 3.1. http_timeout (int) 3.2. http_tlsmode (int) + 3.3. http_reuse (int) 3.1. http_timeout (int) @@ -120,6 +124,20 @@ modparam("ruxc", "http_timeout", 2000) modparam("ruxc", "http_tlsmode", 1) ... +3.3. http_reuse (int) + + Set to 1 in order to reuse the connection for all requests (each + Kamailio process has its own connection). Useful to avoid TCP connect + (and TLS handshake) when all requests are performed against the same + HTTP/S server. + + Default value is 0 (new connection for each request). + + Example 1.3. Set http_reuse parameter +... +modparam("ruxc", "http_reuse", 1) +... + 4. Functions 4.1. ruxc_http_get(url, hdrs, respv) @@ -137,7 +155,7 @@ modparam("ruxc", "http_tlsmode", 1) This function can be used from ANY_ROUTE. - Example 1.3. ruxc_http_get() usage + Example 1.4. ruxc_http_get() usage ... ruxc_http_get("http://api.com/index.php?r_uri=$(ru{s.escape.param})&f_uri=$(fu{s .escape.param})", @@ -160,7 +178,7 @@ switch ($rc) { This function can be used from ANY_ROUTE. - Example 1.4. ruxc_http_post() usage + Example 1.5. ruxc_http_post() usage ... ruxc_http_post("http://api.com/index.php?r_uri=$(ru{s.escape.param})&f_uri=$(fu{ s.escape.param})", @@ -183,7 +201,7 @@ switch ($rc) { installed and its environment configured, then run the following commands: - Example 1.5. Libruxc Usage + Example 1.6. Libruxc Usage ... git clone https://github.com/miconda/ruxc cd ruxc

3 years, 4 months

git:master:600d0e8f: ruxc: added http_reuse modparam

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 600d0e8fb7f24e424873e4ad24beb077d2b45ee4 URL: https://github.com/kamailio/kamailio/commit/600d0e8fb7f24e424873e4ad24beb07… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-07-24T08:16:49+02:00 ruxc: added http_reuse modparam - enable connection reuse --- Modified: src/modules/ruxc/doc/ruxc_admin.xml Modified: src/modules/ruxc/ruxc_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/600d0e8fb7f24e424873e4ad24beb07… Patch: https://github.com/kamailio/kamailio/commit/600d0e8fb7f24e424873e4ad24beb07… --- diff --git a/src/modules/ruxc/doc/ruxc_admin.xml b/src/modules/ruxc/doc/ruxc_admin.xml index e4b0384ea4..e07c05c9f1 100644 --- a/src/modules/ruxc/doc/ruxc_admin.xml +++ b/src/modules/ruxc/doc/ruxc_admin.xml @@ -95,6 +95,27 @@ modparam("ruxc", "http_timeout", 2000) ... modparam("ruxc", "http_tlsmode", 1) ... +</programlisting> + </example> + </section> + <section id="ruxc.p.http_reuse"> + <title><varname>http_reuse</varname> (int)</title> + <para> + Set to 1 in order to reuse the connection for all requests (each &kamailio; + process has its own connection). Useful to avoid TCP connect (and TLS + handshake) when all requests are performed against the same HTTP/S server. + </para> + <para> + <emphasis> + Default value is 0 (new connection for each request). + </emphasis> + </para> + <example> + <title>Set <varname>http_reuse</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("ruxc", "http_reuse", 1) +... </programlisting> </example> </section> diff --git a/src/modules/ruxc/ruxc_mod.c b/src/modules/ruxc/ruxc_mod.c index 2cd1cd27d0..02fa278859 100644 --- a/src/modules/ruxc/ruxc_mod.c +++ b/src/modules/ruxc/ruxc_mod.c @@ -40,6 +40,7 @@ MODULE_VERSION static int _ruxc_http_timeout = 5000; static int _ruxc_http_tlsmode = 0; +static int _ruxc_http_reuse = 0; static int mod_init(void); static int child_init(int); @@ -73,6 +74,7 @@ static cmd_export_t cmds[]={ static param_export_t params[]={ {"http_timeout", PARAM_INT, &_ruxc_http_timeout}, {"http_tlsmode", PARAM_INT, &_ruxc_http_tlsmode}, + {"http_reuse", PARAM_INT, &_ruxc_http_reuse}, {0, 0, 0} }; @@ -132,6 +134,7 @@ static int ki_ruxc_http_get_helper(sip_msg_t *_msg, str *url, str *hdrs, v_http_request.timeout_read = _ruxc_http_timeout; v_http_request.timeout_write = _ruxc_http_timeout; v_http_request.tlsmode = _ruxc_http_tlsmode; + v_http_request.reuse = _ruxc_http_reuse; v_http_request.url = url->s; v_http_request.url_len = url->len; @@ -229,6 +232,7 @@ static int ki_ruxc_http_post_helper(sip_msg_t *_msg, str *url, str *body, str *h v_http_request.timeout_read = _ruxc_http_timeout; v_http_request.timeout_write = _ruxc_http_timeout; v_http_request.tlsmode = _ruxc_http_tlsmode; + v_http_request.reuse = _ruxc_http_reuse; v_http_request.url = url->s; v_http_request.url_len = url->len;

3 years, 4 months

git:master:2042432b: modules: readme files regenerated - rtpengine ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 2042432b598fcceb9de7cf8a659498e99aec9f05 URL: https://github.com/kamailio/kamailio/commit/2042432b598fcceb9de7cf8a659498e… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2021-07-23T19:46:14+02:00 modules: readme files regenerated - rtpengine ... [skip ci] --- Modified: src/modules/rtpengine/README --- Diff: https://github.com/kamailio/kamailio/commit/2042432b598fcceb9de7cf8a659498e… Patch: https://github.com/kamailio/kamailio/commit/2042432b598fcceb9de7cf8a659498e…

3 years, 4 months

← Newer
1
...
13
14
15
16
17
18
19
...
23
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev July 2021