Re: [sr-dev] [kamailio/kamailio] tm: don't reset uac reply if in a continue route (#1063)

2017-04-08 Thread Federico Cabiddu
Thanks Daniel, I've updated the pull request. -- 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/1063#issuecomment-292703227___ sr-dev

Re: [sr-dev] [kamailio/kamailio] Segmentation fault using http_async_client on kamailio 5.0.0 (#1056)

2017-04-07 Thread Federico Cabiddu
The issue is in the tm module. While processing the resumed reply, since we are in a case of final reply for the transaction, we delete it after sending it out. This is causing the carsh when, at the end of t_continue we try to access the branch's reply. I've open a PR

[sr-dev] [kamailio/kamailio] tm: don't reset uac reply if in a continue route (#1063)

2017-04-07 Thread Federico Cabiddu
Fix for https://github.com/kamailio/kamailio/issues/1056. If the reply is continued we need the reply after sending it out, even if it's a final reply. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1063 -- Commit Summary -- * tm:

Re: [sr-dev] [kamailio/kamailio] Segmentation fault using http_async_client on kamailio 5.0.0 (#1056)

2017-04-07 Thread Federico Cabiddu
I'm sorry, I've been quite busy in the last days. I will try to reproduce/investigate this in the next days. The function should indeed work in the scenario. @davyvdm: to be clear, are you calling http_async_query in a reply route or in a failure route? -- You are receiving this because you

Re: [sr-dev] [kamailio/kamailio] Segmentation fault using http_async_client on kamailio 5.0.0 (#1056)

2017-04-04 Thread Federico Cabiddu
Could you please describe the procedure to reproduce the segfault and, in case, the relevant part of your config? Thanks. -- 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] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
Merged #1024. -- 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/1024#event-1002969820___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
Just updated the PR with your suggestion. -- 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/1024#issuecomment-287024005___ sr-dev mailing

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
@grumvalski pushed 1 commit. d2c3f38 core: compile by default support for SO_REUSEPORT when available on the OS -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
Thanks for the feedback Daniel. About the first point, I think that we still need the #define because if SO_REUSEPORT is not defined for the running OS the compilation will fail. As far as I know the option is currently supported in FreeBSD/OpenBSD/NetBSD/Mac OSX, not in Solaris. Maybe we could

[sr-dev] git:5.0:05bd71e3: pkg/kamailio/centos/7: added jansson package

2017-03-09 Thread Federico Cabiddu
Module: kamailio Branch: 5.0 Commit: 05bd71e3c834ae5fa2991a814634b95cbd45a142 URL: https://github.com/kamailio/kamailio/commit/05bd71e3c834ae5fa2991a814634b95cbd45a142 Author: grumvalski <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> Date: 2017

[sr-dev] [kamailio/kamailio] Centos7 spec fixes (#1025)

2017-03-08 Thread Federico Cabiddu
A couple of patches for Centos 7 spec file. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1025 -- Commit Summary -- * pkg/kamailio/centos/7: added jansson package * pkg/kamailio/centos/7: updated version to 5.1.0-dev1 -- File

[sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-08 Thread Federico Cabiddu
As discussed on the mailing list and at FOSDEM this is a PR to start the discussion on the support of SO_REUSEPORT. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1024 -- Commit Summary -- * core: enable reuse of tcp sockets --

[sr-dev] git:4.4:2e85e0b5: core: initialize dns cache used memory

2017-03-07 Thread Federico Cabiddu
Module: kamailio Branch: 4.4 Commit: 2e85e0b58170e490cf3544eb22d250bb94823474 URL: https://github.com/kamailio/kamailio/commit/2e85e0b58170e490cf3544eb22d250bb94823474 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:5.0:64d0737d: core: initialize dns cache used memory

2017-03-07 Thread Federico Cabiddu
Module: kamailio Branch: 5.0 Commit: 64d0737d4dd20461c9a4053077d8ced2a7f7aaa1 URL: https://github.com/kamailio/kamailio/commit/64d0737d4dd20461c9a4053077d8ced2a7f7aaa1 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] [kamailio/kamailio] core: initialize dns cache used memory (#1023)

2017-03-07 Thread Federico Cabiddu
Without the initialization the used memory counter can have random values thus breaking the dns cache. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1023 -- Commit Summary -- * core: initialize dns cache used memory -- File

[sr-dev] git:4.4:9812a49f: http_async_client: fix memory leak in pkg memory in case of error

2017-03-05 Thread Federico Cabiddu
Module: kamailio Branch: 4.4 Commit: 9812a49fb7fa58b9d9f2b29ac963011d4d684bcb URL: https://github.com/kamailio/kamailio/commit/9812a49fb7fa58b9d9f2b29ac963011d4d684bcb Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:5.0:49bafac8: http_async_client: fix memory leak in pkg memory in case of error

2017-03-05 Thread Federico Cabiddu
Module: kamailio Branch: 5.0 Commit: 49bafac8c6f50a17e44ff3ef350b95d1a51efa8c URL: https://github.com/kamailio/kamailio/commit/49bafac8c6f50a17e44ff3ef350b95d1a51efa8c Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:bd35608f: http_async_client: fix memory leak in pkg memory in case of error

2017-03-05 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: bd35608f1799f27c725815637c2c8e99b32d53a2 URL: https://github.com/kamailio/kamailio/commit/bd35608f1799f27c725815637c2c8e99b32d53a2 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

Re: [sr-dev] [kamailio/kamailio] tm: deep cloning of the request for fake environment (#971)

2017-02-14 Thread Federico Cabiddu
Sorry for the late feedback. I did try it with some of our suspend scenarios and I haven't noticed side effects. No load though. -- 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] tm: deep cloning of the request for fake environment (#971)

2017-02-08 Thread Federico Cabiddu
seems ok to me. -- 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/971#issuecomment-278269542___ sr-dev mailing list

Re: [sr-dev] [kamailio/kamailio] pkg/kamailio/centos/7: added missing kamailio.service file (#968)

2017-02-04 Thread Federico Cabiddu
I don't think the Debian service file is fine for Centos. For example the path of the environment file is different in the two systems: /etc/default for Debian, /etc/sysconfig for Centos. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view

[sr-dev] [kamailio/kamailio] pkg/kamailio/centos/7: added missing kamailio.service file (#968)

2017-02-04 Thread Federico Cabiddu
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/968 -- Commit Summary -- * pkg/kamailio/centos/7: added missing kamailio.service file -- File Changes -- A pkg/kamailio/centos/7/kamailio.service (17) -- Patch Links --

[sr-dev] [kamailio/kamailio] pkg/kamailio/centos/7: added http_async_client package, fix http_clie… (#967)

2017-02-04 Thread Federico Cabiddu
…nt package You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/967 -- Commit Summary -- * pkg/kamailio/centos/7: added http_async_client package, fix http_client package -- File Changes -- M pkg/kamailio/centos/7/kamailio.spec

Re: [sr-dev] [kamailio/kamailio] tm: reset T_ASYNC_CONTINUE flag in t_suspend (#955)

2017-01-30 Thread Federico Cabiddu
Merged #955. -- You are receiving this because you commented. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/955#event-941479429___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] tm: reset T_ASYNC_CONTINUE flag in t_suspend (#955)

2017-01-30 Thread Federico Cabiddu
Thanks, should I backport it too? On Mon, Jan 30, 2017 at 4:06 PM, Daniel-Constantin Mierla < notificati...@github.com> wrote: > It's a bit foggy the purpose right now, or better said the duration when > that flag is needed. I didn't have the time to dig into it too much and > don't remember the

Re: [sr-dev] [kamailio/kamailio] tm: reset T_ASYNC_CONTINUE flag in t_suspend (#955)

2017-01-30 Thread Federico Cabiddu
Thinking it better, wouldn't moving the flag reset at the end of t_continue make the check in t_reply useless? I have to admit that I don't fully understand that check anyway :) On Mon, Jan 30, 2017 at 1:21 PM, Federico Cabiddu <notificati...@github.com> wrote: > You're right, I'll try

Re: [sr-dev] [kamailio/kamailio] tm: reset T_ASYNC_CONTINUE flag in t_suspend (#955)

2017-01-30 Thread Federico Cabiddu
You're right, I'll try this 2nd approach. On Mon, Jan 30, 2017 at 1:19 PM, Daniel-Constantin Mierla < notificati...@github.com> wrote: > Unless you think it should be better to be reset when t_continue() is > finished instead of inside t_suspend(), you can merge it. > > — > You are receiving

Re: [sr-dev] [kamailio/kamailio] tm: reset T_ASYNC_CONTINUE flag in t_suspend (#955)

2017-01-30 Thread Federico Cabiddu
Currently it's only used in that t_reply check. On Mon, Jan 30, 2017 at 1:01 PM, Daniel-Constantin Mierla < notificati...@github.com> wrote: > Probably it's ok -- couldn't see a side effect right now. Is it > tested/used anywhere else than src/modules/tm/t_reply.c:1123? > > — > You are receiving

[sr-dev] [kamailio/kamailio] tm: reset T_ASYNC_CONTINUE flag in t_suspend (#955)

2017-01-29 Thread Federico Cabiddu
Related to http://lists.sip-router.org/pipermail/sr-users/2017-January/095772.html Resetting the flag in t_suspend allows adding new branches in a failure route while suspending the transaction. You can view, comment on, or merge this pull request online at:

[sr-dev] git:master:dcb82e22: cnxcc: removed usage of mi

2017-01-04 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: dcb82e22ce1c4b79a5992ac55a2ccce5bcdaabfa URL: https://github.com/kamailio/kamailio/commit/dcb82e22ce1c4b79a5992ac55a2ccce5bcdaabfa Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

Re: [sr-dev] [kamailio/kamailio] http_async_client: ensure username and password initialized (#897)

2016-12-21 Thread Federico Cabiddu
Merged #897. -- 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/897#event-902552607___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] http_async_client: ensure username and password initialized (#897)

2016-12-21 Thread Federico Cabiddu
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/897#issuecomment-268721578___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] source port for TCP & TLS connections

2016-12-07 Thread Federico Cabiddu
Hi, I've been playing with SO_REUSEPORT (included in kernel since 3.9) and Kamailio quite successfully. I have a branch for this in my personal fork ( https://github.com/grumvalski/kamailio/tree/tcp_reuseport) that I've been planning to submit as a PR since a while. If you agree I can do it to

Re: [sr-dev] [kamailio/kamailio] Sipcapture fixes (#874)

2016-12-01 Thread Federico Cabiddu
Should I adapt the fixes for 4.4? Being a fix I'd say yes. On Thu, Dec 1, 2016 at 1:55 PM, Alexandr Dubovikov wrote: > Merged #874 . > > — > You are receiving this because you authored the thread. > Reply to this email

[sr-dev] [kamailio/kamailio] Sipcapture fixes (#874)

2016-11-30 Thread Federico Cabiddu
Two fixes in this PR. The first is to perform the ntohs also on hep messaging carrying payload other than SIP. The scond is to take in count the definition of captid chunk in HEPv3 (uint32 BE) different from the one for HEPv2 (uint16 LE). You can view, comment on, or merge this pull request

[sr-dev] git:4.4:0cd00fc0: modules/tmx: fix mi command t_uac_cancel

2016-10-19 Thread Federico Cabiddu
Module: kamailio Branch: 4.4 Commit: 0cd00fc0c84b1d3d80fff9ae809835057bbe6a4e URL: https://github.com/kamailio/kamailio/commit/0cd00fc0c84b1d3d80fff9ae809835057bbe6a4e Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

Re: [sr-dev] [kamailio/kamailio] modules/tmx: fix mi command t_uac_cancel (#825)

2016-10-19 Thread Federico Cabiddu
Merged #825. -- 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/825#event-828805723___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:master:81be737f: modules/tmx: fix mi command t_uac_cancel

2016-10-19 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 81be737f29cbb9e4fd67a09cf2207150b2649e91 URL: https://github.com/kamailio/kamailio/commit/81be737f29cbb9e4fd67a09cf2207150b2649e91 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:4.4:df4bba3f: tmx: fix condition for private copy of the message

2016-10-19 Thread Federico Cabiddu
Module: kamailio Branch: 4.4 Commit: df4bba3f82b97fb9dc1b86b681060b8250a9be20 URL: https://github.com/kamailio/kamailio/commit/df4bba3f82b97fb9dc1b86b681060b8250a9be20 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] [kamailio/kamailio] modules/tmx: fix mi command t_uac_cancel (#825)

2016-10-19 Thread Federico Cabiddu
fix for mi command t_uac_cancel: without calling prepare_to_cancel, the following call to cancel_uacs gives this error: BUG: tm [t_cancel.c:327]: cancel_branch(): tm: cancel_branch: local_cancel buffer=(nil) != BUSY_BUFFER (trying to continue) and it does not cancel the transaction. You can

Re: [sr-dev] [kamailio/kamailio] tmx: fix condition for private copy of the message (#824)

2016-10-17 Thread Federico Cabiddu
Merged #824. -- 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/824#event-825627230___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:master:6913d30c: tmx: fix condition for private copy of the message

2016-10-17 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 6913d30ccf2ea5a818180053453ace4b482b8d30 URL: https://github.com/kamailio/kamailio/commit/6913d30ccf2ea5a818180053453ace4b482b8d30 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] [kamailio/kamailio] tmx: fix condition for private copy of the message (#824)

2016-10-17 Thread Federico Cabiddu
porting of the changes done for ims_auth and ims_registrar_scscf modules. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/824 -- Commit Summary -- * tmx: fix condition for private copy of the message -- File Changes -- M

[sr-dev] git:master:905961bd: modules/ims_auth, modules/ims_registrar_scscf: fix condition for private copy of the msg

2016-10-14 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 905961bd4aaa69139ea5da22466d97e44b3b18b2 URL: https://github.com/kamailio/kamailio/commit/905961bd4aaa69139ea5da22466d97e44b3b18b2 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

Re: [sr-dev] [kamailio/kamailio] modules/ims_auth, modules/ims_registrar_scscf: fix condition for priv… (#818)

2016-10-14 Thread Federico Cabiddu
Merged #818. -- 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/818#event-823784264___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] modules/ims_auth, modules/ims_registrar_scscf: fix condition for priv… (#818)

2016-10-14 Thread Federico Cabiddu
Thanks. I will try to have a look on tmx as soon as possible. -- 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] modules/ims_auth, modules/ims_registrar_scscf: fix condition for priv… (#818)

2016-10-10 Thread Federico Cabiddu
You're right, tmx too could be affected by the same issue. In my case the memory allocated for the transaction and the uas request was the same as for a recently closed transaction. And adding the check for the message id did not completely solve the issue, because of the its per-process nature

[sr-dev] [kamailio/kamailio] modules/ims_auth, modules/ims_registrar_scscf: fix condition for priv… (#818)

2016-10-09 Thread Federico Cabiddu
…ate copy of the msg I've been facing recently some inconsistent replies generated from the two modules. Those replies didn't belong to the current request, but to some old transaction. The issue is due to the check made to decide if the current private copy of the message is consistent with

Re: [sr-dev] Execute action when user is not registered

2016-09-20 Thread Federico Cabiddu
Hi, you can achieve this in Kamailio suspending the request's transaction and resuming it later when a REGISTER comes from the callee's device. Have a look at my presentation at Kamailio World 2015: http://www.kamailio.org/events/2015-KamailioWorld/Day2/20-Federico.Cabiddu-

[sr-dev] git:master:61adc6e1: http_async_client: set request's body through $http_req pseudovariable

2016-09-16 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 61adc6e16c73070bd3f8a9a04efa18263793a270 URL: https://github.com/kamailio/kamailio/commit/61adc6e16c73070bd3f8a9a04efa18263793a270 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:026eadb8: http_async_client: added pseudo variable http_req_id

2016-09-08 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 026eadb87b64a6646fb460c88aa70439d5f9c792 URL: https://github.com/kamailio/kamailio/commit/026eadb87b64a6646fb460c88aa70439d5f9c792 Author: Federico Cabiddu <fcabi...@libon.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> Date: 2016

Re: [sr-dev] [kamailio/kamailio] Problem when using http_async_client with a POST (#744)

2016-08-12 Thread Federico Cabiddu
Closed #744. -- You are receiving this because you commented. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/744#event-753962804___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] Problem when using http_async_client with a POST (#744)

2016-08-12 Thread Federico Cabiddu
Ok, thank you, going to backport. On Fri, Aug 12, 2016 at 2:51 PM, AlainDubois78 wrote: > I actually already tested as I made the proposed modification on my own PC. > I works just as expected > Thanks for the quick feedback > > — > You are receiving this because you

Re: [sr-dev] [kamailio/kamailio] Problem when using http_async_client with a POST (#744)

2016-08-12 Thread Federico Cabiddu
Definitely a bug, thanks for reporting! I've just pushed a fix on master. If you could try and give feedback I'll backport then. On Fri, Aug 12, 2016 at 2:25 PM, AlainDubois78 wrote: > On a Linux 64 bits Ubuntu, I'm trying to use the http_async_client to send > a POST

[sr-dev] git:master:eb6b48f0: http_async_client: don't stop routing script execution for a POST query if transaction is not suspended

2016-08-12 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: eb6b48f03b794ba7430e50b47402ba9cf091f96d URL: https://github.com/kamailio/kamailio/commit/eb6b48f03b794ba7430e50b47402ba9cf091f96d Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:4.3:68141ae0: tm: check branch uri against current uri when deciding if append a new branch

2016-07-08 Thread Federico Cabiddu
Module: kamailio Branch: 4.3 Commit: 68141ae0610faefafe3de25d98fcc09f9ffb2524 URL: https://github.com/kamailio/kamailio/commit/68141ae0610faefafe3de25d98fcc09f9ffb2524 Author: grumvalski <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> Date: 2016

Re: [sr-dev] [kamailio/kamailio] tm: check branch uri against current uri when deciding if append a ne… (#702)

2016-07-08 Thread Federico Cabiddu
You're right. Maybe adding also the check on the received address? I didn't want to have a too heavy logic there, but it could make sense. On Fri, Jul 8, 2016 at 10:15 AM, Daniel-Constantin Mierla < notificati...@github.com> wrote: > You can merge it. > > I think it covers the most of the cases,

[sr-dev] [kamailio/kamailio] tm: check branch uri against current uri when deciding if append a ne… (#702)

2016-07-08 Thread Federico Cabiddu
…w branch This patch allows t_append_branches to cope with sip.instance usage, at least for the case when the contact uri has changed with regards to a previous registration. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/702 --

Re: [sr-dev] [kamailio/kamailio] http_async_client: Minor code and doc review (#673)

2016-06-16 Thread Federico Cabiddu
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/673#issuecomment-226448144___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] http_async_client: Minor code and doc review (#673)

2016-06-16 Thread Federico Cabiddu
Merged #673. --- 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/673#event-694442774___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] cnxcc: segmentation fault on 2 calls get terminatet - kamailio 4.3.3 (#417)

2016-06-06 Thread Federico Cabiddu
The issue should be solved by https://github.com/kamailio/kamailio/commit/d0bdaa7567240b18794ab7512b09a55767c44240. The fix has been backported to 4.3 and 4.4 branches. --- 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] cnxcc: segmentation fault on 2 calls get terminatet - kamailio 4.3.3 (#417)

2016-06-06 Thread Federico Cabiddu
Closed #417. --- 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/417#event-683332742___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:4.3:5e579714: cnxcc: release credit_data in terminate_all_calls

2016-06-06 Thread Federico Cabiddu
Module: kamailio Branch: 4.3 Commit: 5e5797149bbe2df18014d7c6f88e43e418a237bd URL: https://github.com/kamailio/kamailio/commit/5e5797149bbe2df18014d7c6f88e43e418a237bd Author: grumvalski <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> Date: 2016

[sr-dev] git:4.4:7a0de650: cnxcc: release credit_data in terminate_all_calls

2016-06-06 Thread Federico Cabiddu
Module: kamailio Branch: 4.4 Commit: 7a0de650d5d443f84fd3aeb8d6209663074dc6da URL: https://github.com/kamailio/kamailio/commit/7a0de650d5d443f84fd3aeb8d6209663074dc6da Author: grumvalski <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> Date: 2016

Re: [sr-dev] [kamailio/kamailio] cnxcc: Kamailio crashes with a seg fault at high traffic load (#563)

2016-06-06 Thread Federico Cabiddu
PR https://github.com/kamailio/kamailio/pull/656 have been merged and I will backport it to 4.3 and 4.4 branches. Closing for now. --- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:87d23f9d: Merge pull request #656 from grumvalski/fix-563

2016-06-06 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 87d23f9d9376b6bba56092da30b8f6b5471440c5 URL: https://github.com/kamailio/kamailio/commit/87d23f9d9376b6bba56092da30b8f6b5471440c5 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

Re: [sr-dev] [kamailio/kamailio] cnxcc: Kamailio crashes with a seg fault at high traffic load (#563)

2016-06-06 Thread Federico Cabiddu
Closed #563. --- 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/563#event-682580348___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] cnxcc: release credit_data in terminate_all_calls (#656)

2016-06-06 Thread Federico Cabiddu
Merged #656. --- 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/656#event-682579317___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] cnxcc: Kamailio crashes with a seg fault at high traffic load (#563)

2016-06-05 Thread Federico Cabiddu
@lfzaracho I open a PR for a fix on cnxcc that could be related to this issue. --- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] [kamailio/kamailio] cnxcc: release credit_data in terminate_all_calls (#656)

2016-06-05 Thread Federico Cabiddu
This PR is related to GH #563 and probably to some other cnxcc's crashes that have been reported. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/656 -- Commit Summary -- * cnxcc: release credit_data in terminate_all_calls -- File

Re: [sr-dev] [kamailio/kamailio] Multiple http_async_client operations in single transaction fail (#645)

2016-05-30 Thread Federico Cabiddu
As usual, thanks for the feedback Daniel! I can work on it and report back. --- 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] Multiple http_async_client operations in single transaction fail (#645)

2016-05-29 Thread Federico Cabiddu
Hi, sorry for the late answer, I have been travelling last week. You're right, the problem you are facing does not strictly depend on http_async_client module. Kamailio's async framework doesn't look to me to allow suspending again a transaction after having been resumed, even if it does not

[sr-dev] git:master:2449f859: http_async_client: workaround for dealing with HTTP Via format in replies

2016-05-17 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 2449f8591261b207c091f9a44ce6cf40bdb71ab6 URL: https://github.com/kamailio/kamailio/commit/2449f8591261b207c091f9a44ce6cf40bdb71ab6 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <fcabi...@libon.com> Date: 2016

[sr-dev] git:4.4:63dea59a: http_async_client: workaround for dealing with HTTP Via format in replies

2016-05-17 Thread Federico Cabiddu
Module: kamailio Branch: 4.4 Commit: 63dea59a507c1e949c4eb605ca5b56163b25c8d4 URL: https://github.com/kamailio/kamailio/commit/63dea59a507c1e949c4eb605ca5b56163b25c8d4 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <fcabi...@libon.com> Date: 2016-05

[sr-dev] git:4.3:5f2c257b: cnxcc: change some log levels from INFO to DBG

2016-03-02 Thread Federico Cabiddu
Module: kamailio Branch: 4.3 Commit: 5f2c257b47ecc4f074cd840a944538d0a4affd27 URL: https://github.com/kamailio/kamailio/commit/5f2c257b47ecc4f074cd840a944538d0a4affd27 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:8f209d4d: http_async_client: documentation fixes

2016-02-26 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 8f209d4d5c9c0632fc225d338f29d2d900ed752b URL: https://github.com/kamailio/kamailio/commit/8f209d4d5c9c0632fc225d338f29d2d900ed752b Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:778e4a1e: http_async_client: add "suspend" to $http_req keys

2016-02-26 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 778e4a1e7ea2143089749bab317d87fcfdcd6629 URL: https://github.com/kamailio/kamailio/commit/778e4a1e7ea2143089749bab317d87fcfdcd6629 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:8740bf3d: http_async_client: updated README

2016-02-26 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 8740bf3d636f063cf4759d7ded231f1d86a94a6b URL: https://github.com/kamailio/kamailio/commit/8740bf3d636f063cf4759d7ded231f1d86a94a6b Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:e12bf2cd: http_async_client: some module's logs fixes

2016-02-26 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: e12bf2cd3b7b79b1e31030e555dfe379ed12fe33 URL: https://github.com/kamailio/kamailio/commit/e12bf2cd3b7b79b1e31030e555dfe379ed12fe33 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:50fca238: http_async_client: non-blocking async HTTP client module

2016-02-11 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 50fca238aa618d0c9dbf5c2d3681afa6a4a3a769 URL: https://github.com/kamailio/kamailio/commit/50fca238aa618d0c9dbf5c2d3681afa6a4a3a769 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <fcabi...@libon.com> Date: 2016

[sr-dev] git:master:a5286513: pv: export some PV functions

2016-02-11 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: a5286513c56f2a6e94385cb75bff1fdca3110e93 URL: https://github.com/kamailio/kamailio/commit/a5286513c56f2a6e94385cb75bff1fdca3110e93 Author: Camille Oudot <camille.ou...@orange.com> Committer: Federico Cabiddu <fcabi...@libon.com> Date: 2016-02

Re: [sr-dev] http_async

2016-02-05 Thread Federico Cabiddu
Hi Olle, On Thu, Feb 4, 2016 at 3:30 PM, Olle E. Johansson wrote: > > > On 04 Feb 2016, at 12:23, Camille Oudot > wrote: > > > > Hi Olle, > > > > It was Daniel’s recommendation to load the TLS module first since he > had fixed some generic OpenSSL

[sr-dev] http_async_client: renamed module and functions

2016-02-03 Thread Federico Cabiddu
Hi, we renamed the module and functions according to the discussed guidelines. If you are fine I can merge the module. Do you prefer a squashed commit? Cheers, Federico ___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] http_async_client: renamed module and functions

2016-02-03 Thread Federico Cabiddu
Thank Daniel, merge done. Cheers, Federico On Wed, Feb 3, 2016 at 2:19 PM, Daniel-Constantin Mierla <mico...@gmail.com> wrote: > Hello, > > I am fine to merge what ever way comes more convenient. > > Cheers, > Daniel > > > On 03/02/16 14:16, Federico Ca

[sr-dev] git:master:3ba6c445: Merge pull request #480 from grumvalski/async_http_mod

2016-02-03 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 3ba6c4451a98bf365e18b964354bd08381dd4f8f URL: https://github.com/kamailio/kamailio/commit/3ba6c4451a98bf365e18b964354bd08381dd4f8f Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

Re: [sr-dev] [kamailio] async_http: non-blocking HTTP client module (#480)

2016-02-03 Thread Federico Cabiddu
Merged #480. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/480#event-537615394___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] [kamailio] cnxcc: change some log levels from INFO to DBG (#494)

2016-02-02 Thread Federico Cabiddu
When using redis as cnxcc backend, too much log is printed, specially all the logs related to the periodic user's wallet check. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/494 -- Commit Summary -- * cnxcc: change some log levels

Re: [sr-dev] [kamailio] cnxcc: change some log levels from INFO to DBG (#494)

2016-02-02 Thread Federico Cabiddu
Thanks, can I also backport it to 4.3 branch? --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/494#issuecomment-178528033___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio] cnxcc: change some log levels from INFO to DBG (#494)

2016-02-02 Thread Federico Cabiddu
Merged #494. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/494#event-535722102___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:319904b7: Merge pull request #494 from grumvalski/cnxcc_log

2016-02-02 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 319904b7fb674855d72461550c40f0c3adeab5cc URL: https://github.com/kamailio/kamailio/commit/319904b7fb674855d72461550c40f0c3adeab5cc Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

[sr-dev] git:master:f604bc4b: cnxcc: change some log levels from INFO to DBG

2016-02-02 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: f604bc4baf7b3332e661dba1c935a259112a4e72 URL: https://github.com/kamailio/kamailio/commit/f604bc4baf7b3332e661dba1c935a259112a4e72 Author: Federico Cabiddu <federico.cabi...@gmail.com> Committer: Federico Cabiddu <federico.cabi...@gmail.com> D

Re: [sr-dev] http_query/Curl Async Module

2016-01-29 Thread Federico Cabiddu
Thank you Hugh! See you at FOSDEM :) Cheers, Federico On Fri, Jan 29, 2016 at 10:55 AM, Waite, Hugh wrote: > I look forward to discussing this at FOSDEM (which is only tomorrow!). > > I’ve started a wiki page > http://www.kamailio.org/wiki/devel/libcurl_integration which

Re: [sr-dev] [kamailio] async_http: non-blocking HTTP client module (#480)

2016-01-28 Thread Federico Cabiddu
Some commits following received suggestions: - allow global and per query cert and key configuration - memory manager configuration - allow to send a query without suspending the transaction --- Reply to this email directly or view it on GitHub:

Re: [sr-dev] http_query/Curl Async Module : custom http headers feature request

2016-01-27 Thread Federico Cabiddu
Agree, FOSDEM will be a perfect opportunity for discussing about this! Cheers, Federico On Wed, Jan 27, 2016 at 3:47 PM, Olle E. Johansson wrote: > > On 27 Jan 2016, at 15:07, Waite, Hugh wrote: > > I should add, I had it in the pipeline for the current

Re: [sr-dev] http_query/Curl Async Module : custom http headers feature request

2016-01-27 Thread Federico Cabiddu
Hi, this feature is implemented in the new async_http module (the name is provisional). Hopefully it will be merged soon. Here you can find the pull request https://github.com/kamailio/kamailio/pull/480 Regards, Federico On Wed, Jan 27, 2016 at 2:53 PM, Tim Chubb

Re: [sr-dev] [kamailio] async_http: non-blocking HTTP client module (#480)

2016-01-20 Thread Federico Cabiddu
Thanks Hugh for the feedback! On Wed, Jan 20, 2016 at 5:34 PM, Hugh Waite wrote: > Thanks for this module - a very useful feature! > I have a couple of use cases for curl/async_http which are possibly off > topic (major enhancements), I mention them here so that if

Re: [sr-dev] [kamailio] async_http: non-blocking HTTP client module (#480)

2016-01-20 Thread Federico Cabiddu
Thanks for the feedback Daniel. I just pushed 2 commits from Camille to implement pv module api and use it in the new module. I'll also add a note on the documentation about the conflict with the curl module. About the name, http_client could be a good one, the current one having been chosen

[sr-dev] [kamailio] async_http: non-blocking HTTP client module (#480)

2016-01-18 Thread Federico Cabiddu
This new module, based on libevent and cURL multi interface, implements non blocking HTTP queries. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/480 -- Commit Summary -- * pv: expose some PV API function * async_http:

[sr-dev] New async HTTP module

2016-01-18 Thread Federico Cabiddu
Hi all, with two colleagues (Camille Oudot and Giacomo Vacca) we have been working to an asynchronous HTTP module. The module uses libevent (http://libevent.org/) and cURL's multi inteface ( http://curl.haxx.se/libcurl/c/libcurl-multi.html). It instantiates N workers at startup, each able to

Re: [sr-dev] [kamailio] async_http: non-blocking HTTP client module (#480)

2016-01-18 Thread Federico Cabiddu
I agree on having a single curl based module. I started looking at curl module when this module was almost finished, in its core part, with this purpose but, due to the chronic lack of time, the result was that the job on the async module was stagnating beacuse an integration would have meant a

  1   2   3   >