sr-dev August 2020

sr-dev@lists.kamailio.org

17 participants
245 discussions

Releasing Kamailio v5.4.1
by Daniel-Constantin Mierla 01 Sep '20

01 Sep '20

Hello, I am considering to release Kamailio v5.4.1 as the first patch release in the 5.4 series, likely to be done next week, on the 1st of September, 2020. There are a few issues reported on the tracker, hopefully they will be tackled till then. If you are aware of something else not working as expected, report them to issue tracker asap to give a better chance to be fixed till the release day. Cheers, Daniel -- Daniel-Constantin Mierla -- www.asipto.com www.twitter.com/miconda -- www.linkedin.com/in/miconda Funding: https://www.paypal.me/dcmierla

1 2

Kamailio World Online Event - Schedule and access details
by Daniel-Constantin Mierla 31 Aug '20

31 Aug '20

Hello, the schedule for Kamailio World Updates has been published - it is an online event happening this week, on Wed-Thu, Sep 2-3, 2020, from 13:00UTC to 17:00UTC, more details are available at: * https://www.kamailioworld.com/k08-online/ The agenda includes presentations, live demos and open discussion sessions, as well as an online party to celebrate 19 years of development for Kamailio project. Being shorter than the usual Kamailio World, the event has an obvious focus on Kamailio only, with sessions about what is new in Kamailio ecosystem. The sessions are done live, in a self-hosted Jitsi Meet video conferencing room (hosting sponsored by lod.com), with live streaming to YouTube KamailioWorld channel: * https://www.youtube.com/c/KamailioWorld/ If streaming goes without problems, the recordings of presentations and live demos will be made available on YouTube channel after the event. If you want to join the video conferencing room, be sure you are muted and with video turned off, in order not to disturb the sessions with unexpected noise. Un-mute/turn video on only when you want to participate in a discussion. Asking questions and text discussions can be done on our self hosted Matrix server, in the room "kamailio" (full room id: #kamailio:matrix.kamailio.dev): * https://riot.kamailio.dev/#/room/#kamailio:matrix.kamailio.dev The server is federated, so if you have a Matrix account on another server, you can join easily. During the event, we will allow guests to join the room, so it is not really required to have a Matrix account. Should anyone have issues joining the chat room during the event, reply on sr-users and we will try to assist you. As an alternative for posting questions, I will keep an eye on YouTube comments during the live streaming and try to pass them to the speaker, if something pops up there. The last session on the 2nd day is the online celebration party for 19 years of development for Kamailio project. Grab a drink, prepare your birthday speech and say it live, tell an interesting story about your experiences in RTC or just enjoy open discussions with the other participants! It is the first time we are experimenting with a live online event, testing looked good so far and we hope everything will run smooth during the sessions! Anyhow, I am confident everyone will enjoy the event and be able to (re-)connect with people from Kamailio community, no matter where they are located around the world! Cheers, Daniel -- Daniel-Constantin Mierla -- www.asipto.com www.twitter.com/miconda -- www.linkedin.com/in/miconda

1 0

git:5.4:b1ce252f: keepalive: removed function name from log messages
by Daniel-Constantin Mierla 31 Aug '20

31 Aug '20

Module: kamailio Branch: 5.4 Commit: b1ce252f8090f66de290cec95dbb6e1762ee6bdf URL: https://github.com/kamailio/kamailio/commit/b1ce252f8090f66de290cec95dbb6e1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-31T12:16:05+02:00 keepalive: removed function name from log messages (cherry picked from commit 17d6a88090e2a67f5c0b06e6aae76d629595b37e) --- Modified: src/modules/keepalive/keepalive_core.c --- Diff: https://github.com/kamailio/kamailio/commit/b1ce252f8090f66de290cec95dbb6e1… Patch: https://github.com/kamailio/kamailio/commit/b1ce252f8090f66de290cec95dbb6e1… --- diff --git a/src/modules/keepalive/keepalive_core.c b/src/modules/keepalive/keepalive_core.c index 36cf81aef1..00f40c46bf 100644 --- a/src/modules/keepalive/keepalive_core.c +++ b/src/modules/keepalive/keepalive_core.c @@ -61,7 +61,7 @@ ticks_t ka_check_timer(ticks_t ticks, struct timer_ln* tl, void* param) ka_dest = (ka_dest_t *)param; - LM_DBG("ka_check_timer dest:%.*s\n", ka_dest->uri.len, ka_dest->uri.s); + LM_DBG("dest: %.*s\n", ka_dest->uri.len, ka_dest->uri.s); if(ka_counter_del > 0 && ka_dest->counter > ka_counter_del) { return (ticks_t)(0); /* stops the timer */ @@ -104,7 +104,7 @@ static void ka_options_callback( uri.s = t->to.s + 5; uri.len = t->to.len - 8; - LM_DBG("OPTIONS-Request was finished with code %d (to %.*s)\n", ps->code, + LM_DBG("OPTIONS request was finished with code %d (to %.*s)\n", ps->code, ka_dest->uri.len, ka_dest->uri.s); //uri.len, uri.s); @@ -149,7 +149,7 @@ static void ka_run_route(sip_msg_t *msg, str *uri, char *route) return; } - LM_DBG("ka_run_route event_route[%s]\n", route); + LM_DBG("run event_route[%s]\n", route); rt = route_get(&event_rt, route); if(rt < 0 || event_rt.rlist[rt] == NULL) {

1 0

git:master:17d6a880: keepalive: removed function name from log messages
by Daniel-Constantin Mierla 31 Aug '20

31 Aug '20

Module: kamailio Branch: master Commit: 17d6a88090e2a67f5c0b06e6aae76d629595b37e URL: https://github.com/kamailio/kamailio/commit/17d6a88090e2a67f5c0b06e6aae76d6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-31T12:15:00+02:00 keepalive: removed function name from log messages --- Modified: src/modules/keepalive/keepalive_core.c --- Diff: https://github.com/kamailio/kamailio/commit/17d6a88090e2a67f5c0b06e6aae76d6… Patch: https://github.com/kamailio/kamailio/commit/17d6a88090e2a67f5c0b06e6aae76d6… --- diff --git a/src/modules/keepalive/keepalive_core.c b/src/modules/keepalive/keepalive_core.c index 36cf81aef1..00f40c46bf 100644 --- a/src/modules/keepalive/keepalive_core.c +++ b/src/modules/keepalive/keepalive_core.c @@ -61,7 +61,7 @@ ticks_t ka_check_timer(ticks_t ticks, struct timer_ln* tl, void* param) ka_dest = (ka_dest_t *)param; - LM_DBG("ka_check_timer dest:%.*s\n", ka_dest->uri.len, ka_dest->uri.s); + LM_DBG("dest: %.*s\n", ka_dest->uri.len, ka_dest->uri.s); if(ka_counter_del > 0 && ka_dest->counter > ka_counter_del) { return (ticks_t)(0); /* stops the timer */ @@ -104,7 +104,7 @@ static void ka_options_callback( uri.s = t->to.s + 5; uri.len = t->to.len - 8; - LM_DBG("OPTIONS-Request was finished with code %d (to %.*s)\n", ps->code, + LM_DBG("OPTIONS request was finished with code %d (to %.*s)\n", ps->code, ka_dest->uri.len, ka_dest->uri.s); //uri.len, uri.s); @@ -149,7 +149,7 @@ static void ka_run_route(sip_msg_t *msg, str *uri, char *route) return; } - LM_DBG("ka_run_route event_route[%s]\n", route); + LM_DBG("run event_route[%s]\n", route); rt = route_get(&event_rt, route); if(rt < 0 || event_rt.rlist[rt] == NULL) {

1 0

git:master:8aeaee0f: pv: new transformation {uri.saor}
by Daniel-Constantin Mierla 31 Aug '20

31 Aug '20

Module: kamailio Branch: master Commit: 8aeaee0fda4013e1fe0477ce1dcb31738dc18aea URL: https://github.com/kamailio/kamailio/commit/8aeaee0fda4013e1fe0477ce1dcb317… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-31T09:37:45+02:00 pv: new transformation {uri.saor} - return sip aor (sip:[user@]domain) --- Modified: src/modules/pv/pv_trans.c Modified: src/modules/pv/pv_trans.h --- Diff: https://github.com/kamailio/kamailio/commit/8aeaee0fda4013e1fe0477ce1dcb317… Patch: https://github.com/kamailio/kamailio/commit/8aeaee0fda4013e1fe0477ce1dcb317… --- diff --git a/src/modules/pv/pv_trans.c b/src/modules/pv/pv_trans.c index 91e9bca516..4de9849165 100644 --- a/src/modules/pv/pv_trans.c +++ b/src/modules/pv/pv_trans.c @@ -1409,6 +1409,7 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, val->rs = (_tr_parsed_uri.passwd.s)?_tr_parsed_uri.passwd:_tr_empty; break; case TR_URI_DURI: + case TR_URI_SAOR: case TR_URI_SURI: if(_tr_uri.len >= TR_BUFFER_SIZE) { LM_WARN("uri too long [%.*s] (%d)\n", @@ -1436,7 +1437,8 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, memcpy(_tr_buffer, sv.s, sv.len); sv.s = _tr_buffer; sv.len++; - if(subtype == TR_URI_SURI && _tr_parsed_uri.user.len > 0) { + if((_tr_parsed_uri.user.len > 0) + && (subtype != TR_URI_DURI)) { memcpy(sv.s + sv.len, _tr_parsed_uri.user.s, _tr_parsed_uri.user.len); sv.len += _tr_parsed_uri.user.len; @@ -1448,19 +1450,21 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, _tr_parsed_uri.host.len); sv.len += _tr_parsed_uri.host.len; } - if(_tr_parsed_uri.port.len > 0) { - sv.s[sv.len] = ':'; - sv.len++; - memcpy(sv.s + sv.len, _tr_parsed_uri.port.s, - _tr_parsed_uri.port.len); - sv.len += _tr_parsed_uri.port.len; - } - if(_tr_parsed_uri.transport_val.len > 0) { - memcpy(sv.s + sv.len, ";transport=", 11); - sv.len += 11; - memcpy(sv.s + sv.len, _tr_parsed_uri.transport_val.s, - _tr_parsed_uri.transport_val.len); - sv.len += _tr_parsed_uri.transport_val.len; + if(subtype != TR_URI_SAOR) { + if(_tr_parsed_uri.port.len > 0) { + sv.s[sv.len] = ':'; + sv.len++; + memcpy(sv.s + sv.len, _tr_parsed_uri.port.s, + _tr_parsed_uri.port.len); + sv.len += _tr_parsed_uri.port.len; + } + if(_tr_parsed_uri.transport_val.len > 0) { + memcpy(sv.s + sv.len, ";transport=", 11); + sv.len += 11; + memcpy(sv.s + sv.len, _tr_parsed_uri.transport_val.s, + _tr_parsed_uri.transport_val.len); + sv.len += _tr_parsed_uri.transport_val.len; + } } sv.s[sv.len] = '\0'; val->rs = sv; @@ -2845,6 +2849,9 @@ char* tr_parse_uri(str* in, trans_t *t) } else if(name.len==4 && strncasecmp(name.s, "port", 4)==0) { t->subtype = TR_URI_PORT; goto done; + } else if(name.len==4 && strncasecmp(name.s, "saor", 4)==0) { + t->subtype = TR_URI_SAOR; + goto done; } else if(name.len==4 && strncasecmp(name.s, "suri", 4)==0) { t->subtype = TR_URI_SURI; goto done; diff --git a/src/modules/pv/pv_trans.h b/src/modules/pv/pv_trans.h index d15df89057..c587464bed 100644 --- a/src/modules/pv/pv_trans.h +++ b/src/modules/pv/pv_trans.h @@ -50,7 +50,8 @@ enum _tr_uri_subtype { TR_URI_NONE=0, TR_URI_USER, TR_URI_HOST, TR_URI_PASSWD, TR_URI_PORT, TR_URI_PARAMS, TR_URI_PARAM, TR_URI_HEADERS, TR_URI_TRANSPORT, TR_URI_TTL, TR_URI_UPARAM, TR_URI_MADDR, TR_URI_METHOD, TR_URI_LR, - TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_DURI, TR_URI_SURI + TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_SAOR, TR_URI_DURI, + TR_URI_SURI }; enum _tr_param_subtype { TR_PL_NONE=0, TR_PL_VALUE, TR_PL_VALUEAT, TR_PL_NAME, TR_PL_COUNT

1 0

git:5.4:e9baba91: tls: proper ifdef on libssl version for tls_init_locks()
by Daniel-Constantin Mierla 30 Aug '20

30 Aug '20

Module: kamailio Branch: 5.4 Commit: e9baba91c6c7e3d1817e8a07bbabbce3a8efbe79 URL: https://github.com/kamailio/kamailio/commit/e9baba91c6c7e3d1817e8a07bbabbce… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-30T18:05:43+02:00 tls: proper ifdef on libssl version for tls_init_locks() - avoid compile warning on unused label (cherry picked from commit 60bd3aa284a5d9eb117a9763aa590aecf276ba16) --- Modified: src/modules/tls/tls_locking.c --- Diff: https://github.com/kamailio/kamailio/commit/e9baba91c6c7e3d1817e8a07bbabbce… Patch: https://github.com/kamailio/kamailio/commit/e9baba91c6c7e3d1817e8a07bbabbce… --- diff --git a/src/modules/tls/tls_locking.c b/src/modules/tls/tls_locking.c index 3eb6f033a5..21078e82d9 100644 --- a/src/modules/tls/tls_locking.c +++ b/src/modules/tls/tls_locking.c @@ -140,7 +140,7 @@ unsigned long sr_ssl_id_f() /* returns -1 on error, 0 on success */ int tls_init_locks() { -/* OpenSSL is thread-safe since 1.1.0 */ +/* OpenSSL is no longer supporting to set locking callbacks since 1.1.0 */ #if OPENSSL_VERSION_NUMBER < 0x10100000L /* init "static" tls locks */ n_static_locks=CRYPTO_num_locks(); @@ -185,10 +185,13 @@ int tls_init_locks() * (only atomic_inc), fallback to the default use-locks mode * CRYPTO_set_add_lock_callback(atomic_add_f); */ -#endif return 0; error: tls_destroy_locks(); return -1; + +#else + return 0; +#endif }

1 0

git:master:60bd3aa2: tls: proper ifdef on libssl version for tls_init_locks()
by Daniel-Constantin Mierla 30 Aug '20

30 Aug '20

Module: kamailio Branch: master Commit: 60bd3aa284a5d9eb117a9763aa590aecf276ba16 URL: https://github.com/kamailio/kamailio/commit/60bd3aa284a5d9eb117a9763aa590ae… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-30T18:04:04+02:00 tls: proper ifdef on libssl version for tls_init_locks() - avoid compile warning on unused label --- Modified: src/modules/tls/tls_locking.c --- Diff: https://github.com/kamailio/kamailio/commit/60bd3aa284a5d9eb117a9763aa590ae… Patch: https://github.com/kamailio/kamailio/commit/60bd3aa284a5d9eb117a9763aa590ae… --- diff --git a/src/modules/tls/tls_locking.c b/src/modules/tls/tls_locking.c index 3eb6f033a5..21078e82d9 100644 --- a/src/modules/tls/tls_locking.c +++ b/src/modules/tls/tls_locking.c @@ -140,7 +140,7 @@ unsigned long sr_ssl_id_f() /* returns -1 on error, 0 on success */ int tls_init_locks() { -/* OpenSSL is thread-safe since 1.1.0 */ +/* OpenSSL is no longer supporting to set locking callbacks since 1.1.0 */ #if OPENSSL_VERSION_NUMBER < 0x10100000L /* init "static" tls locks */ n_static_locks=CRYPTO_num_locks(); @@ -185,10 +185,13 @@ int tls_init_locks() * (only atomic_inc), fallback to the default use-locks mode * CRYPTO_set_add_lock_callback(atomic_add_f); */ -#endif return 0; error: tls_destroy_locks(); return -1; + +#else + return 0; +#endif }

1 0

git:master:564940f5: pv: new transformation {uri.duri}
by Daniel-Constantin Mierla 29 Aug '20

29 Aug '20

Module: kamailio Branch: master Commit: 564940f5ef923df7935a7b737a70e25c0d5f6a7b URL: https://github.com/kamailio/kamailio/commit/564940f5ef923df7935a7b737a70e25… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-29T19:45:20+02:00 pv: new transformation {uri.duri} - return uri needed for destination address (e.g., sip:host:port;transport=tcp) --- Modified: src/modules/pv/pv_trans.c Modified: src/modules/pv/pv_trans.h --- Diff: https://github.com/kamailio/kamailio/commit/564940f5ef923df7935a7b737a70e25… Patch: https://github.com/kamailio/kamailio/commit/564940f5ef923df7935a7b737a70e25… --- diff --git a/src/modules/pv/pv_trans.c b/src/modules/pv/pv_trans.c index b9eb291b99..91e9bca516 100644 --- a/src/modules/pv/pv_trans.c +++ b/src/modules/pv/pv_trans.c @@ -1408,6 +1408,7 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, case TR_URI_PASSWD: val->rs = (_tr_parsed_uri.passwd.s)?_tr_parsed_uri.passwd:_tr_empty; break; + case TR_URI_DURI: case TR_URI_SURI: if(_tr_uri.len >= TR_BUFFER_SIZE) { LM_WARN("uri too long [%.*s] (%d)\n", @@ -1435,7 +1436,7 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, memcpy(_tr_buffer, sv.s, sv.len); sv.s = _tr_buffer; sv.len++; - if(_tr_parsed_uri.user.len > 0) { + if(subtype == TR_URI_SURI && _tr_parsed_uri.user.len > 0) { memcpy(sv.s + sv.len, _tr_parsed_uri.user.s, _tr_parsed_uri.user.len); sv.len += _tr_parsed_uri.user.len; @@ -2847,6 +2848,9 @@ char* tr_parse_uri(str* in, trans_t *t) } else if(name.len==4 && strncasecmp(name.s, "suri", 4)==0) { t->subtype = TR_URI_SURI; goto done; + } else if(name.len==4 && strncasecmp(name.s, "duri", 4)==0) { + t->subtype = TR_URI_DURI; + goto done; } else if(name.len==6 && strncasecmp(name.s, "params", 6)==0) { t->subtype = TR_URI_PARAMS; goto done; diff --git a/src/modules/pv/pv_trans.h b/src/modules/pv/pv_trans.h index bf4c922243..d15df89057 100644 --- a/src/modules/pv/pv_trans.h +++ b/src/modules/pv/pv_trans.h @@ -50,7 +50,7 @@ enum _tr_uri_subtype { TR_URI_NONE=0, TR_URI_USER, TR_URI_HOST, TR_URI_PASSWD, TR_URI_PORT, TR_URI_PARAMS, TR_URI_PARAM, TR_URI_HEADERS, TR_URI_TRANSPORT, TR_URI_TTL, TR_URI_UPARAM, TR_URI_MADDR, TR_URI_METHOD, TR_URI_LR, - TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_SURI + TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_DURI, TR_URI_SURI }; enum _tr_param_subtype { TR_PL_NONE=0, TR_PL_VALUE, TR_PL_VALUEAT, TR_PL_NAME, TR_PL_COUNT

1 0

git:master:4a3ad2bf: pv: new transformation {uri.suri}
by Daniel-Constantin Mierla 28 Aug '20

28 Aug '20

Module: kamailio Branch: master Commit: 4a3ad2bf58d6c18f522bb73330030403a50cafbf URL: https://github.com/kamailio/kamailio/commit/4a3ad2bf58d6c18f522bb7333003040… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-28T20:42:15+02:00 pv: new transformation {uri.suri} - get the simple routing uri - keep only transport parameter, if exists --- Modified: src/modules/pv/pv_trans.c Modified: src/modules/pv/pv_trans.h --- Diff: https://github.com/kamailio/kamailio/commit/4a3ad2bf58d6c18f522bb7333003040… Patch: https://github.com/kamailio/kamailio/commit/4a3ad2bf58d6c18f522bb7333003040… --- diff --git a/src/modules/pv/pv_trans.c b/src/modules/pv/pv_trans.c index d07e55e719..b9eb291b99 100644 --- a/src/modules/pv/pv_trans.c +++ b/src/modules/pv/pv_trans.c @@ -1408,6 +1408,62 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, case TR_URI_PASSWD: val->rs = (_tr_parsed_uri.passwd.s)?_tr_parsed_uri.passwd:_tr_empty; break; + case TR_URI_SURI: + if(_tr_uri.len >= TR_BUFFER_SIZE) { + LM_WARN("uri too long [%.*s] (%d)\n", + _tr_uri.len, _tr_uri.s, _tr_uri.len); + val->rs = _tr_empty; + break; + } + + tr_set_crt_buffer(); + sv.s = _tr_uri.s; + sv.len = 0; + while(sv.len<_tr_uri.len) { + if(_tr_uri.s[sv.len]==':') { + break; + } + sv.len++; + } + if(_tr_uri.s[sv.len]!=':') { + LM_WARN("uri schema not found [%.*s] (%d)\n", + _tr_uri.len, _tr_uri.s, _tr_uri.len); + val->rs = _tr_empty; + break; + } + sv.len++; + memcpy(_tr_buffer, sv.s, sv.len); + sv.s = _tr_buffer; + sv.len++; + if(_tr_parsed_uri.user.len > 0) { + memcpy(sv.s + sv.len, _tr_parsed_uri.user.s, + _tr_parsed_uri.user.len); + sv.len += _tr_parsed_uri.user.len; + sv.s[sv.len] = '@'; + sv.len++; + } + if(_tr_parsed_uri.host.len > 0) { + memcpy(sv.s + sv.len, _tr_parsed_uri.host.s, + _tr_parsed_uri.host.len); + sv.len += _tr_parsed_uri.host.len; + } + if(_tr_parsed_uri.port.len > 0) { + sv.s[sv.len] = ':'; + sv.len++; + memcpy(sv.s + sv.len, _tr_parsed_uri.port.s, + _tr_parsed_uri.port.len); + sv.len += _tr_parsed_uri.port.len; + } + if(_tr_parsed_uri.transport_val.len > 0) { + memcpy(sv.s + sv.len, ";transport=", 11); + sv.len += 11; + memcpy(sv.s + sv.len, _tr_parsed_uri.transport_val.s, + _tr_parsed_uri.transport_val.len); + sv.len += _tr_parsed_uri.transport_val.len; + } + sv.s[sv.len] = '\0'; + val->rs = sv; + break; case TR_URI_PORT: val->flags |= PV_TYPE_INT|PV_VAL_INT; val->rs = (_tr_parsed_uri.port.s)?_tr_parsed_uri.port:_tr_empty; @@ -2788,6 +2844,9 @@ char* tr_parse_uri(str* in, trans_t *t) } else if(name.len==4 && strncasecmp(name.s, "port", 4)==0) { t->subtype = TR_URI_PORT; goto done; + } else if(name.len==4 && strncasecmp(name.s, "suri", 4)==0) { + t->subtype = TR_URI_SURI; + goto done; } else if(name.len==6 && strncasecmp(name.s, "params", 6)==0) { t->subtype = TR_URI_PARAMS; goto done; diff --git a/src/modules/pv/pv_trans.h b/src/modules/pv/pv_trans.h index 6494852246..bf4c922243 100644 --- a/src/modules/pv/pv_trans.h +++ b/src/modules/pv/pv_trans.h @@ -50,7 +50,7 @@ enum _tr_uri_subtype { TR_URI_NONE=0, TR_URI_USER, TR_URI_HOST, TR_URI_PASSWD, TR_URI_PORT, TR_URI_PARAMS, TR_URI_PARAM, TR_URI_HEADERS, TR_URI_TRANSPORT, TR_URI_TTL, TR_URI_UPARAM, TR_URI_MADDR, TR_URI_METHOD, TR_URI_LR, - TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET + TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_SURI }; enum _tr_param_subtype { TR_PL_NONE=0, TR_PL_VALUE, TR_PL_VALUEAT, TR_PL_NAME, TR_PL_COUNT

1 0

[kamailio/kamailio] TCP : queue full, x requests queued (total handled y) (#2392)
by ThomasSevestre 28 Aug '20

28 Aug '20

Hello, Do you know what can cause this issue ? Do you need more information ? ### Description I had an issue on a kamailio instance with TLS connections. User agents where not able to register to kamailio using TLS With the follonwing setup : UDP => Kamailio => TLS Kamailio was able to route from UDP to TLS but not from TLS to UDP. ### Troubleshooting #### Log Messages Here are some unusual logs during the issue : ``` ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 27 , queue entry 0, retries 17, connection 0x7f913e1d7fd8, tcp socket 134, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 23 , queue entry 1, retries 13, connection 0x7f913e1a0eb0, tcp socket 105, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 25 , queue entry 2, retries 11, connection 0x7f913e1ada08, tcp socket 137, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 27 , queue entry 3, retries 9, connection 0x7f913e1c2cf0, tcp socket 170, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 23 , queue entry 0, retries 7, connection 0x7f913e1cb480, tcp socket 281, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 25 , queue entry 1, retries 5, connection 0x7f913e1e4b30, tcp socket 398, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 27 , queue entry 2, retries 3, connection 0x7f913e1e8ef8, tcp socket 406, errno=11 (Resource temporarily unavailable) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 1, socket 23: queue full, 304 requests queued (total handled 425) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 2, socket 25: queue full, 304 requests queued (total handled 4940) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 3, socket 27: queue full, 304 requests queued (total handled 310) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 1, socket 23: queue full, 305 requests queued (total handled 426) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 2, socket 25: queue full, 305 requests queued (total handled 4941) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 3, socket 27: queue full, 305 requests queued (total handled 311) ``` Just before the CRITICAL log, there is this comment in sources : /* FIXME: remove after debugging */ ### Additional Information ``` version: kamailio 5.3.5 (x86_64/linux) 9e70e8 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 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: 9e70e8 compiled on 03:49:09 Jun 24 2020 with /usr/bin/gcc 5.4.0 ``` -- 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/2392

3 7

← Newer
1
...
7
8
9
10
11
12
13
...
25
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 2020