sr-dev August 2024

sr-dev@lists.kamailio.org

10 participants
157 discussions

git:master:3acf1296: modules: readme files regenerated - siputils ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 3acf129660be4e2d9f4dfa68ad3ec6d25c43fbf1 URL: https://github.com/kamailio/kamailio/commit/3acf129660be4e2d9f4dfa68ad3ec6d… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-08-26T15:46:11+02:00 modules: readme files regenerated - siputils ... [skip ci] --- Modified: src/modules/siputils/README --- Diff: https://github.com/kamailio/kamailio/commit/3acf129660be4e2d9f4dfa68ad3ec6d… Patch: https://github.com/kamailio/kamailio/commit/3acf129660be4e2d9f4dfa68ad3ec6d… --- diff --git a/src/modules/siputils/README b/src/modules/siputils/README index d3eda806d4d..f66c7391747 100644 --- a/src/modules/siputils/README +++ b/src/modules/siputils/README @@ -107,6 +107,7 @@ Gabriel Vasile 4.41. contact_param_rm(pname) 4.42. contact_param_check(pname) 4.43. hdr_date_check(tdiff) + 4.44. sip_parse_headers() 5. Exported pseudo-variables @@ -170,6 +171,7 @@ Gabriel Vasile 1.50. contact_param_rm usage 1.51. contact_param_check usage 1.52. hdr_date_check usage + 1.53. sip_parse_headers usage Chapter 1. Admin Guide @@ -238,6 +240,7 @@ Chapter 1. Admin Guide 4.41. contact_param_rm(pname) 4.42. contact_param_check(pname) 4.43. hdr_date_check(tdiff) + 4.44. sip_parse_headers() 5. Exported pseudo-variables @@ -455,6 +458,7 @@ modparam("siputils", "e164_max_len", 20) 4.41. contact_param_rm(pname) 4.42. contact_param_check(pname) 4.43. hdr_date_check(tdiff) + 4.44. sip_parse_headers() 4.1. options_reply() @@ -1203,6 +1207,19 @@ if (!hdr_date_check("10")) { } ... +4.44. sip_parse_headers() + + Return true if parsing (fast mode) all SIP headers is successful. + + This function can be used from ANY_ROUTE. + + Example 1.53. sip_parse_headers usage +... +if (sip_parse_headers()) { + ... +} +... + 5. Exported pseudo-variables 5.1. $pcv(all)

3 weeks, 3 days

git:master:52036e27: siputils: docs for sip_parse_headers() function

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 52036e2772821459daea6d5ac98a3710d9daec13 URL: https://github.com/kamailio/kamailio/commit/52036e2772821459daea6d5ac98a371… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-08-26T15:39:20+02:00 siputils: docs for sip_parse_headers() function --- Modified: src/modules/siputils/doc/siputils_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/52036e2772821459daea6d5ac98a371… Patch: https://github.com/kamailio/kamailio/commit/52036e2772821459daea6d5ac98a371… --- diff --git a/src/modules/siputils/doc/siputils_admin.xml b/src/modules/siputils/doc/siputils_admin.xml index cf08771326c..6850f53e9bf 100644 --- a/src/modules/siputils/doc/siputils_admin.xml +++ b/src/modules/siputils/doc/siputils_admin.xml @@ -1474,6 +1474,27 @@ if (!hdr_date_check("10")) { exit; } ... +</programlisting> + </example> + </section> + <section id="siputils.f.sip_parse_headers"> + <title> + <function moreinfo="none">sip_parse_headers()</function> + </title> + <para> + Return true if parsing (fast mode) all SIP headers is successful. + </para> + <para> + This function can be used from ANY_ROUTE. + </para> + <example> + <title><function>sip_parse_headers</function> usage</title> + <programlisting format="linespecific"> +... +if (sip_parse_headers()) { + ... +} +... </programlisting> </example> </section>

3 weeks, 3 days

git:master:c04e2367: siputils: added function to parse headers

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: c04e236721f9572a2655e39b8c5136bd23f95dd3 URL: https://github.com/kamailio/kamailio/commit/c04e236721f9572a2655e39b8c5136b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-08-26T15:00:07+02:00 siputils: added function to parse headers - could be useful after applying changes or replacing message buffer content --- Modified: src/modules/siputils/siputils.c --- Diff: https://github.com/kamailio/kamailio/commit/c04e236721f9572a2655e39b8c5136b… Patch: https://github.com/kamailio/kamailio/commit/c04e236721f9572a2655e39b8c5136b… --- diff --git a/src/modules/siputils/siputils.c b/src/modules/siputils/siputils.c index 372da25b041..ffbeacd00a5 100644 --- a/src/modules/siputils/siputils.c +++ b/src/modules/siputils/siputils.c @@ -111,6 +111,8 @@ static int w_contact_param_check(sip_msg_t *msg, char *pnparam, char *p2); static int w_hdr_date_check(sip_msg_t *msg, char *ptdiff, char *p2); +static int w_sip_parse_headers(sip_msg_t *msg, char *p1, char *p2); + /* Fixup functions to be defined later */ static int fixup_set_uri(void **param, int param_no); static int fixup_free_set_uri(void **param, int param_no); @@ -208,6 +210,7 @@ static cmd_export_t cmds[] = { fixup_spve_null, fixup_free_spve_null, ANY_ROUTE}, {"hdr_date_check", (cmd_function)w_hdr_date_check, 1, fixup_igp_null, fixup_free_igp_null, ANY_ROUTE}, + {"sip_parse_headers", (cmd_function)w_sip_parse_headers, 0, 0, 0, ANY_ROUTE}, {"bind_siputils", (cmd_function)bind_siputils, 1, 0, 0, 0}, @@ -605,6 +608,17 @@ static int w_hdr_date_check(sip_msg_t *msg, char *ptdiff, char *p2) return ki_hdr_date_check(msg, tdiff); } +/** + * + */ +static int w_sip_parse_headers(sip_msg_t *msg, char *p1, char *p2) +{ + if(parse_headers(msg, HDR_EOH_F, 0) < 0) { + return -1; + } + return 1; +} + /** * */

3 weeks, 3 days

[kamailio/kamailio] During de-registration procedure, clean-up is not happening if contact has * value (Issue #3848)

by deepak-git-account

### Description If Deregistration contains * in a contact field as described in RFC-3261, then the Kamailio is not cleaning the in-memory and database record. And the next iteration is not working correctly. Use of the "*" Contact header field value allows a registering UA to remove all bindings associated with an address-of-record without knowing their precise values.  Step:1 REGISTER USER UE-A Step:2: Wait for the certain duration Step:3: De-REGISTER USER UE-A (Contact header is using *) Step:4: Repeat from the step-1 After step:3, clean-up is not happening. Record can be seen in SQL. Note: If de-register contains specific CONTACT header, then cleanup is happening. ### Troubleshooting #### Reproduction  #### Debugging Data  ``` (paste your debugging data here) ``` #### Log Messages  ``` May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: DEBUG: <script>: Contact headervalue: * May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=25 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=26 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: DEBUG: <script>: Full URI from sip:49404965200004@open-ims.test: sip:49404965200004@open-ims.test May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=33 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=34 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=36 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=37 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=38 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=39 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=46 a=16 n=if May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=43 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=54 a=16 n=if May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=46 a=25 n=is_present_hf May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=47 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=50 a=25 n=append_hf May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=54 a=26 n=xlog May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=55 a=25 n=pcscf_save_pending May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: ims_registrar_pcscf [save.c:318]: save_pending(): No contact headers May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: DEBUG: ims_registrar_pcscf [save.c:492]: save_pending(): Error saving pending contact May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=67 a=16 n=if May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=65 a=63 n=assign May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[] c=[] l=0 a=12 n=seturi May 13 19:47:22 openims-tvmcfg-node-focal pcscf/pcscf[35816]: ERROR: *** cfgtrace:dbg_cfg_trace(): request_route=[REGISTER] c=[/usr/local/etc/kamailio/pcscf/route/register.cfg] l=71 a=26 n=xlog ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` kamailio -v version: kamailio 5.7.5 (x86_64/linux) 58499a-dirty flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, MEM_JOIN_FREE, 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, 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: 58499a -dirty compiled on 11:23:55 May 8 2024 with gcc 9.4.0 ``` * **Operating System**:  ``` lsb_release -a No LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.6 LTS Release: 20.04 Codename: focal ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3848 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3848(a)github.com>

3 weeks, 5 days

Re: [kamailio/kamailio] core: main - safety check for invalid syntax in listen parameter (9783c27)

by Henning Westerholt

Right now it will just ignore the additional slash and cut the string there: henning@app01:~/repositories/kamailio/src$ killall kamailio; ./kamailio -l udp:127.0.0.1:5060//te/st -f ../etc/test.cfg -Y /tmp/ -E -L modules Listening on udp: 127.0.0.1:5060 name te Aliases: udp: localhost:5060 I think the logic could be extended easily to allow e.g. a fourth field for the listen, when we need it. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd8… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd864b1de592/145744930(a)github.com>

3 weeks, 6 days

Re: [kamailio/kamailio] core: main - safety check for invalid syntax in listen parameter (9783c27)

by Daniel-Constantin Mierla

Maybe a check should be added that there is no `/` is in the last field now/socket name, so in the future `/` can be used as separator for new fields. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd8… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd864b1de592/145744612(a)github.com>

3 weeks, 6 days

git:master:fad4fbc3: modules: readme files regenerated - tm ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: fad4fbc3131ec261cd552d821eaf5a20157cfc65 URL: https://github.com/kamailio/kamailio/commit/fad4fbc3131ec261cd552d821eaf5a2… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-08-23T12:31:10+02:00 modules: readme files regenerated - tm ... [skip ci] --- Modified: src/modules/tm/README --- Diff: https://github.com/kamailio/kamailio/commit/fad4fbc3131ec261cd552d821eaf5a2… Patch: https://github.com/kamailio/kamailio/commit/fad4fbc3131ec261cd552d821eaf5a2…

3 weeks, 6 days

git:master:f2c31f59: tm: docs for headers_mode parameter

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: f2c31f5948cc8f6df652d193372973b1da748d51 URL: https://github.com/kamailio/kamailio/commit/f2c31f5948cc8f6df652d193372973b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-08-23T12:18:13+02:00 tm: docs for headers_mode parameter --- Modified: src/modules/tm/doc/params.xml --- Diff: https://github.com/kamailio/kamailio/commit/f2c31f5948cc8f6df652d193372973b… Patch: https://github.com/kamailio/kamailio/commit/f2c31f5948cc8f6df652d193372973b… --- diff --git a/src/modules/tm/doc/params.xml b/src/modules/tm/doc/params.xml index 2c42e2cea5e..ffef9a4eec3 100644 --- a/src/modules/tm/doc/params.xml +++ b/src/modules/tm/doc/params.xml @@ -1355,13 +1355,14 @@ modparam("tm", "e2e_cancel_reason", 0) </programlisting> </example> </section> + <section id="e2e_cancel_hop_by_hop"> <title><varname>e2e_cancel_hop_by_hop</varname> (int)</title> <para> Enable/disable sending CANCEL in hop-by-hop mode. </para> <para> - Default value: 1 ( hop-by-hop mode). + Default value: 1 (hop-by-hop mode). </para> <example> <title>Set <varname>e2e_cancel_hop_by_hop</varname> parameter</title> @@ -1373,6 +1374,25 @@ modparam("tm", "e2e_cancel_hop_by_hop", 0) </example> </section> + <section id="headers_mode"> + <title><varname>headers_mode</varname> (int)</title> + <para> + Control if headers from incoming CANCEL should be propagated to + hop-by-hop generated CANCEL. + </para> + <para> + Default value: 0 (do not propagate). + </para> + <example> + <title>Set <varname>headers_mode</varname> parameter</title> + <programlisting> +... +modparam("tm", "headers_mode", 1) +... + </programlisting> + </example> + </section> + <section id="remap_503_500"> <title><varname>remap_503_500</varname> (boolean)</title> <para>

3 weeks, 6 days

git:master:e84ef549: tm: new parameter headers_mode

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: e84ef54979dd05e92c65f96c7b1336aefd28faa2 URL: https://github.com/kamailio/kamailio/commit/e84ef54979dd05e92c65f96c7b1336a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-08-23T12:18:13+02:00 tm: new parameter headers_mode - control if headers from incoming CANCEL should be propagated to hop-by-hop generated CANCEL - default 0, do not propagate (exiting behaviour) --- Modified: src/modules/tm/t_cancel.c Modified: src/modules/tm/t_cancel.h Modified: src/modules/tm/t_fwd.c Modified: src/modules/tm/t_msgbuilder.c Modified: src/modules/tm/t_msgbuilder.h Modified: src/modules/tm/t_reply.c Modified: src/modules/tm/timer.c Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/e84ef54979dd05e92c65f96c7b1336a… Patch: https://github.com/kamailio/kamailio/commit/e84ef54979dd05e92c65f96c7b1336a…

3 weeks, 6 days

git:master:9783c270: core: main - safety check for invalid syntax in listen parameter

by Henning Westerholt

Module: kamailio Branch: master Commit: 9783c270eae580b22e73349ac72afd864b1de592 URL: https://github.com/kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd8… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-08-23T10:17:02Z core: main - safety check for invalid syntax in listen parameter --- Modified: src/main.c --- Diff: https://github.com/kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd8… Patch: https://github.com/kamailio/kamailio/commit/9783c270eae580b22e73349ac72afd8… --- diff --git a/src/main.c b/src/main.c index cbe67350004..993181c7326 100644 --- a/src/main.c +++ b/src/main.c @@ -2765,7 +2765,7 @@ int main(int argc, char **argv) fprintf(stderr, "error during processing -l parameter\n"); } tbuf_tmp = tbuf; - while ((p = strsep(&tbuf, "/")) != NULL) { + while ((p = strsep(&tbuf, "/")) != NULL && listen_field_count < 3) { listen_fields[listen_field_count++] = p; } /* empty advertise only allowed with a name field */

3 weeks, 6 days

← Newer
1
...
5
6
7
8
9
10
11
...
16
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2024