sr-dev May 2016

sr-dev@lists.kamailio.org

28 participants
348 discussions

git:master:2d1a7e45: evapi: start evapi dispatcher process with rpc rank
by Daniel-Constantin Mierla 30 May '16

30 May '16

Module: kamailio Branch: master Commit: 2d1a7e453301cfdee2c71ccebcd66d86f4c62b3a URL: https://github.com/kamailio/kamailio/commit/2d1a7e453301cfdee2c71ccebcd66d8… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-05-30T12:38:32+02:00 evapi: start evapi dispatcher process with rpc rank - it may be used for handling responses from external app with sip related functions - reported by GH #622 --- Modified: modules/evapi/evapi_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/2d1a7e453301cfdee2c71ccebcd66d8… Patch: https://github.com/kamailio/kamailio/commit/2d1a7e453301cfdee2c71ccebcd66d8… --- diff --git a/modules/evapi/evapi_mod.c b/modules/evapi/evapi_mod.c index 055c9a3..cf29471 100644 --- a/modules/evapi/evapi_mod.c +++ b/modules/evapi/evapi_mod.c @@ -184,7 +184,7 @@ static int child_init(int rank) return 0; } - pid=fork_process(PROC_NOCHLDINIT, "EvAPI Dispatcher", 1); + pid=fork_process(PROC_RPC, "EvAPI Dispatcher", 1); if (pid<0) return -1; /* error */ if(pid==0) {

1 0

[kamailio/kamailio] Multiple http_async_client operations in single transaction fail (#645)
by Thom Seddon 30 May '16

30 May '16

Hi All, I've been doing some work this week with the new http_async_client module, it seems that with it's default behaviour it's not possible to execute another async query in the block immediately following the response, for example the following config: ``` request_route { if (!t_newtran()) { sl_reply_error(); exit(); } http_async_query("http://localhost:8090/req_1", '{"user":"$fU"}', "REQ_1"); } route[REQ_1] { xlog("L_INFO", "route[REQ_1]: INIT\n"); if ($http_ok) { xlog("L_INFO", "route[REQ_1]: status $http_rs\n"); xlog("L_INFO", "route[REQ_1]: Sending REQ_2\n"); http_async_query("http://localhost:8090/req_2", '{"user":"$fU"}', "REQ_2"); } else { xlog("L_INFO", "route[REQ_1]: error $http_err)\n"); } } route[REQ_2] { xlog("L_INFO", "route[REQ_2]: INIT\n"); if ($http_ok) { xlog("L_INFO", "route[REQ_2]: status $http_rs\n"); } else { xlog("L_INFO", "route[REQ_2]: error $http_err)\n"); } } ``` Generates the following error: ``` 10(11832) INFO: <script>: route[REQ_1]: INIT 10(11832) INFO: <script>: route[REQ_1]: status 200 10(11832) INFO: <script>: route[REQ_1]: Sending REQ_2 10(11832) WARNING: tm [t_suspend.c:186]: t_continue(): transaction is not suspended [53195:560306836] ``` Looking at the source, http_async_client calls t_suspend, then goes away and performs the request in the background. On response it calls t_continue and then this block in t_continue will continue execution, the second query will cause http_async_client to call t_suspend whilst sill executing from the previous t_continue. Once they query is sent, the first t_continue will complete, removing the T_ASYNC_SUSPENDED flag. When the second query returns it doesn't find the flag and so fails as shown. I'm far from an expect on this so would really appreciate some pointers, it seems to fundamental to be a race condition in tm but the usage in http_async_client seems to match usage in other modules so I'm not sure on the best tactic. I've tried peppering append_branch and async_route call's in various places to no avail. Full debug log here: http://pastebin.com/raw/rFmm0fmf Any help appreciated, --- 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/645

4 8

git:master:d1fbec62: ndb_mongodb: note about mongodb c driver version and tls
by Daniel-Constantin Mierla 30 May '16

30 May '16

Module: kamailio Branch: master Commit: d1fbec620bdb923aace665170ddabedddd2a3697 URL: https://github.com/kamailio/kamailio/commit/d1fbec620bdb923aace665170ddabed… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-05-30T12:03:02+02:00 ndb_mongodb: note about mongodb c driver version and tls --- Modified: modules/ndb_mongodb/README Modified: modules/ndb_mongodb/doc/ndb_mongodb_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/d1fbec620bdb923aace665170ddabed… Patch: https://github.com/kamailio/kamailio/commit/d1fbec620bdb923aace665170ddabed… --- diff --git a/modules/ndb_mongodb/README b/modules/ndb_mongodb/README index 01b1ce5..e9a9702 100644 --- a/modules/ndb_mongodb/README +++ b/modules/ndb_mongodb/README @@ -10,7 +10,7 @@ Daniel-Constantin Mierla <miconda(a)gmail.com> - Copyright Â© 2014 asipto.com + Copyright © 2014 asipto.com __________________________________________________________________ Table of Contents @@ -98,6 +98,8 @@ Chapter 1. Admin Guide running Kamailio with this module loaded: * mongo-c-driver - available at https://github.com/mongodb/mongo-c-driver + Note: if you use tls module, use at least mongo-c-driver v1.2.1 - + there were reports of issues inside earlier versions of the driver. 3. Parameters @@ -133,7 +135,7 @@ modparam("ndb_mongodb", "server", "name=mgs2;uri='mongodb://127.0.0.2/kamailio'" 4.5. mongodb_next(replyid) 4.6. mongodb_free(replyid) -4.1. mongodb_cmd(srvname, dbname, cname, command, replyid) +4.1. mongodb_cmd(srvname, dbname, cname, command, replyid) Send a valid command to MongoDB server identified by srvname. The reply will be stored in a local container identified by replyid. All the @@ -165,7 +167,7 @@ if(mongodb_cmd("mgs1", "kamailio", "acc", "{ \"collStats\": \"acc\" }", "mgr1")) } ... -4.2. mongodb_cmd_simple(srvname, dbname, cname, command, replyid) +4.2. mongodb_cmd_simple(srvname, dbname, cname, command, replyid) Send a valid command to MongoDB server identified by srvname. The reply will be stored in a local container identified by replyid. All the @@ -191,7 +193,7 @@ mgr1")) { } ... -4.3. mongodb_find(srvname, dbname, cname, command, replyid) +4.3. mongodb_find(srvname, dbname, cname, command, replyid) Send a find command to MongoDB server identified by srvname. The reply will be stored in a local container identified by replyid. All the @@ -220,7 +222,7 @@ if(mongodb_find("mgs1", "kamailio", "acc", "{ \"src_user\" : \"111\" }", "mgr1") } ... -4.4. mongodb_find_one(srvname, dbname, cname, command, replyid) +4.4. mongodb_find_one(srvname, dbname, cname, command, replyid) Similar to mongodb_find(...), but it stops searching after first match, returning the result with one object - faster when expecting to have a @@ -237,7 +239,7 @@ if(mongodb_find_one("mgs1", "kamailio", "subscriber", "{ \"username\" : \"111\" } ... -4.5. mongodb_next(replyid) +4.5. mongodb_next(replyid) Moves to next document in a MongoDB reply. This function can be used after a succesful mongodb_cmd() or mongodb_find(). It returns true if @@ -256,7 +258,7 @@ if(mongodb_find("mgs1", "kamailio", "acc", "{ \"src_user\" : \"111\" }", "mgr1") mongodb_free("mgr1"); ... -4.6. mongodb_free(replyid) +4.6. mongodb_free(replyid) Frees data in a previous reply from memory. After this function call, accessing to a freed replyid returns null value. diff --git a/modules/ndb_mongodb/doc/ndb_mongodb_admin.xml b/modules/ndb_mongodb/doc/ndb_mongodb_admin.xml index ec43474..d6e5a3c 100644 --- a/modules/ndb_mongodb/doc/ndb_mongodb_admin.xml +++ b/modules/ndb_mongodb/doc/ndb_mongodb_admin.xml @@ -52,6 +52,10 @@ <emphasis>mongo-c-driver</emphasis> - available at <ulink url="https://github.com/mongodb/mongo-c-driver">https://github.com/mongodb/mongo-c-driver</ulink> </para> + <para> + Note: if you use tls module, use at least mongo-c-driver v1.2.1 + - there were reports of issues inside earlier versions of the driver. + </para> </listitem> </itemizedlist> </para>

1 0

git:master:59daee29: db_mongodb: note about mongodb c driver version and tls
by Daniel-Constantin Mierla 30 May '16

30 May '16

Module: kamailio Branch: master Commit: 59daee299909a7e27b5d0ec5d9a9967cd1e8e7e3 URL: https://github.com/kamailio/kamailio/commit/59daee299909a7e27b5d0ec5d9a9967… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-05-30T12:02:23+02:00 db_mongodb: note about mongodb c driver version and tls --- Modified: modules/db_mongodb/README Modified: modules/db_mongodb/doc/db_mongodb_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/59daee299909a7e27b5d0ec5d9a9967… Patch: https://github.com/kamailio/kamailio/commit/59daee299909a7e27b5d0ec5d9a9967… --- diff --git a/modules/db_mongodb/README b/modules/db_mongodb/README index b0e9cff..99f6eda 100644 --- a/modules/db_mongodb/README +++ b/modules/db_mongodb/README @@ -10,7 +10,7 @@ Daniel-Constantin Mierla <miconda(a)gmail.com> - Copyright Â© 2014 asipto.com + Copyright © 2014 asipto.com __________________________________________________________________ Table of Contents @@ -79,6 +79,8 @@ Chapter 1. Admin Guide running Kamailio with this module loaded: * mongo-c-driver - available at https://github.com/mongodb/mongo-c-driver + Note: if you use tls module, use at least mongo-c-driver v1.2.1 - + there were reports of issues inside earlier versions of the driver. 4. Usage diff --git a/modules/db_mongodb/doc/db_mongodb_admin.xml b/modules/db_mongodb/doc/db_mongodb_admin.xml index cfa36d8..ef6f1c3 100644 --- a/modules/db_mongodb/doc/db_mongodb_admin.xml +++ b/modules/db_mongodb/doc/db_mongodb_admin.xml @@ -68,6 +68,10 @@ <emphasis>mongo-c-driver</emphasis> - available at <ulink url="https://github.com/mongodb/mongo-c-driver">https://github.com/mongodb/mongo-c-driver</ulink> </para> + <para> + Note: if you use tls module, use at least mongo-c-driver v1.2.1 + - there were reports of issues inside earlier versions of the driver. + </para> </listitem> </itemizedlist> </para>

1 0

[kamailio/kamailio] [Support] Config SIP-I (#641)
by xuansinhmedia 30 May '16

30 May '16

Hi, I am a newbie here. i am using Kamailio version 4.4 on CentOS 6. i need configuration to send INVITE encapsulate ISUP. Please guide me about this configuaration. Thank you. --- 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/641

4 4

git:master:827ec4fc: tm: reset T_ASYNC_SUSPENDED flag at beginning of t_continue()
by Daniel-Constantin Mierla 30 May '16

30 May '16

Module: kamailio Branch: master Commit: 827ec4fc2c9fcc2bf9f814b0545ae64784d8e3b0 URL: https://github.com/kamailio/kamailio/commit/827ec4fc2c9fcc2bf9f814b0545ae64… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-05-30T11:54:34+02:00 tm: reset T_ASYNC_SUSPENDED flag at beginning of t_continue() - should allow doing t_suspend() inside the route executed by t_continue() - related to GH #644 and #645 --- Modified: modules/tm/t_suspend.c --- Diff: https://github.com/kamailio/kamailio/commit/827ec4fc2c9fcc2bf9f814b0545ae64… Patch: https://github.com/kamailio/kamailio/commit/827ec4fc2c9fcc2bf9f814b0545ae64… --- diff --git a/modules/tm/t_suspend.c b/modules/tm/t_suspend.c index 61f742d..828aa38 100644 --- a/modules/tm/t_suspend.c +++ b/modules/tm/t_suspend.c @@ -204,8 +204,11 @@ int t_continue(unsigned int hash_index, unsigned int label, t->flags |= T_ASYNC_CONTINUE; /* we can now know anywhere in kamailio * that we are executing post a suspend */ + /* transaction is no longer suspended, resetting the SUSPEND flag */ + t->flags &= ~T_ASYNC_SUSPENDED; + /* which route block type were we in when we were suspended */ - cb_type = FAILURE_CB_TYPE;; + cb_type = FAILURE_CB_TYPE; switch (t->async_backup.backup_route) { case REQUEST_ROUTE: cb_type = FAILURE_CB_TYPE; @@ -231,7 +234,6 @@ int t_continue(unsigned int hash_index, unsigned int label, /* Either t_continue() has already been * called or the branch has already timed out. * Needless to continue. */ - t->flags &= ~T_ASYNC_SUSPENDED; UNLOCK_ASYNC_CONTINUE(t); UNREF(t); /* t_unref would kill the transaction */ return 1; @@ -456,14 +458,10 @@ int t_continue(unsigned int hash_index, unsigned int label, t->uac[branch].reply = 0; } - /*This transaction is no longer suspended so unsetting the SUSPEND flag*/ - t->flags &= ~T_ASYNC_SUSPENDED; - return 0; kill_trans: - t->flags &= ~T_ASYNC_SUSPENDED; /* The script has hopefully set the error code. If not, * let us reply with a default error. */ if ((kill_transaction_unsafe(t,

1 0

git:master:d85dd4ea: tm: coherent indentation and whitespacing for t suspend code
by Daniel-Constantin Mierla 30 May '16

30 May '16

Module: kamailio Branch: master Commit: d85dd4eab4eaf797ecf89a1d3540c02c207d2de4 URL: https://github.com/kamailio/kamailio/commit/d85dd4eab4eaf797ecf89a1d3540c02… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-05-30T10:35:38+02:00 tm: coherent indentation and whitespacing for t suspend code --- Modified: modules/tm/t_suspend.c Modified: modules/tm/t_suspend.h --- Diff: https://github.com/kamailio/kamailio/commit/d85dd4eab4eaf797ecf89a1d3540c02… Patch: https://github.com/kamailio/kamailio/commit/d85dd4eab4eaf797ecf89a1d3540c02…

1 0

[kamailio/kamailio] $dlg_var variables not available in event rotues (#646)
by Minh PHAN 30 May '16

30 May '16

I'm trying to have an external application to write CDR to a database by doing this: event_route[dialog:start] { evapi_relay("{'cdr_id': $dlg_var(cdr_id), 'event': 'answered'}"); } event_route[dialog:end] { evapi_relay("{'cdr_id': $dlg_var(cdr_id), 'event': 'ended'}"); } event_route[dialog:failed] { evapi_relay("{'cdr_id': $dlg_var(cdr_id), 'event': 'failed'}"); } $dlg_var(cdr_id) is set inside a request route. However, in these event routes, its value is always null. Its seems that $dlg_var variables are not available in those event routes. --- 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/646

1 0

http_async_client segfault
by Jason Penton 30 May '16

30 May '16

Hi, Has anyone experienced segfault using http_async_query from http_async_client module? Here is BT: #0 __GI_fgets_unlocked (buf=0x7fca23ffe660 "127.0.0.1", n=n@entry=1032, fp=0x7fca23ffe660) at iofgets_u.c:53 53 iofgets_u.c: No such file or directory. (gdb) bt #0 __GI_fgets_unlocked (buf=0x7fca23ffe660 "127.0.0.1", n=n@entry=1032, fp=0x7fca23ffe660) at iofgets_u.c:53 #1 0x00007fca68f08afa in get_contents (stream=<optimized out>, len=1032, linebuf=0x7fca23ffe660 "127.0.0.1") at nss_files/files-XXX.c:201 #2 internal_getent (result=0x7fca23ffe5c0, buffer=buffer@entry=0x7fca23ffe640 "\177", buflen=buflen@entry=1064, errnop=errnop@entry=0x7fca23ffebb0, herrnop=herrnop@entry=0x7fca23ffec00, af=af@entry=0, flags=flags@entry=0) at nss_files/files-XXX.c:246 #3 0x00007fca68f09ca3 in _nss_files_gethostbyname4_r (name=name@entry=0x7fca2e790fa0 "localhost", pat=pat@entry=0x7fca23ffebe0, buffer=buffer@entry=0x7fca23ffe640 "\177", buflen=buflen@entry=1064, errnop=errnop@entry=0x7fca23ffebb0, herrnop=herrnop@entry=0x7fca23ffec00, ttlp=ttlp@entry=0x0) at nss_files/files-hosts.c:402 #4 0x00007fca79eb7757 in gaih_inet (name=<optimized out>, name@entry=0x7fca2e790fa0 "localhost", service=<optimized out>, req=req@entry=0x7fca341c7490, pai=pai@entry=0x7fca23ffed10, naddrs=naddrs@entry=0x7fca23ffed00) at ../sysdeps/posix/getaddrinfo.c:850 #5 0x00007fca79ebae0d in __GI_getaddrinfo (name=0x7fca2e790fa0 "localhost", service=0x7fca23ffeec0 "18000", hints=0x7fca341c7490, pai=0x7fca23ffee78) at ../sysdeps/posix/getaddrinfo.c:2406 #6 0x00007fca6e0f80f4 in ?? () from /usr/lib/x86_64-linux-gnu/libcurl.so.4 #7 0x00007fca6e104fa4 in ?? () from /usr/lib/x86_64-linux-gnu/libcurl.so.4 #8 0x00007fca6e10284b in ?? () from /usr/lib/x86_64-linux-gnu/libcurl.so.4 #9 0x00007fca723c9182 in start_thread (arg=0x7fca23fff700) at pthread_create.c:312 #10 0x00007fca79ee547d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111 Camille, are you maintaining this module now?

2 2

git:4.4:e5f76d42: Merge branch '4.4' of https://github.com/kamailio/kamailio into 4.4
by Olle E. Johansson 27 May '16

27 May '16

Module: kamailio Branch: 4.4 Commit: e5f76d42e98b70ad77b275c654b48a7f22b2df66 URL: https://github.com/kamailio/kamailio/commit/e5f76d42e98b70ad77b275c654b48a7… Author: Olle E. Johansson <oej(a)edvina.net> Committer: Olle E. Johansson <oej(a)edvina.net> Date: 2016-05-27T16:14:28+02:00 Merge branch '4.4' of https://github.com/kamailio/kamailio into 4.4 Apologies * '4.4' of https://github.com/kamailio/kamailio: uac: proper skip of uacreg records with empty attributes uac: proper handling of empty attribute for uacreg refresh Makefile.defs: detect gcc 6.x --- Modified: Makefile.defs Modified: modules/uac/uac_reg.c --- Diff: https://github.com/kamailio/kamailio/commit/e5f76d42e98b70ad77b275c654b48a7… Patch: https://github.com/kamailio/kamailio/commit/e5f76d42e98b70ad77b275c654b48a7…

1 0

← Newer
1
...
4
5
6
7
8
9
10
...
35
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev May 2016