sr-dev January 2013

sr-dev@lists.kamailio.org

46 participants
325 discussions

git:master: usrloc(k): option to store xavp per contact
by Daniel-Constantin Mierla 03 Jan '13

03 Jan '13

Module: sip-router Branch: master Commit: e6ad428f6699621b7ee622984eeea3e3e2f6cb80 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e6ad428… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Mon Dec 24 10:03:10 2012 +0100 usrloc(k): option to store xavp per contact - stored only in memory for the moment - the xavp can contain a list of xavps - new config parameter to specify the name of xavp --- modules_k/usrloc/README | 107 +++++++++++++++++++-------------- modules_k/usrloc/doc/usrloc_admin.xml | 21 +++++++ modules_k/usrloc/ucontact.c | 46 ++++++++++++++ modules_k/usrloc/ul_mod.c | 6 ++ modules_k/usrloc/ul_mod.h | 1 + modules_k/usrloc/usrloc.h | 9 +++ 6 files changed, 144 insertions(+), 46 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=e6a…

2 3

Kamailio and MariaDB
by Pirjo Ahvenainen 03 Jan '13

03 Jan '13

Hi All, Out of curiosity, can Kamailio be used with MariaDB, or any plans making a connector for it? Cheers, Pirjo -- Pirjo Ahvenainen

2 1

patches to no_naptr_srv_sip_resolvehost()
by Daniel-Constantin Mierla 02 Jan '13

02 Jan '13

Hello, please check the two patches I committed to your recently added function: 1) this is for trying all protocols in SRV -- if I didn't get it wrong, that should be done when no protocol is set. It seemed that was done when a protocol is set. Check it to be sure I haven't misunderstood. http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=17b3c70… 2) catch ip addresses - your function was replacing srv_sip_resolvehost(), which catches IP addresses. no_naptr_srv_sip_resolvehost() is wrapper of srv_sip_resolvehost() only for no-cache mode, for cache mode it calls dns_srv_get_he(). I haven't gotten deep in the code, but Juha reported a dns related error when trying to resolve IP addresses. The only change seems to be the new function introduced. http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b2e5040… Cheers, Daniel -- Daniel-Constantin Mierla - http://www.asipto.com http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda

3 2

avps in onreply_route
by Juha Heinanen 02 Jan '13

02 Jan '13

when reading mediaproxy module ice related sections, i found this: 5.7. ice_candidate_avp (string) Specification of the AVP which holds the ICE candidate that will be inserted in the SDP. The value specified in this AVP will override the value in ice_candidate module parameter. Note that if use_media_proxy() and end_media_session() functions are being used, the AVP will not be available in the reply route unless you set onreply_avp_mode from the tm module to '1', and if the AVP is not set, the default value will be used. kamailio tm module does not have onreply_avp_mode parameter. is it so that avps are automatically available in kamailio onreply_routes? if so, i'll fix the above text. -- juha ps. how mediaproxy module/madiaproxy handle ice makes sense to me, because it avoids the need for a separate turn server with its authentication problems.

2 1

git:master: kamailio.cfg: xhttp left only in kamailio-oob.cfg
by Daniel-Constantin Mierla 02 Jan '13

02 Jan '13

Module: sip-router Branch: master Commit: 8660e2f031a159d0ac1edb87e20738f50a87d5c6 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8660e2f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Jan 2 14:09:53 2013 +0100 kamailio.cfg: xhttp left only in kamailio-oob.cfg --- etc/kamailio.cfg | 57 ------------------------------------------------------ 1 files changed, 0 insertions(+), 57 deletions(-) diff --git a/etc/kamailio.cfg b/etc/kamailio.cfg index e1d7ff1..aa83fad 100644 --- a/etc/kamailio.cfg +++ b/etc/kamailio.cfg @@ -67,18 +67,6 @@ # - define WITH_XMLRPC # - adjust route[XMLRPC] for access policy # -# *** To enable the embedded http server: -# - define WITH_XHTTP -# -# *** To enable the RPC web interface execute: -# - enable xhttp -# - define WITH_XHTTP_RPC -# -# *** To enable the provisioning web interface execute: -# - enable mysql -# - enable xhttp -# - define WITH_XHTTP_PI -# # *** To enable anti-flood detection execute: # - adjust pike and htable=>ipban settings as needed (default is # block if more than 16 requests in 2 seconds and ban for 300 seconds) @@ -186,10 +174,6 @@ enable_tls=yes # - a bit higher than registration expires to cope with UA behind NAT tcp_connection_lifetime=3605 -#!ifdef WITH_XHTTP -tcp_accept_no_cl=yes -#!endif - ####### Custom Parameters ######### # These parameters can be modified runtime via RPC interface @@ -298,18 +282,6 @@ loadmodule "xmlrpc.so" loadmodule "debugger.so" #!endif -#!ifdef WITH_XHTTP -loadmodule "xhttp.so" -#!endif - -#!ifdef WITH_XHTTP_RPC -loadmodule "xhttp_rpc.so" -#!endif - -#!ifdef WITH_XHTTP_PI -loadmodule "xhttp_pi.so" -#!endif - # ----------------- setting module-specific parameters --------------- @@ -473,15 +445,6 @@ modparam("xmlrpc", "url_match", "^/RPC") modparam("debugger", "cfgtrace", 1) #!endif -#!ifdef WITH_XHTTP_RPC -modparam("xhttp_rpc", "xhttp_rpc_root", "http_rpc") -#!endif - -#!ifdef WITH_XHTTP_PI -modparam("xhttp_pi", "xhttp_pi_root", "http_pi") -modparam("xhttp_pi", "framework", "/usr/local/etc/kamailio/pi_framework.xml") -#!endif - ####### Routing Logic ######## @@ -958,23 +921,3 @@ failure_route[MANAGE_FAILURE] { } #!endif } - -#!ifdef WITH_XHTTP -event_route[xhttp:request] { -#!ifdef WITH_XHTTP_RPC - $var(xhttp_rpc_root) = $(hu{s.substr,0,9}); - if ($var(xhttp_rpc_root) == "/http_rpc") { - dispatch_xhttp_rpc(); - } -#!endif -#!ifdef WITH_XHTTP_PI - $var(xhttp_rpc_root) = $(hu{s.substr,0,8}); - if ($var(xhttp_rpc_root) == "/http_pi") { - dispatch_xhttp_pi(); - } -#!endif - xhttp_reply("200", "OK", "text/html", - "<html><body>Wrong URL $hu</body></html>"); -} -#!endif -

1 0

git:master: ims modules: don't link with -lrt on macosx
by Daniel-Constantin Mierla 02 Jan '13

02 Jan '13

Module: sip-router Branch: master Commit: 539f97bdaa4b6c12fec3a943b6adb7902b748827 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=539f97b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Jan 2 14:38:22 2013 +0100 ims modules: don't link with -lrt on macosx - define MAXINT on mac os x --- modules/icscf/Makefile | 6 +++++- modules/icscf/scscf_list.h | 9 ++++++++- modules/isc/Makefile | 6 +++++- modules/registrar_pcscf/Makefile | 6 +++++- modules/registrar_scscf/Makefile | 6 +++++- 5 files changed, 28 insertions(+), 5 deletions(-) diff --git a/modules/icscf/Makefile b/modules/icscf/Makefile index a94dd9c..0c2e907 100644 --- a/modules/icscf/Makefile +++ b/modules/icscf/Makefile @@ -13,7 +13,11 @@ LIBS= DEFS += -DOPENSER_MOD_INTERFACE DEFS += -I/usr/include/libxml2 -LIBS += -L$(LOCALBASE)/lib -lxml2 -lrt +LIBS += -L$(LOCALBASE)/lib -lxml2 + +ifneq ($(OS),darwin) + LIBS += -lrt +endif SERLIBPATH=../../lib diff --git a/modules/icscf/scscf_list.h b/modules/icscf/scscf_list.h index 2911c28..181caee 100644 --- a/modules/icscf/scscf_list.h +++ b/modules/icscf/scscf_list.h @@ -49,14 +49,21 @@ #include "../../sr_module.h" #include "../../modules/tm/tm_load.h" #include "mod.h" +#ifndef __OS_darwin #include <values.h> - +#endif #include "../../mem/shm_mem.h" #include "../../dset.h" #include "../../timer.h" +#ifdef __OS_darwin +#ifndef MAXINT +#define MAXINT INT_MAX +#endif +#endif + /** S-CSCF list element */ typedef struct _scscf_entry { diff --git a/modules/isc/Makefile b/modules/isc/Makefile index 4c8bd32..4f66fbf 100644 --- a/modules/isc/Makefile +++ b/modules/isc/Makefile @@ -12,7 +12,11 @@ LIBS= DEFS += -DOPENSER_MOD_INTERFACE DEFS += -I/usr/include/libxml2 -LIBS += -L$(LOCALBASE)/lib -lxml2 -lrt +LIBS += -L$(LOCALBASE)/lib -lxml2 + +ifneq ($(OS),darwin) + LIBS += -lrt +endif SERLIBPATH=../../lib SER_LIBS+=$(SERLIBPATH)/ims/kamailio_ims diff --git a/modules/registrar_pcscf/Makefile b/modules/registrar_pcscf/Makefile index 54d3100..9169f87 100644 --- a/modules/registrar_pcscf/Makefile +++ b/modules/registrar_pcscf/Makefile @@ -12,7 +12,11 @@ LIBS= DEFS+=-DOPENSER_MOD_INTERFACE -I/usr/include/libxml2 -I$(LOCALBASE)/include/libxml2 -LIBS += -L$(LOCALBASE)/lib -lrt +LIBS += -L$(LOCALBASE)/lib + +ifneq ($(OS),darwin) + LIBS += -lrt +endif SERLIBPATH=../../lib SER_LIBS+=$(SERLIBPATH)/kcore/kcore diff --git a/modules/registrar_scscf/Makefile b/modules/registrar_scscf/Makefile index 651f3e6..f2c046a 100644 --- a/modules/registrar_scscf/Makefile +++ b/modules/registrar_scscf/Makefile @@ -12,7 +12,11 @@ LIBS= DEFS+=-DOPENSER_MOD_INTERFACE DEFS += -I/usr/include/libxml2 -LIBS += -L$(LOCALBASE)/lib -lxml2 -lrt +LIBS += -L$(LOCALBASE)/lib -lxml2 + +ifneq ($(OS),darwin) + LIBS += -lrt +endif SERLIBPATH=../../lib SER_LIBS+=$(SERLIBPATH)/kcore/kcore

1 0

git:master: auth_ims: don't link with -lrt on darwin os
by Daniel-Constantin Mierla 02 Jan '13

02 Jan '13

Module: sip-router Branch: master Commit: cf93bf2097660c093645550f1d3b54fdc1154288 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=cf93bf2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Jan 2 14:22:39 2013 +0100 auth_ims: don't link with -lrt on darwin os --- modules/auth_ims/Makefile | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/modules/auth_ims/Makefile b/modules/auth_ims/Makefile index 09ca633..a70c403 100644 --- a/modules/auth_ims/Makefile +++ b/modules/auth_ims/Makefile @@ -13,7 +13,11 @@ LIBS= DEFS+=-DOPENSER_MOD_INTERFACE DEFS += -I/usr/include/libxml2 -LIBS += -L$(LOCALBASE)/lib -lxml2 -lrt +LIBS += -L$(LOCALBASE)/lib -lxml2 + +ifneq ($(OS),darwin) + LIBS += -lrt +endif SERLIBPATH=../../lib SER_LIBS+=$(SERLIBPATH)/kcore/kcore

1 0

Contact parameter parsing broken?
by Peter Dunkley 02 Jan '13

02 Jan '13

Hi, When working on the outbound support I noticed that this Contact header (as per RFC 5626): Contact: <sip:foo@192.168.1.18:5061>;reg-id=1;+sip.instance="<urn:uuid:00000000-0000-1000-8000-AABBCCDDEEFF>" does not get parsed properly (specifically, the param_hooks_t.hooks.reg_id is not filled in by parse_params()). However, some investigation showed that: Contact: <sip:foo@192.168.1.18:5061;reg-id=1;+sip.instance="<urn:uuid:00000000-0000-1000-8000-AABBCCDDEEFF>"> It looks like Kamailio is treating reg-id (and probably +sip.instance) parameters as Contact-URI parameters instead of Contact header parameters. Regards, Peter -- Peter Dunkley Technical Director Crocodile RCS Ltd

3 4

dns failover and send socket
by Jasmin Schnatterbeck 02 Jan '13

02 Jan '13

Hello, when using dns failover (SRV records) with kamailio 3.2 I noticed, that the failover sip message (to the failover host specified by DNS) is sent from a different socket. For example: incoming sip message is on socket ip1:5060 => send socket for relayed message (original) is ip1:5060 but send socket for relayed message (failover) is ip2:5060 reparse_on_dns_failover is left on its default value (1). In the the relayed message (failover): Reord-Route-Header is as in original message (ip1:5060) - correct Via-Header is ip2:5060 (matching the send socket) - incorrect This is exactly the opposite case regarding the README: "Disadvantage: only the via header is replaced in the message buffer, so the outgoing socket address is not corrected in any other part of the message. It is dangerous on multihomed hosts: when the new SIP request after the DNS failover is sent via different interface than the first request, the message can contain incorrect ip address in the Record-Route header for instance." I think the message was rebuilt, because only VIA header and send socket changed. Looking into forward.c: 598 #ifdef USE_DNS_FAILOVER 599 if (prev_send_sock!=send_info->send_sock){ 600 /* rebuild the message only if the send_sock changed */ But I did not change the send socket... any ideas why this condition is true? Cheers and a happy new 2013 Jasmin

1 0

Next major release proposed as v4.0.0
by Daniel-Constantin Mierla 02 Jan '13

02 Jan '13

Hello, using v4.0.0 for the next major release was proposed and debated at previous development IRC meeting. First was about the addition of a new transport layer (websockets) along with other important new features. During the debate was touched the completion of kamailio-ser integration (no more duplicated modules) as another milestone for increasing the first digit (which was finished just before the end of last year). Now we also have the IMS modules, pushed in the GIT master branch few days ago. It seems that there are lot of good reasons for numbering the next major version 4.0.0. Other comments, opinions? Have a great and happy year ahead, Daniel -- Daniel-Constantin Mierla - http://www.asipto.com http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda

3 3

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2013