Re: [sr-dev] [kamailio/kamailio] http_async_client: memory mismanagement (#1407)

2018-01-23 Thread Federico Cabiddu
set_quey_params is called in mod_init by init_query_params (https://github.com/kamailio/kamailio/blob/master/src/modules/http_async_client/async_http.c#L631), so whenever ah_set_req is called the query_params struct should already have been initialized. headers.t is a double pointer, the

Re: [sr-dev] [kamailio/kamailio] presence: dmq integration (#1402)

2018-01-23 Thread Charles Chance
@charlesrchance pushed 3 commits. c21cae5 schema: add ruid column to presentity table 0c2a427 kamctl: regenerated db scripts to include presentity ruid column e60c4a8 presence: added enable_dmq parameter to module docs -- You are receiving this because you are subscribed to this thread.

Re: [sr-dev] [kamailio/kamailio] http_async_client: memory mismanagement (#1407)

2018-01-23 Thread Dragos Oancea
@grumvalski , set_query_params() is called in 3 places, for us kamailio is crashing at startup, and I suspect header_list_add(_params.headers, >rs); is sometimes called before set_query_params(_params) in ah_set_req() , thus leading to a kamailio crash at startup, under traffic. -- You are

Re: [sr-dev] [kamailio/kamailio] presence: dmq integration (#1402)

2018-01-23 Thread Charles Chance
Thanks! -- 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/1402#issuecomment-359867760___ Kamailio (SER) - Development Mailing List

[sr-dev] git:master:43434ee8: modules: readme files regenerated - cfg_rpc ... [skip ci]

2018-01-23 Thread Kamailio Dev
Module: kamailio Branch: master Commit: 43434ee8a1d65276dfbfce2f866f91d4bf0682fc URL: https://github.com/kamailio/kamailio/commit/43434ee8a1d65276dfbfce2f866f91d4bf0682fc Author: Kamailio Dev Committer: Kamailio Dev Date:

Re: [sr-dev] [kamailio/kamailio] cfg_rpc: extending the functionality of cfg.get command (#1321)

2018-01-23 Thread Daniel-Constantin Mierla
The issue with `n = rpc->scan(c, "S*S", , );` was due to a bug in `ctl` module, using `kamcmd` revealed that. Using `kamctl` with `jsonrpcs` module was not affected. The bug is fixed now in master and it will be backported. Besides using the `*` specifier for optional params, your patch in

Re: [sr-dev] [kamailio/kamailio] cfg_rpc: extending the functionality of cfg.get command (#1321)

2018-01-23 Thread Daniel-Constantin Mierla
Closed #1321. -- 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/1321#event-1437725237___ Kamailio (SER) - Development Mailing List

[sr-dev] git:master:a9972863: ctl: proper handling of optional param specifier (*) in rpc scan

2018-01-23 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: a9972863681059c647af4bafd0d9a20f7648c801 URL: https://github.com/kamailio/kamailio/commit/a9972863681059c647af4bafd0d9a20f7648c801 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:95e82d91: cfg_rcp: allow getting 0 values in rpc cfg.get

2018-01-23 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 95e82d9135180be519192029d3f6edcf56eddb59 URL: https://github.com/kamailio/kamailio/commit/95e82d9135180be519192029d3f6edcf56eddb59 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:4920d1c5: cfg_rpc: rpc command cfg.get accepts group name to list its vars

2018-01-23 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 4920d1c5df277526dcb27d34bfedab4cc2815bde URL: https://github.com/kamailio/kamailio/commit/4920d1c5df277526dcb27d34bfedab4cc2815bde Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:e98bd848: cfg_rpc: documentation about group vars listing with rpc cfg.get

2018-01-23 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: e98bd84810b8faa7faf7af3384fc2c979725fe86 URL: https://github.com/kamailio/kamailio/commit/e98bd84810b8faa7faf7af3384fc2c979725fe86 Author: Huseyin Dikme Committer: Daniel-Constantin Mierla Date:

[sr-dev] [kamailio/kamailio] http_async_client: memory mismanagement (#1407)

2018-01-23 Thread Dragos Oancea
Pre-Submission Checklist - [ ] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] No commits to README files

Re: [sr-dev] [kamailio/kamailio] presence: dmq integration (#1402)

2018-01-23 Thread Daniel-Constantin Mierla
For changes to db schema, you have to update the xml files from `src/lib/srdb1/schema/`. The you can run `make dbschema` in the root folder of kamailio source code tree and the sql/db scripts for `kamdbctl` will be updated (they have to be committed, too). -- You are receiving this because

Re: [sr-dev] [kamailio/kamailio] presence: dmq integration (#1402)

2018-01-23 Thread Charles Chance
Thanks - I planned to document the parameter prior to merging. The DB schema definition for presently table needs updating too - where should this be done? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] json: add transformation (#1404)

2018-01-23 Thread Emmanuel Schmidbauer
transformation is not the same -- it is entirely changed to not conflict with `kazoo` or `nsq` example transformations: - json module: `$(rb{json.parse,Event-Package}` - nsq module: `$(rb{nsq.json,Event-Package}` - kazoo module: `$(rb{kz.json,Event-Package}` -- You are receiving this

Re: [sr-dev] [kamailio/kamailio] presence: dmq integration (#1402)

2018-01-23 Thread Daniel-Constantin Mierla
I think the new parameter is not documented, can be done in a follow up commit. If there are no other comments from devs, then it can be merged. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] json: add transformation (#1404)

2018-01-23 Thread Daniel-Constantin Mierla
Is the name of the transformation the same across these modules? If yes, that creates a conflict if all modules are going to be loaded -- I understand you are going to remove from nsq, but there is also in kazoo. -- You are receiving this because you are subscribed to this thread. Reply to

Re: [sr-dev] [SR-Users] Fosdem dinner

2018-01-23 Thread Giovanni Maruzzelli
Hi, please count me too :) -giovanni On 22 January 2018 at 11:13, Alexandr Dubovikov < alexandr.dubovi...@gmail.com> wrote: > Hi, > > count on me as well +2 > > Wbr, > Alexandr > > On 17 January 2018 at 00:34, Julien Chavanton > wrote: > >> Hi Daniel, I will be happy to

[sr-dev] git:5.0:d99e2f81: kazoo: change string parameters to PARAM_STR

2018-01-23 Thread Luis Azedo
Module: kamailio Branch: 5.0 Commit: d99e2f8118dca4cd55a3e4138809cebeb8cdb139 URL: https://github.com/kamailio/kamailio/commit/d99e2f8118dca4cd55a3e4138809cebeb8cdb139 Author: lazedo Committer: Luis Azedo Date: 2018-01-23T10:24:17Z kazoo: change

[sr-dev] git:5.1:47821c58: kazoo: change string parameters to PARAM_STR

2018-01-23 Thread Luis Azedo
Module: kamailio Branch: 5.1 Commit: 47821c5871d188ac04b6e366717a314a8d710699 URL: https://github.com/kamailio/kamailio/commit/47821c5871d188ac04b6e366717a314a8d710699 Author: lazedo Committer: Luis Azedo Date: 2018-01-23T10:24:53Z kazoo: change

[sr-dev] git:master:3018595d: kazoo: change string parameters to PARAM_STR

2018-01-23 Thread lazedo
Module: kamailio Branch: master Commit: 3018595d13dde08d15e8063cecb42a01ea271b66 URL: https://github.com/kamailio/kamailio/commit/3018595d13dde08d15e8063cecb42a01ea271b66 Author: lazedo Committer: lazedo Date: 2018-01-23T10:20:52Z