sr-dev April 2020

sr-dev@lists.kamailio.org

33 participants
420 discussions

by Victor Seva

Hi, We started to migrate our Lua scripts to KEMI but we use [0] sr.xavp.get_keys() and sr.xavp.get() functions that return an array and a dictionary. It seems that KEMI has no support for complex data types. So, I think we still need a way to extend the interpreters with richer functions that must be implemented on all flavours or create some c abstractions to support arrays and dictionaries in KEMI. Comments/Ideas? Victor [0] https://www.kamailio.org/wiki/embeddedapi/devel/lua#srxavp

4 years, 1 month

Error in kamailio - 5.3.3

by Pavithra M

Hi, I have configured kamailio as Ims server with pcscf,icscf,scsf and hss in 4 different vms . After that I use zoiper client for testing the call flow . Registration is successful .It goes via UE -> Pcscf -> icscf -> scscf and returns back with 200 OK . Registration is happening with authentication from hss . When I make an INVITE , it goes from UE -> PCSCF -> SCSCF -> ICSCF -> SCSF and s-cscf gives *403 - Domain not served *to icscf and i-cscf returns to p-cscf with the same error . I am not seeing any errors in scscf and i-cscf . In pcscf , I am seeing an error like *Error : <core> [core/lvalue.c] :lval_pvar_assign() : setting pvar failed * *Error : <core> [core/lvalue.c] :* *lval_assign():assignment failed at pos* What should I do? Kindly advise. I am struggling with this issue for a week.kindly do the needful. Version - kamailio - 5.3.3 I need a normal call flow to be established between two clients.kindly help .

4 years, 1 month

git:master:c2512ac4: siptrace: docs for trace_init_mode parameter

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: c2512ac4b93dfc82264fc6d50d9b27a12755980f URL: https://github.com/kamailio/kamailio/commit/c2512ac4b93dfc82264fc6d50d9b27a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-04-07T17:35:25+02:00 siptrace: docs for trace_init_mode parameter --- Modified: src/modules/siptrace/doc/siptrace_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/c2512ac4b93dfc82264fc6d50d9b27a… Patch: https://github.com/kamailio/kamailio/commit/c2512ac4b93dfc82264fc6d50d9b27a… --- diff --git a/src/modules/siptrace/doc/siptrace_admin.xml b/src/modules/siptrace/doc/siptrace_admin.xml index 511dda3a98..2192b1b199 100644 --- a/src/modules/siptrace/doc/siptrace_admin.xml +++ b/src/modules/siptrace/doc/siptrace_admin.xml @@ -462,6 +462,44 @@ modparam("siptrace", "trace_delayed", 1) ... modparam("siptrace", "force_send_sock", "sip:10.1.1.2:5000") ... +</programlisting> + </example> + </section> + <section id="siptrace.p.trace_init_mode"> + <title><varname>trace_init_mode</varname> (integer)</title> + <para> + Control what tracing modes are initialized. + </para> + <para> + The value of the parameter can be a combination of next values: + <itemizedlist> + <listitem> + <para> + 0 - all modes are initialized. + </para> + </listitem> + <listitem> + <para> + 1 - module initialized to do tracing only with core callback + functions (see also 'trace_mode' parameter). + </para> + </listitem> + <listitem> + <para> + 2 - module initialized to do tracing only using config script flags + and functions. + </para> + </listitem> + </itemizedlist> + <para> + Default value is <emphasis>0</emphasis>. + </para> + <example> + <title>Set <varname>trace_init_mode</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("siptrace", "trace_init_mode", 1) +... </programlisting> </example> </section>

4 years, 1 month

git:master:66582c49: rr: exported record_route_advertised_address() to kemi

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 66582c49ee16a15a6859bfb094c2faaed7960d4d URL: https://github.com/kamailio/kamailio/commit/66582c49ee16a15a6859bfb094c2faa… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-04-08T08:49:29+02:00 rr: exported record_route_advertised_address() to kemi --- Modified: src/modules/rr/rr_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/66582c49ee16a15a6859bfb094c2faa… Patch: https://github.com/kamailio/kamailio/commit/66582c49ee16a15a6859bfb094c2faa… --- diff --git a/src/modules/rr/rr_mod.c b/src/modules/rr/rr_mod.c index b7b979f25e..236373a288 100644 --- a/src/modules/rr/rr_mod.c +++ b/src/modules/rr/rr_mod.c @@ -453,6 +453,23 @@ static int w_record_route_advertised_address(struct sip_msg *msg, char *addr, ch return 1; } +/** + * + */ +static int ki_record_route_advertised_address(sip_msg_t *msg, str *addr) +{ + if (msg->msg_flags & FL_RR_ADDED) { + LM_ERR("Double attempt to record-route\n"); + return -1; + } + + if ( record_route_advertised_address(msg, addr) < 0) + return -1; + + msg->msg_flags |= FL_RR_ADDED; + return 1; +} + static int w_add_rr_param(struct sip_msg *msg, char *key, char *foo) { @@ -865,6 +882,11 @@ static sr_kemi_t sr_kemi_rr_exports[] = { { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } }, + { str_init("rr"), str_init("record_route_advertised_address"), + SR_KEMIP_INT, ki_record_route_advertised_address, + { SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, + SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } + }, { str_init("rr"), str_init("next_hop_route"), SR_KEMIP_INT, ki_rr_next_hop_route, { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE,

4 years, 1 month

git:master:0edff017: siptrace: skip grepping local socket till sending function

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 0edff017fe4259f01ea62d5aa78d5305a6d8164e URL: https://github.com/kamailio/kamailio/commit/0edff017fe4259f01ea62d5aa78d530… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-04-08T08:49:29+02:00 siptrace: skip grepping local socket till sending function - have a single place where to decide what local socket should be used --- Modified: src/modules/siptrace/siptrace.c --- Diff: https://github.com/kamailio/kamailio/commit/0edff017fe4259f01ea62d5aa78d530… Patch: https://github.com/kamailio/kamailio/commit/0edff017fe4259f01ea62d5aa78d530… --- diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c index 284fa45ba2..c3876c9108 100644 --- a/src/modules/siptrace/siptrace.c +++ b/src/modules/siptrace/siptrace.c @@ -69,7 +69,7 @@ MODULE_VERSION #define SIPTRACE_ANYADDR_LEN (sizeof(SIPTRACE_ANYADDR) - 1) #define trace_is_off(_msg) \ - (((_msg)->msg_flags & FL_SIPTRACE) == 0) + (((_msg)->msg_flags & FL_SIPTRACE) == 0) #define is_null_pv(_str) \ (!str_strcmp(&_str, pv_get_null_str())) @@ -321,7 +321,7 @@ static int mod_init(void) } if(!DB_CAPABILITY(db_funcs, DB_CAP_INSERT)) { LM_ERR("database modules does not provide all functions needed" - " by module\n"); + " by module\n"); return -1; } } @@ -354,7 +354,7 @@ static int mod_init(void) } memset(trace_send_sock_uri, 0, sizeof(sip_uri_t)); if(parse_uri(trace_send_sock_str.s, trace_send_sock_str.len, - trace_send_sock_uri) + trace_send_sock_uri) < 0) { LM_ERR("bad send sock address\n"); return -1; @@ -422,7 +422,7 @@ static int mod_init(void) } if(pv_get_avp_name( - 0, &avp_spec.pvp, &traced_user_avp, &traced_user_avp_type) + 0, &avp_spec.pvp, &traced_user_avp, &traced_user_avp_type) != 0) { LM_ERR("[%.*s] - invalid AVP definition\n", traced_user_avp_str.len, traced_user_avp_str.s); @@ -441,7 +441,7 @@ static int mod_init(void) } if(pv_get_avp_name( - 0, &avp_spec.pvp, &trace_table_avp, &trace_table_avp_type) + 0, &avp_spec.pvp, &trace_table_avp, &trace_table_avp_type) != 0) { LM_ERR("[%.*s] - invalid AVP definition\n", trace_table_avp_str.len, trace_table_avp_str.s); @@ -476,7 +476,7 @@ static int child_init(int rank) db_con = db_funcs.init(&db_url); if(!db_con) { LM_ERR("unable to connect to database. Please check " - "configuration.\n"); + "configuration.\n"); return -1; } if(DB_CAPABILITY(db_funcs, DB_CAP_QUERY)) { @@ -851,9 +851,9 @@ static int sip_trace_helper(sip_msg_t *msg, dest_info_t *dst, str *duri, goto trace_current; } - /* if sip_trace is called over an incoming CANCEL, skip - * capturing it if the cancelled transaction is already being traced - */ + /* if sip_trace is called over an incoming CANCEL, skip + * capturing it if the cancelled transaction is already being traced + */ if (msg->REQ_METHOD==METHOD_CANCEL) { t_invite=tmb.t_lookup_original(msg); if (t_invite!=T_NULL_CELL) { @@ -865,7 +865,7 @@ static int sip_trace_helper(sip_msg_t *msg, dest_info_t *dst, str *duri, tmb.t_unref(msg); } } - + /* if sip_trace is called over an incoming ACK, skip * capturing it if it's an ACK for a negative reply for * an already traced transaction @@ -1112,18 +1112,6 @@ static int sip_trace(sip_msg_t *msg, dest_info_t *dst, return -1; } - if(dst) { - if(dst->send_sock == 0) { - dst->send_sock = get_send_socket(0, &dst->to, dst->proto); - if(dst->send_sock == 0) { - LM_ERR("can't forward to af %d, proto %d no corresponding" - " listening socket\n", - dst->to.s.sa_family, dst->proto); - return -1; - } - } - } - memset(&sto, 0, sizeof(siptrace_data_t)); if(traced_user_avp.n != 0)

4 years, 1 month

Re: [sr-dev] Unable to make SIP Calls (403 Forbidden - Domain not served)

by Pavithra M

Hi , I am also facing the same issue . Could you please tell me the workaround if it worked . Thanks.

4 years, 1 month

git:master:fbd101ee: modules: readme files regenerated - dialog ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: fbd101eebc17b92a80cbf4867527d2ff8075f6e5 URL: https://github.com/kamailio/kamailio/commit/fbd101eebc17b92a80cbf4867527d2f… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2020-04-08T04:16:09+02:00 modules: readme files regenerated - dialog ... [skip ci] --- Modified: src/modules/dialog/README --- Diff: https://github.com/kamailio/kamailio/commit/fbd101eebc17b92a80cbf4867527d2f… Patch: https://github.com/kamailio/kamailio/commit/fbd101eebc17b92a80cbf4867527d2f…

4 years, 1 month

[kamailio/kamailio] dialog: adding dlg.dump (#2272)

by Julien Chavanton

alternative command to output dialogs data to file, far much faster than dlg.list   #### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] New feature (non-breaking change which adds new functionality) #### Checklist:  - [x] Tested changes locally - [x] Related to issue #2269 #### Description This command can output 100K dialogs in less than a second. Almost the exact same format as dlg.list, except no using arrays for `variables` and `profiles` since they can have a mixture of values and key / value pairs. ``` { "h_entry": 16383, "h_id": 4428, "ref": 2, "call_id": "ehm839mk6HwtIXfCpluxkNlbDq8H9im2", "from_uri": "sip:voip_perf@147.75.65.147", "to_uri": "sip:12063722016@52.42.112.121", "state": 4, "start_ts": 1585863821, "init_ts": 1585863821, "end_ts": 0, "timeout": 1585885421, "lifetime": 21600, "dflags": 1539, "sflags": 0, "iflags": 0, "caller": { "tag": "KK4KtrhQ--OjGMBNCvpXfKX-tinxQ3EK", "contact": "sip:voip_perf@147.75.65.147:5072;transport=udp", "cseq": "25945", "route_set": "", "socket": "udp:0.0.0.0:5060" }, "callee": { "tag": "jlf1PLBMkC3utNwg-qwdkolYbaMx0no5", "contact": "sip:voip_perf@147.75.65.147:5060;transport=udp", "cseq": "0", "route_set": "", "socket": "udp:0.0.0.0:5060" }, "profiles": { "outbound_call": "" }, "variables": { "foo": "bar" } } ``` You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2272 -- Commit Summary -- * dialog: adding dlg.dump -- File Changes -- M src/modules/dialog/Makefile (15) M src/modules/dialog/dialog.c (155) M src/modules/dialog/doc/dialog_admin.xml (18) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2272.patch https://github.com/kamailio/kamailio/pull/2272.diff -- 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/pull/2272

4 years, 1 month

[kamailio/kamailio] dialog: extract/dump a large amount (#2269)

by Julien Chavanton

### Description The commands available to extract dialog information `dlg.profile_list` and `dlg.list` Are not fast enough to output a large amount of dialogs. After doing some benchmark the time is spent in in `rpc->struct_add`, `rpc->add etc`, etc. it appears to be slowing down linearly. ### Possible Solutions I made a quick POC to verify what I can get if I output to file instead and I got blazing fast performance, this is good news since it means that iterating over the hash table etc. is super fast. https://github.com/jchavanton/kamailio/commit/f28c5a6b6364d79ee35acb5be439e… ``` dialog: outputing large amount of dialogs to file Test: 1) injecting 100K dialogs with voip_perf 2) confirmation that we have 100K+ dialogs kamcmd dlg.profile_get_size outbound_call 100879 3) execute the quickly hacked modifications to output to /tmp/dialogs.txt kamctl rpc dlg.profile_list outbound_call 3.1) verify how long it took 2020-03-31T22:28:18.398916476Z 12(31) : dialog [dialog.c:2174]: internal_rpc_profile_print_dlgs(): profile->lock 2020-03-31T22:28:18.554379411Z 12(31) : dialog [dialog.c:2190]: internal_rpc_profile_print_dlgs(): profile->lock release // command took less then 20ms 3.2) verify the content of the output wc /tmp/dialogs.txt 124355 117324 20547037 /tmp/dialogs.txt Note about the content // not everything is outputed in this quick hack but we can imagine that the impact will be very low. call-id[uV.YzlhIZd1inE8SYDAlzmOLG12tYqVS]from[sip:voip_perf@147.75.65.247]to[sip:12060412757@52.42.112.21]h_entry[6207]h_id[44579]ref[2]contact[sip:voip_perf@147.75.65.247:5072;transport=udp]cseq[19528] ``` ### Additional Information I am thinking about adding commands to output to file in `json` and/or `csv` -- 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/2269

4 years, 1 month

git:master:776f2f65: Merge pull request #2272 from kamailio/jchavanton/dlg_dump

by GitHub

Module: kamailio Branch: master Commit: 776f2f65e28028e5a67da30a8f10d2917b806108 URL: https://github.com/kamailio/kamailio/commit/776f2f65e28028e5a67da30a8f10d29… Author: Julien Chavanton <jchavanton(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2020-04-07T19:06:40-07:00 Merge pull request #2272 from kamailio/jchavanton/dlg_dump dialog: adding dlg.dump --- Modified: src/modules/dialog/dialog.c Modified: src/modules/dialog/doc/dialog_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/776f2f65e28028e5a67da30a8f10d29… Patch: https://github.com/kamailio/kamailio/commit/776f2f65e28028e5a67da30a8f10d29…

4 years, 1 month

← Newer
1
...
31
32
33
34
35
36
37
...
42
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev April 2020