[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 Committer: Federico Cabiddu Date: 2016-09-08T09:54:30+02:00 http_async_client: added pseudo

[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 Committer: Federico Cabiddu Date: 2016-09-16T09:46:52+02:00 http_async_client: set request's

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- Kamailio-In-A-Mobile-Wor

[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 th

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

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: https://github.com/kamailio/kamailio/pull/818#issuecomment-253764911___

[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 Committer: Federico Cabiddu Date: 2016-10-10T07:30:29+02:00 modules/ims_auth, modules

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 http://

[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 modules

[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 Committer: Federico Cabiddu Date: 2016-10-17T09:50:36+02:00 tmx: fix condition for private copy

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 http://

[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 vie

[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 Committer: Federico Cabiddu Date: 2016-10-19T11:41:46+02:00 tmx: fix condition for private copy of

[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 Committer: Federico Cabiddu Date: 2016-10-19T11:32:25+02:00 modules/tmx: fix mi command

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 http://

[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 Committer: Federico Cabiddu Date: 2016-10-19T12:00:36+02:00 modules/tmx: fix mi command t_uac_cancel

[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 onlin

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 directly, view it on GitHub >

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 star

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 http://

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 http:

[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 Committer: Federico Cabiddu Date: 2017-01-04T10:27:21+01:00 cnxcc: removed usage of mi

[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: https://github.com/kamailio/kamaili

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

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 this

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 wrote: > You're right, I'll try this 2nd ap

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
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 http://lists.sip-router.org

[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 (26

[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 -- https://

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 i

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 sr-dev@lists.sip-router.o

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: https://github.com/kamailio/kamailio/pull/971#i

[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 Committer: Federico Cabiddu Date: 2017-03-05T10:11:22+01:00 http_async_client: fix memory leak in

[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 Committer: Federico Cabiddu Date: 2017-03-05T10:12:34+01:00 http_async_client: fix memory leak in

[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 Committer: Federico Cabiddu Date: 2017-03-05T10:15:17+01:00 http_async_client: fix memory leak in

[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 Change

[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 Committer: Federico Cabiddu Date: 2017-03-07T19:57:34+01:00 core: initialize dns cache used memory

[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 Committer: Federico Cabiddu Date: 2017-03-07T19:58:20+01:00 core: initialize dns cache used memory

[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 -- Fil

[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 Cha

[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 Committer: Federico Cabiddu Date: 2017-03-09T09:43:04+01:00 pkg/kamailio/centos/7: added jansson package

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

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: https://github.com/kamailio/kamailio/pull/1024/files/78f0108a718a6a68321ff064937fbb0467b6d081

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 lis

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

2017-03-16 Thread Federico Cabiddu
Thank you, I will update the core to ad the new parameter. -- 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-287094617___ s

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 http

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: https://github.com/kamailio/kamailio/issues/1056#issue

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

2017-04-06 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 ar

[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: d

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 https://github.com/kamaili

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

2017-04-08 Thread Federico Cabiddu
The same change is probably also needed on line 1282 of t_reply.c, in case the transaction has failure route set. -- 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-29

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 mailing

Re: [sr-dev] memory leak in memcached module

2013-10-03 Thread Federico Cabiddu
where pv_get_mcd_value_helper is called. Best regards, Federico Cabiddu On Thu, Oct 3, 2013 at 7:27 PM, Charles Chance < charles.cha...@sipcentric.com> wrote: > I can take a look this evening. Assuming nobody has already started? > > Best, > > Charles > On 2 Oct 2013 20:23, "

Re: [sr-dev] memory leak in memcached module

2013-10-04 Thread Federico Cabiddu
Hi Charles, I just tried the patch running the test I used to reproduce the leak. It's ok, the memory debug shows that the memory is correctly released. I tested with memory parameter set to 0 and to 1. Thank you. Best regards, Federico On Fri, Oct 4, 2013 at 1:00 AM, Charles Chance < charles.c

[sr-dev] Permissions and address_reload

2013-12-02 Thread Federico Cabiddu
master branch. I attach a patch for this. Regards, Federico Cabiddu diff --git a/kamailio-4.0.3/modules/permissions/address.c b/kamailio-4.0.3/modules/permissions/address.c index 501d59e..dc39c05 100644 --- a/kamailio-4.0.3/modules/permissions/address.c +++ b/kamailio-4.0.3/modules/permissions

[sr-dev] Dialog timeout and in-dialog requests

2014-01-06 Thread Federico Cabiddu
db schema, so for the moment I left it simple. Regards, Federico Cabiddu From d54bb7f473efe38dc60190974927ac5740cd58d2 Mon Sep 17 00:00:00 2001 From: Federico Cabiddu Date: Mon, 6 Jan 2014 13:15:39 +0100 Subject: [PATCH] modules/dialog: added parameter to enable/disable dialog timeout reset --

Re: [sr-dev] Dialog timeout and in-dialog requests

2014-01-06 Thread Federico Cabiddu
Hi Carlos, thanks for the hint, I will have a look on the module. Regards, Federico Cabiddu On Mon, Jan 6, 2014 at 1:45 PM, Carlos Ruiz Díaz wrote: > Hi, > > maybe cnxcc [1] could help you get through this. > > [1] http://kamailio.org/docs/modules/stable/modules/cnxcc#idm26

Re: [sr-dev] Dialog timeout and in-dialog requests

2014-01-06 Thread Federico Cabiddu
Hi Daniel, thanks for the suggestion, I will try to use the internal flags to store it. Please found attached the previous patch without reformatting. Regards, Federico Cabiddu On Mon, Jan 6, 2014 at 1:46 PM, Daniel-Constantin Mierla wrote: > Hello, > > since it is about 0|1 state,

[sr-dev] Memory stats via SNMP

2014-02-13 Thread Federico Cabiddu
long to. I would tend to put them into KAMAILIO-MIB, under kamailioServer sub-tree. What do you think about? Regards, Federico Cabiddu ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] new module tsilo

2014-09-09 Thread Federico Cabiddu
ful for the community and I look forward to hearing your feedback, thoughts, suggestions. Best regards, Federico Cabiddu ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] new module tsilo

2014-09-10 Thread Federico Cabiddu
> Daniel > > > On 10/09/14 07:36, Federico Cabiddu wrote: > > Hi all Kamailio developers, > I would like to propose a new module that I've written recently, the tsilo > module where tsilo stands for "transaction silo" (thanks Daniel for > suggesting the n

[sr-dev] git:master: tm: new api function t_append_branches

2014-09-10 Thread Federico Cabiddu
Module: sip-router Branch: master Commit: 468a2c645f5fcb66376b81b06e69eb8f5377bb02 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=468a2c645f5fcb66376b81b06e69eb8f5377bb02 Author: Federico Cabiddu Committer: Federico Cabiddu Date: Tue Sep 9 14:14:46 2014 +

[sr-dev] git:master: registrar: new api function lookup_to_dset

2014-09-10 Thread Federico Cabiddu
Module: sip-router Branch: master Commit: ccaa53b1227e6ac516efc42db1315ea96fe4e95a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ccaa53b1227e6ac516efc42db1315ea96fe4e95a Author: Federico Cabiddu Committer: Federico Cabiddu Date: Tue Sep 9 07:32:41 2014 +

[sr-dev] git:master: tsilo: initial commit

2014-09-10 Thread Federico Cabiddu
Module: sip-router Branch: master Commit: 84be6b8e4ddeda5955e67678c8af1080a292b707 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=84be6b8e4ddeda5955e67678c8af1080a292b707 Author: Federico Cabiddu Committer: Federico Cabiddu Date: Wed Sep 10 13:23:10 2014 +0200

[sr-dev] git:master: tsilo: fixed documentation for t_store

2014-09-13 Thread Federico Cabiddu
/modules/tsilo/README +++ b/modules/tsilo/README @@ -24,9 +24,12 @@ Federico Cabiddu 2.2. External libraries or applications 3. Parameters + + 3.1. hash_size (integer) + 4. Functions - 4.1. tstore() + 4.1. t_store

Re: [sr-dev] TSILO or TMSILO or TM_SILO

2014-09-17 Thread Federico Cabiddu
On Wed, Sep 17, 2014 at 1:09 PM, Daniel-Constantin Mierla wrote: > > On 17/09/14 13:01, Olle E. Johansson wrote: > >> On 17 Sep 2014, at 12:54, Daniel-Constantin Mierla >> wrote: >> >> Hello, >>> >>> historically 'tm' stays for 'transaction management' -- the the x for >>> the other module was

[sr-dev] git:master: tsilo: prefixed module's functions with ts_

2014-09-28 Thread Federico Cabiddu
Module: sip-router Branch: master Commit: a135e7956fcf780fd44dddcfbc2c671630b5d44d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a135e7956fcf780fd44dddcfbc2c671630b5d44d Author: Federico Cabiddu Committer: Federico Cabiddu Date: Mon Sep 29 07:57:12 2014 +0200

[sr-dev] tm extra checks for CANCEL

2014-09-28 Thread Federico Cabiddu
Hi, a recent commit on tm module (b66033d) adds a check for cancelled transactions to t_forward_nonack (cf. http://sip-router.org/tracker/index.php?do=details&task_id=468). This made me think that the t_append_branches function, recently added to the module, needs the same kind of checks: - before

[sr-dev] git:master: tm: t_append_branches extra checks for cancelled transaction

2014-09-29 Thread Federico Cabiddu
Module: sip-router Branch: master Commit: ed1a5476e82fa675861a86fb3c7c578555d2 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ed1a5476e82fa675861a86fb3c7c578555d2 Author: fcabiddu Committer: Federico Cabiddu Date: Fri Sep 26 18:19:39 2014 +0200 tm

[sr-dev] [kamailio] ims_auth: fixed crash when receiving a MAA without SIP-Number-Auth-It… (#326)

2015-09-15 Thread Federico Cabiddu
This fix prevent kamailio from crashing when MAA is missing SIP-Number-Auth-Items or SIP-Auth-Data-Item AVPs. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/326 -- Commit Summary -- * ims_auth: fixed crash when receiving a MAA witho

[sr-dev] git:master:5aedfdf3: ims_auth: fixed crash when receiving a MAA without SIP-Number-Auth-Items or SIP-Auth-Data-Item

2015-09-17 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 5aedfdf30088901fa0692d2f04c45883e87612ce URL: https://github.com/kamailio/kamailio/commit/5aedfdf30088901fa0692d2f04c45883e87612ce Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-15T10:51:44+02:00 ims_auth: fixed crash when receiving

[sr-dev] git:master:4c32cfd7: tsilo: updated module's documentation

2015-09-18 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 4c32cfd717bca062d383f5bf19e899606e4a79ec URL: https://github.com/kamailio/kamailio/commit/4c32cfd717bca062d383f5bf19e899606e4a79ec Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-18T10:57:05+02:00 tsilo: updated module's document

[sr-dev] git:master:c89ae44d: tsilo: added module's satistics

2015-09-18 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: c89ae44d3838153284c16006c3689866b87c391a URL: https://github.com/kamailio/kamailio/commit/c89ae44d3838153284c16006c3689866b87c391a Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-18T10:56:54+02:00 tsilo: added module's sati

[sr-dev] git:master:9581d7c0: tm: make t_append_branches return the number of added branches

2015-09-18 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 9581d7c03b21b0f7a360e504126499bef1e37300 URL: https://github.com/kamailio/kamailio/commit/9581d7c03b21b0f7a360e504126499bef1e37300 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-18T10:44:31+02:00 tm: make t_append_branches return the

[sr-dev] git:master:0038fa4d: tsilo: regenerated README

2015-09-18 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 0038fa4de8a6399652d5b6efff3190dcb43fa577 URL: https://github.com/kamailio/kamailio/commit/0038fa4de8a6399652d5b6efff3190dcb43fa577 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-18T10:57:19+02:00 tsilo: regenerated README

[sr-dev] [kamailio] problem with app_perl loading order (#333)

2015-09-18 Thread Federico Cabiddu
All the modules loaded after app_perl have their statistics go under "app_perl". When the mod_init function defined in the module's exports is called for any module after app_perl, exports.name is always "app_perl". Seems that the exports structure is fine in sr_module when we initialize the mod

Re: [sr-dev] git:master:1e5bad01: tsilo: use current r-uri for ts_store()

2015-09-18 Thread Federico Cabiddu
Hi Daniel, I saw it and I'm fine with it. Thank you for the contribution :) Regards, Federico On Fri, Sep 18, 2015 at 3:29 PM, Daniel-Constantin Mierla wrote: > Hi Federico, > > I just pushed a patch to tsilo module to use the current r-uri instead > of the incoming one. As written in the comm

Re: [sr-dev] git:master:19e608f8: tsilot: t_store(...) accepts uri as parameter

2015-09-18 Thread Federico Cabiddu
Hi Daniel, I woke up this morning with the idea to implement this, I swear! I'm just wondering now if it still makes sense to use the current uri by default to store the transactions. If we use the current uri in the "basic" scenario we have to explicitly call ts_store("$ou") after lookup(), while

[sr-dev] git:master:2c736877: app_perl: renamed the exported structure "_app_perl_exports" to avoid naming conflicts

2015-09-21 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 2c736877dc215592f3f8889ffd47e90ab68066af URL: https://github.com/kamailio/kamailio/commit/2c736877dc215592f3f8889ffd47e90ab68066af Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-21T12:43:39+02:00 app_perl: renamed the exported

Re: [sr-dev] [kamailio] problem with app_perl loading order (#333)

2015-09-21 Thread Federico Cabiddu
Solved by commits 6c7d38ee0fcbd482ff0527178481345fc93a7fc8 and 2c736877. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/333#issuecomment-141997465___ sr-dev mailing list sr-dev@lists.sip-router.org http

Re: [sr-dev] [kamailio] problem with app_perl loading order (#333)

2015-09-21 Thread Federico Cabiddu
Closed #333. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/333#event-414800403___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] tsilo - t_append() uri

2015-09-22 Thread Federico Cabiddu
Hi, I'm not sure I fully understand the problem. See my comments inline On Tue, Sep 22, 2015 at 2:24 PM, Daniel-Constantin Mierla wrote: > Hello, > > tsilo is not using the uri parameter for looking up location for new > destinations. It passes the uas request from transaction module to > regist

Re: [sr-dev] tsilo - t_append() uri

2015-09-22 Thread Federico Cabiddu
action->uas (and no explicit > URI parameter). So the lookup is done for > sip:randomstring@deviceip:deviceport), because that is the r-uri in T-uas > reques. Obviously that will fail to find new contacts. > > Hopefully I could expose the issue better now. > > Cheer

[sr-dev] git:master:6bdddb99: tsilo: don't force destination protocol on branches added by t_append_branches

2015-09-22 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 6bdddb99d7bca42897b2c0f96a87c49b97efd8ac URL: https://github.com/kamailio/kamailio/commit/6bdddb99d7bca42897b2c0f96a87c49b97efd8ac Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-22T15:53:28+02:00 tsilo: don't force destin

[sr-dev] git:4.3:9b1b8a70: tsilo: don't force destination protocol on branches added by t_append_branches

2015-09-22 Thread Federico Cabiddu
Module: kamailio Branch: 4.3 Commit: 9b1b8a70aaac447624c4633c169dde3b84542b66 URL: https://github.com/kamailio/kamailio/commit/9b1b8a70aaac447624c4633c169dde3b84542b66 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-22T16:19:03+02:00 tsilo: don't force destination pro

[sr-dev] git:4.2:de64dbef: tsilo: don't force destination protocol on branches added by t_append_branches

2015-09-22 Thread Federico Cabiddu
Module: kamailio Branch: 4.2 Commit: de64dbefeb04889bd0d2477918e3c84abb203d74 URL: https://github.com/kamailio/kamailio/commit/de64dbefeb04889bd0d2477918e3c84abb203d74 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-22T16:19:22+02:00 tsilo: don't force destination pro

Re: [sr-dev] git:master:6bdddb99: tsilo: don't force destination protocol on branches added by t_append_branches

2015-09-22 Thread Federico Cabiddu
I'm sorry for the incorrect commit message, I realized too late that I wrote "tsilo" instead of "tm". Regards, Federico On Tue, Sep 22, 2015 at 3:54 PM, Federico Cabiddu < federico.cabi...@gmail.com> wrote: > Module: kamailio > Branch: master > Commit:

Re: [sr-dev] git:master:cffe6756: tsilo: removed check of use_domain for uri parameter in ts_store()

2015-09-22 Thread Federico Cabiddu
Hi Daniel, at this point I think we can totally remove use_domain and dependency from usrloc module. Do you agree? Regards, Federico On Wed, Sep 23, 2015 at 1:17 AM, Daniel-Constantin Mierla wrote: > Module: kamailio > Branch: master > Commit: cffe67566f206a6c4e76d76a36e0e1783d056369 > URL: >

Re: [sr-dev] [kamailio] tsilo: Fix pointer comparison in ts_set_tm_callbacks (#335)

2015-09-23 Thread Federico Cabiddu
Merged #335. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/335#event-416894996___ 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:b846c216: Merge pull request #335 from doublec/tsilo_pointer_comparison

2015-09-23 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: b846c216f5244049463c49ccefaaadb1c2bcfb10 URL: https://github.com/kamailio/kamailio/commit/b846c216f5244049463c49ccefaaadb1c2bcfb10 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-23T14:08:10+02:00 Merge pull request #335 from doublec

[sr-dev] git:master:c379188e: Merge pull request #336 from doublec/tsilo_null_ptr_dereference

2015-09-23 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: c379188e0f2b7377afae41ee618444c69a8d39e3 URL: https://github.com/kamailio/kamailio/commit/c379188e0f2b7377afae41ee618444c69a8d39e3 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-23T14:22:32+02:00 Merge pull request #336 from doublec

Re: [sr-dev] [kamailio] tsilo: Call LM_DBG with correct transaction information (#336)

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

[sr-dev] [kamailio] tm: fake msg and environment to append new branches (#346)

2015-09-28 Thread Federico Cabiddu
This fix try to solve the crash reported in http://lists.sip-router.org/pipermail/sr-users/2015-September/089846.html. The cause of the crash was that the sip message used to append branch was the shmem clone of the original message. When the new branches were going through their configured br

[sr-dev] git:master:3c086a85: tsilo: unref the transaction after appending new branches

2015-10-07 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 3c086a85b99d59dabc48f77893d4c68199af0577 URL: https://github.com/kamailio/kamailio/commit/3c086a85b99d59dabc48f77893d4c68199af0577 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-10-07T09:29:39+02:00 tsilo: unref the transaction after

Re: [sr-dev] [kamailio] tm: fake msg and environment to append new branches (#346)

2015-10-07 Thread Federico Cabiddu
Thanks for looking at it Daniel. If you prefer I can wait your analysis before merging. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/346#issuecomment-146207063___ sr-dev mailing list sr-dev@lists.sip-r

[sr-dev] git:master:3de200b3: Merge pull request #346 from grumvalski/t_append_branches_fix

2015-10-07 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 3de200b3c965586b3dd9370eca073f10c556e3b3 URL: https://github.com/kamailio/kamailio/commit/3de200b3c965586b3dd9370eca073f10c556e3b3 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-10-07T16:20:43+02:00 Merge pull request #346 from

Re: [sr-dev] [kamailio] tm: fake msg and environment to append new branches (#346)

2015-10-07 Thread Federico Cabiddu
Merged #346. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/346#event-429175307___ 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:6845d190: tm: fake msg and environment to append new branches

2015-10-07 Thread Federico Cabiddu
Module: kamailio Branch: master Commit: 6845d190f46de820fa4586b30eb2fba0aae1eb01 URL: https://github.com/kamailio/kamailio/commit/6845d190f46de820fa4586b30eb2fba0aae1eb01 Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-09-28T09:09:59+02:00 tm: fake msg and environment to

[sr-dev] git:4.3:42f3e1f0: tm: fake msg and environment to append new branches

2015-10-07 Thread Federico Cabiddu
Module: kamailio Branch: 4.3 Commit: 42f3e1f081d2606a500f93e5e1fd8c97690956de URL: https://github.com/kamailio/kamailio/commit/42f3e1f081d2606a500f93e5e1fd8c97690956de Author: Federico Cabiddu Committer: Federico Cabiddu Date: 2015-10-07T16:34:08+02:00 tm: fake msg and environment to append

  1   2   3   >