Re: [sr-dev] [kamailio/kamailio] define flag_t as unsigned long (#1288)

2018-04-04 Thread juha-h
Daniel-Constantin Mierla writes: > As I wrote on the mailing list, flags should not be migrated to the > transaction after t_newtran() unless one uses t_flush_flags() -- that > was the desired functionality, otherwise t_flush_flags() has no > purpose. I have used flags for many years and for me i

Re: [sr-dev] [kamailio/kamailio] define flag_t as unsigned long (#1288)

2018-04-04 Thread Daniel-Constantin Mierla
As I wrote on the mailing list, flags should not be migrated to the transaction after t_newtran() unless one uses t_flush_flags() -- that was the desired functionality, otherwise t_flush_flags() has no purpose. You opened another item on this tracker related to this unexpected behaviour (#1490)

Re: [sr-dev] [kamailio/kamailio] define flag_t as unsigned long (#1288)

2018-04-04 Thread juha-h
Reopened #1288. -- 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/1288#event-1558010774___ Kamailio (SER) - Development Mailing List sr-dev

Re: [sr-dev] [kamailio/kamailio] define flag_t as unsigned long (#1288)

2018-04-04 Thread juha-h
As long as xflags do not work the same way as flags, I do not consider this issue closed. Flags do not require call of t_flush_flags() after t_newtrans() in order to make flags available in failure and onreply routes, but xflags does require call of t_flush_xflags(). This makes xflags cumberso

Re: [sr-dev] [kamailio/kamailio] define flag_t as unsigned long (#1288)

2018-04-04 Thread Daniel-Constantin Mierla
Closing, xflags being implemented. -- 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/1288#issuecomment-378830173___ Kamailio (SER) - Develo

Re: [sr-dev] [kamailio/kamailio] define flag_t as unsigned long (#1288)

2018-04-04 Thread Daniel-Constantin Mierla
Closed #1288. -- 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/1288#event-1557989344___ Kamailio (SER) - Development Mailing List sr-dev@l

Re: [sr-dev] [kamailio/kamailio] Load dialogs from db at runtime, without restarting kamailio (#1274)

2018-04-04 Thread paolovisintin
In our case we will have an autoscaling system of Kamailio based on geographic requests and load so we will not be able to know in advance how many instances ok kamailio will be provisioned. As you suggested for this temporary workaround we should have : Server 1 : 1, 101,201,301,401... Server

Re: [sr-dev] [kamailio/kamailio] Load dialogs from db at runtime, without restarting kamailio (#1274)

2018-04-04 Thread Daniel-Constantin Mierla
If there are 3 servers, then the step should at least 3, resulting in: ``` Server1: 1, 4, 7, 10, ... Server2: 2, 5, 8, 11, ... Server3: 3, 6, 9, 12, ... ``` So the start value should be the `index` of the server and the increment value has to be at least the number of servers. There is no confli

Re: [sr-dev] [kamailio/kamailio] Load dialogs from db at runtime, without restarting kamailio (#1274)

2018-04-04 Thread Henning Westerholt
Am Mittwoch, 4. April 2018, 09:52:23 CEST schrieb Daniel-Constantin Mierla: > I just committed the config (and kemi) function `dlg_db_load_callid(val)`, > which should load dialog record from database by matching on callid (it > loads also associated variables). > > While not tested yet given lack

Re: [sr-dev] [kamailio/kamailio] [WIP] tls: add support for OpenSSL engine and private keys in HSM (#1484)

2018-04-04 Thread Henning Westerholt
* About the question: "The files tls_map.{c,h} seems to be imported from external source, being under MIT license. tls module seems to be under BSD, anyone knows if there is any conflict between the two or something needs to be mentioned in the README of the tls module?" Acc

[sr-dev] git:master:4f11fbef: modules: readme files regenerated - dialog ... [skip ci]

2018-04-04 Thread Kamailio Dev
Module: kamailio Branch: master Commit: 4f11fbef1ed90d1366cadfad82fa5afbeb0d56f6 URL: https://github.com/kamailio/kamailio/commit/4f11fbef1ed90d1366cadfad82fa5afbeb0d56f6 Author: Kamailio Dev Committer: Kamailio Dev Date: 2018-04-04T18:46:33+02:00 modules: readme files regenerated - dialog ...

[sr-dev] git:master:7db5d4c4: dialog: docs - documentation for h_id_start and h_id_step parameters

2018-04-04 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 7db5d4c4efb2131a7393e638f109ddbe746ffeff URL: https://github.com/kamailio/kamailio/commit/7db5d4c4efb2131a7393e638f109ddbe746ffeff Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-04-04T18:37:28+02:00 dialog: docs - documen

[sr-dev] git:master:8f9b6ea5: dialog: added h_id_start and h_id_step parameters

2018-04-04 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 8f9b6ea5637b26092e591585b82eb59ed2f42775 URL: https://github.com/kamailio/kamailio/commit/8f9b6ea5637b26092e591585b82eb59ed2f42775 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-04-04T18:37:28+02:00 dialog: added h_id_sta

Re: [sr-dev] kamailio packaging, change in carrierroute Makefile

2018-04-04 Thread Daniel-Constantin Mierla
Hello, ok, merge it if doesn't break anymore the jenking nightly builds -- revert also my revert to Henning's commit. Cheers, Daniel On 04.04.18 16:53, Victor Seva wrote: > Hello, > > 2018-04-03 23:03 GMT+02:00 Daniel-Constantin Mierla >: > > changes expected to a

Re: [sr-dev] kamailio packaging, change in carrierroute Makefile

2018-04-04 Thread Victor Seva
Hello, 2018-04-03 23:03 GMT+02:00 Daniel-Constantin Mierla : > changes expected to affect packaging should be discussed first and then > committed, along with related updates to packaging specs. > > They can break nightly builds for packages, resulting in unnecessary > alerts to the mailing list,

Re: [sr-dev] [kamailio/kamailio] [WIP] tls: add support for OpenSSL engine and private keys in HSM (#1484)

2018-04-04 Thread aalba6675
1. Yes - HSM private keys are stored in worker local memory and are not referenced in old structures during SIP connections. We make one reference during mod_child: we install it into the shmem SSL_CTX structure once (proc_no == 0) just to check the the private key corresponds to the cert; subse

[sr-dev] git:5.1:6770fe86: carrierroute: fixed duplicate options definition error when parsing src file

2018-04-04 Thread Lucian Balaceanu
Module: kamailio Branch: 5.1 Commit: 6770fe866dc6ed301b536a6132dc50e9844e720e URL: https://github.com/kamailio/kamailio/commit/6770fe866dc6ed301b536a6132dc50e9844e720e Author: Lucian Balaceanu Committer: Lucian Balaceanu Date: 2018-04-04T14:04:55+03:00 carrierroute: fixed duplicate options def

Re: [sr-dev] [kamailio/kamailio] Kamailio start failed as carrierrouter (#1492)

2018-04-04 Thread Lucian Balaceanu
Closed #1492. -- 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/1492#event-1556172938___ Kamailio (SER) - Development Mailing List sr-dev@l

Re: [sr-dev] [kamailio/kamailio] Kamailio start failed as carrierrouter (#1492)

2018-04-04 Thread Oleh Horbachov
Hi @lbalaceanu . I rebuilt kamailio 5.1.2 with your fix. Problem solved. I think you can close issue. 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/1492#issuecomment-3

Re: [sr-dev] [kamailio/kamailio] ndb_redis: adds support for sentinel (#1493)

2018-04-04 Thread Julien Ammous
After a quick look on redis website the sentinel is shipped with redis since 2.8 so I suppose this is the minimum required but if the sentinel options are not used the driver should work as before with any redis version. -- You are receiving this because you are subscribed to this thread. Reply

[sr-dev] [kamailio/kamailio] RPM spec update (#1494)

2018-04-04 Thread sergey-safarov
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1494 -- Commit Summary -- * pkg/kamailio/obs: packaged db_redis module * pkg/kamailio/obs: Added acc_json module * pkg/kamailio/obs: Added pua_json module * pkg/kamailio/obs: Remo

Re: [sr-dev] [kamailio/kamailio] [WIP] tls: add support for OpenSSL engine and private keys in HSM (#1484)

2018-04-04 Thread Daniel-Constantin Mierla
Just to confirm I haven't missed something -- the private keys stored in worker-local memory refer to keeping them in the map structure you introduced with the new files tls_map.{c,h}. They are not referenced from old structures of the tls module, right? I see that the define conditions are on

Re: [sr-dev] [kamailio/kamailio] ndb_redis: adds support for sentinel (#1493)

2018-04-04 Thread Daniel-Constantin Mierla
Thanks! One question before attempting to merge, does this require a specific version of redis server or redis client library? -- 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/1493#is

Re: [sr-dev] [kamailio/kamailio] Kamailio start failed as carrierrouter (#1492)

2018-04-04 Thread Lucian Balaceanu
I have pushed https://github.com/kamailio/kamailio/commit/dc8faaf57a9c85b7133f112bdedd328a9520eb73 on the master branch. For me it solved the problem. Could you confirm that this is also the case for you so that we can close the issue? -- You are receiving this because you are subscribed to th

[sr-dev] git:master:dc8faaf5: carrierroute: fixed duplicate options definition error when parsing src file

2018-04-04 Thread Lucian Balaceanu
Module: kamailio Branch: master Commit: dc8faaf57a9c85b7133f112bdedd328a9520eb73 URL: https://github.com/kamailio/kamailio/commit/dc8faaf57a9c85b7133f112bdedd328a9520eb73 Author: Lucian Balaceanu Committer: Lucian Balaceanu Date: 2018-04-04T10:52:56+03:00 carrierroute: fixed duplicate options

Re: [sr-dev] [kamailio/kamailio] Load dialogs from db at runtime, without restarting kamailio (#1274)

2018-04-04 Thread Daniel-Constantin Mierla
I just committed the config (and kemi) function `dlg_db_load_callid(val)`, which should load dialog record from database by matching on callid (it loads also associated variables). While not tested yet given lack of proper environment to simulate such scenario (hopefully you can do it), there i

Re: [sr-dev] module FAQ, private contact replacement

2018-04-04 Thread Daniel-Constantin Mierla
Hello, On 04.04.18 08:14, Henning Westerholt wrote: > Am Dienstag, 3. April 2018, 22:59:42 CEST schrieb Daniel-Constantin Mierla: >>> I will change it to involve only security bugs, this way we could easily >>> change it when we have a dedicated security contact address. If we get to >>> much spa

[sr-dev] git:master:b2747551: modules: readme files regenerated - dialog ... [skip ci]

2018-04-04 Thread Kamailio Dev
Module: kamailio Branch: master Commit: b27475511756b04fe4d4ee5f76bcc14e4186bc58 URL: https://github.com/kamailio/kamailio/commit/b27475511756b04fe4d4ee5f76bcc14e4186bc58 Author: Kamailio Dev Committer: Kamailio Dev Date: 2018-04-04T09:01:55+02:00 modules: readme files regenerated - dialog ...