sr-dev October 2011

sr-dev@lists.kamailio.org

37 participants
284 discussions

git:master: dialog(k): Note dependency between dlg_manage() and transactions
by Timo Reimann 17 Oct '11

17 Oct '11

Module: sip-router Branch: master Commit: 807f07499b4875cf77463c3bdb6a47e099f18f58 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=807f074… Author: Timo Reimann <timo.reimann(a)1und1.de> Committer: Timo Reimann <timo.reimann(a)1und1.de> Date: Mon Oct 17 11:46:45 2011 +0200 dialog(k): Note dependency between dlg_manage() and transactions in docs. --- modules_k/dialog/doc/dialog_admin.xml | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/modules_k/dialog/doc/dialog_admin.xml b/modules_k/dialog/doc/dialog_admin.xml index 2fccc78..9bcbea1 100644 --- a/modules_k/dialog/doc/dialog_admin.xml +++ b/modules_k/dialog/doc/dialog_admin.xml @@ -1434,6 +1434,15 @@ dlg_refer("caller", "sip:annoucement@kamailio.org"); <para> This function can be used from REQUEST_ROUTE. </para> + <para> + <b>IMPORTANT</b>: Users of this function should make sure that the + dialog created is further processed statefully. Specifically, if a + stateless response is sent out after dlg_manage() is called, the + dialog cannot be handled properly. So make sure that a transaction + exists or create it explicitly using the tm module.<br>This is a + shortcoming of the current implementation that may be resolved in a + future version hopefully. + </para> <example> <title><function>dlg_manage</function> usage</title> <programlisting format="linespecific">

1 0

[tracker] Comment added: app_python doesn't work (Attachment added)
by sip-router 17 Oct '11

17 Oct '11

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#137 - app_python doesn't work User who did this - Michal Karas (Largon) ---------- Hello, we had problem with loading python script too. After debugging we found out problem was in dname variable which changes it's value after calling memcpy on tname. I attach patch we've used to solve the issue for us. ---------- One or more files have been attached. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=137#comment305 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

git:master: modules_k/(auth|misc)_radius: added cast
by Juha Heinanen 15 Oct '11

15 Oct '11

Module: sip-router Branch: master Commit: 242e0ada7df9d105f0045d68cb7481f33e2f04f3 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=242e0ad… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sat Oct 15 17:07:12 2011 +0300 modules_k/(auth|misc)_radius: added cast - Added cast of int to long in order to avoid compiler warning on 64 bit operating system. --- modules_k/auth_radius/extra.c | 2 +- modules_k/misc_radius/extra.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules_k/auth_radius/extra.c b/modules_k/auth_radius/extra.c index a31b8bd..46a812c 100644 --- a/modules_k/auth_radius/extra.c +++ b/modules_k/auth_radius/extra.c @@ -190,7 +190,7 @@ int extra2strar( struct extra_attr *extra, struct sip_msg *rq, str *val_arr) val_arr[n].len = 0; } else if (value.flags&PV_VAL_INT) { /* len = -1 denotes int type */ - val_arr[n].s = (char *)value.ri; + val_arr[n].s = (char *)(long)value.ri; val_arr[n].len = -1; } else { /* set the value into the acc buffer */ diff --git a/modules_k/misc_radius/extra.c b/modules_k/misc_radius/extra.c index a31b8bd..46a812c 100644 --- a/modules_k/misc_radius/extra.c +++ b/modules_k/misc_radius/extra.c @@ -190,7 +190,7 @@ int extra2strar( struct extra_attr *extra, struct sip_msg *rq, str *val_arr) val_arr[n].len = 0; } else if (value.flags&PV_VAL_INT) { /* len = -1 denotes int type */ - val_arr[n].s = (char *)value.ri; + val_arr[n].s = (char *)(long)value.ri; val_arr[n].len = -1; } else { /* set the value into the acc buffer */

1 0

git:master: kamailio.cfg: added storage of src ip to extra acc example
by Daniel-Constantin Mierla 14 Oct '11

14 Oct '11

Module: sip-router Branch: master Commit: 5b208d94b6c3d2eee09849af31f589efb9a36c04 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5b208d9… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Sat Oct 15 00:07:11 2011 +0200 kamailio.cfg: added storage of src ip to extra acc example = renamed define id LOCAL_TEST_RUN to WITH_SRCPATH to match the pattern used for default config define switches and to suggest better the purpose of setting modules location to sources path --- etc/kamailio.cfg | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/etc/kamailio.cfg b/etc/kamailio.cfg index f76b9e1..b7e469d 100644 --- a/etc/kamailio.cfg +++ b/etc/kamailio.cfg @@ -87,11 +87,13 @@ #!ifdef ACCDB_COMMENT ALTER TABLE acc ADD COLUMN src_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN src_domain VARCHAR(128) NOT NULL DEFAULT ''; + ALTER TABLE acc ADD COLUMN src_ip varchar(64) NOT NULL default ''; ALTER TABLE acc ADD COLUMN dst_ouser VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN dst_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN dst_domain VARCHAR(128) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_domain VARCHAR(128) NOT NULL DEFAULT ''; + ALTER TABLE missed_calls ADD COLUMN src_ip varchar(64) NOT NULL default ''; ALTER TABLE missed_calls ADD COLUMN dst_ouser VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN dst_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN dst_domain VARCHAR(128) NOT NULL DEFAULT ''; @@ -196,8 +198,8 @@ voicemail.srv_port = "5060" desc "VoiceMail Port" ####### Modules Section ######## -# set paths to location of modules -#!ifdef LOCAL_TEST_RUN +# set paths to location of modules (to sources or installation folders) +#!ifdef WITH_SRCPATH mpath="modules_k:modules" #!else mpath="/usr/local/lib/kamailio/modules_k/:/usr/local/lib/kamailio/modules/" @@ -319,7 +321,8 @@ modparam("acc", "detect_direction", 0) modparam("acc", "log_flag", FLT_ACC) modparam("acc", "log_missed_flag", FLT_ACCMISSED) modparam("acc", "log_extra", - "src_user=$fU;src_domain=$fd;dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") + "src_user=$fU;src_domain=$fd;src_ip=$si;" + "dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") modparam("acc", "failed_transaction_flag", FLT_ACCFAILED) /* enhanced DB accounting */ #!ifdef WITH_ACCDB @@ -327,7 +330,8 @@ modparam("acc", "db_flag", FLT_ACC) modparam("acc", "db_missed_flag", FLT_ACCMISSED) modparam("acc", "db_url", DBURL) modparam("acc", "db_extra", - "src_user=$fU;src_domain=$fd;dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") + "src_user=$fU;src_domain=$fd;src_ip=$si;" + "dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") #!endif

1 0

git:master: kamailio.cfg: load cfg_rpc module to allow runtime updates
by Daniel-Constantin Mierla 14 Oct '11

14 Oct '11

Module: sip-router Branch: master Commit: 751114fd6ee5eb54b73b9214941ee689b9d70450 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=751114f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Oct 14 21:34:08 2011 +0200 kamailio.cfg: load cfg_rpc module to allow runtime updates - use request_route { ... } instead of route { ... } for easier reference in docs and discussions --- etc/kamailio.cfg | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/etc/kamailio.cfg b/etc/kamailio.cfg index 7c6f64c..f76b9e1 100644 --- a/etc/kamailio.cfg +++ b/etc/kamailio.cfg @@ -222,6 +222,7 @@ loadmodule "siputils.so" loadmodule "xlog.so" loadmodule "sanity.so" loadmodule "ctl.so" +loadmodule "cfg_rpc.so" loadmodule "mi_rpc.so" loadmodule "acc.so" @@ -438,7 +439,8 @@ modparam("debugger", "cfgtrace", 1) # Main SIP request routing logic # - processing of any incoming SIP request starts with this route -route { +# - note: this is the same as route { ... } +request_route { # per request initial checks route(REQINIT);

1 0

git:master: cfg_rpc: improved documentation
by Daniel-Constantin Mierla 14 Oct '11

14 Oct '11

Module: sip-router Branch: master Commit: 9fc41bd1a157d7cfe20c389ed3220e559f0b2712 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9fc41bd… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Oct 14 21:33:08 2011 +0200 cfg_rpc: improved documentation - aliased set_now_int to seti and set_now_string to sets --- modules/cfg_rpc/README | 252 ++++++++++++++++++++++++++++----------- modules/cfg_rpc/cfg_rpc.c | 2 + modules/cfg_rpc/doc/cfg_rpc.xml | 48 +++++++- modules/cfg_rpc/doc/rpc.xml | 152 ++++++++++++++++-------- 4 files changed, 331 insertions(+), 123 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=9fc…

1 0

Force socket on reply
by Jijo 14 Oct '11

14 Oct '11

Hi All, Our system has two IP address, one is used for kamailio and 2nd one is used for data. The problem is suppose a request came in to kamailio on a TCP connection with first IP address and connection is torned down before sending the response. Later when the response is send out kamialio is using the 2nd IP to create the connection. The reason is kamailio doesn't have force socket in in reply, so it uses INADDR_ANY for soruce addres .so kernel uses source address based on the detaintion IP. I tired to force the socket using pseudo variable $fs or force_send_socket(), but neither of them worked from reply_route. So i modfied the code relay_replay to set the SND_F_FORCE_SOCKET which will use the address where request is recieved as source address to make the TCP connection The code change is shown below. Please let me know is there anyother way we could acheive it? }else{ buf = build_res_buf_from_sip_res( relayed_msg, &res_len ); /* if we build a message from shmem, we need to remove via delete lumps which are now stirred in the shmem-ed structure */ if (branch!=relay) { free_via_clen_lump(&relayed_msg->add_rm); } /* update send_flags with possible additions from the reply route */ SND_FLAGS_OR(&uas_rb->dst.send_flags, &uas_rb->dst.send_flags, &relayed_msg->rpl_send_flags); /* Make the response to use the same IP where it receives the message */ * * * uas_rb->dst.send_flags.f |= SND_F_FORCE_SOCKET; * } Thanks JIjo

2 4

new wiki system
by Daniel-Constantin Mierla 14 Oct '11

14 Oct '11

Hello, I installed a new wiki site, targeting to collect documentation for upcoming v3.2.0 and further releases. You can find it at: * http://www.kamailio.org/wiki/ Hopefully, it will make the process of updating/migrating the relevant documentation from old wiki (http://www.kamailio.org/dokuwiki/) straightforward and offer a clean reference for upcoming stable version. Old wiki is in place since 2005, collected a lot of pages, but many are for very old versions of kamailio, making difficult, at least for new comers, to sort out documentation for latest stable. All of you are invited to help with this process of adding documentation for 3.2.0 on the new wiki. The old wiki will stay in place, it will not be removed. Both are based on Dokuwiki, using different themes. The new one is at the beginning, so no much content in there yet, but by the time of releasing 3.2.0, should contain sufficient documentation for get started with that version. One very demanded thing was the alphabetic index for parameters, functions and mi commands from modules. This page contains new generated indexes for version 3.2.0. More details in a separate email. If you have questions about the new wiki, don't know where to place some tutorial, feel free to ask on mailing lists. Ultimately, pages can be relocated, but try to use namespaces for related topics. Cheers, Daniel -- Daniel-Constantin Mierla -- http://www.asipto.com Kamailio Advanced Training, Dec 5-8, Berlin: http://asipto.com/u/kat http://linkedin.com/in/miconda -- http://twitter.com/miconda

4 6

[tracker] Comment added: $hdr(SomeHeader)[$var(i)] fails when $var(i) is 0
by sip-router 14 Oct '11

14 Oct '11

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#167 - $hdr(SomeHeader)[$var(i)] fails when $var(i) is 0 User who did this - Walter Doekes (wdoekes) ---------- Ah. This was indeed a non-issue in kamailio. I'm sorry for wasting your time. Regards, Walter ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=167#comment304 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

git:master: modules_k/osp, modules_k/siputils: The bind_siputils function takes 1 parameter, but the bind command in here (and as used in osp) was specified as taking 0 parameters.
by Peter Dunkley 14 Oct '11

14 Oct '11

Module: sip-router Branch: master Commit: 0c36970e4f72f775a29607919a3c9a83df6e0940 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0c36970… Author: pd <peter.dunkley(a)crocodile-rcs.com> Committer: pd <peter.dunkley(a)crocodile-rcs.com> Date: Fri Oct 14 14:06:56 2011 +0100 modules_k/osp, modules_k/siputils: The bind_siputils function takes 1 parameter, but the bind command in here (and as used in osp) was specified as taking 0 parameters. --- modules_k/osp/osp_mod.c | 2 +- modules_k/siputils/siputils.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules_k/osp/osp_mod.c b/modules_k/osp/osp_mod.c index 8dde3bd..5eb8502 100644 --- a/modules_k/osp/osp_mod.c +++ b/modules_k/osp/osp_mod.c @@ -182,7 +182,7 @@ static int ospInitMod(void) } /* Load the AUTH API */ - bind_su = (bind_siputils_t)find_export("bind_siputils", 0, 0); + bind_su = (bind_siputils_t)find_export("bind_siputils", 1, 0); if ((bind_su == NULL) || (bind_su(&osp_siputils) != 0)) { LM_WARN("failed to load the SIPUTILS API. Check if you load the auth module.\n"); LM_WARN("rpid_avp & rpid_avp_type is required for calling number translation\n"); diff --git a/modules_k/siputils/siputils.c b/modules_k/siputils/siputils.c index e50fa3d..9970f45 100644 --- a/modules_k/siputils/siputils.c +++ b/modules_k/siputils/siputils.c @@ -154,7 +154,7 @@ static cmd_export_t cmds[]={ fixup_free_set_uri, ANY_ROUTE}, {"set_uri_host", (cmd_function)set_uri_host, 2, fixup_set_uri, fixup_free_set_uri, ANY_ROUTE}, - {"bind_siputils", (cmd_function)bind_siputils, 0, 0, + {"bind_siputils", (cmd_function)bind_siputils, 1, 0, 0, 0}, {"is_request", (cmd_function)w_is_request, 0, 0, 0, ANY_ROUTE},

1 0

← Newer
1
...
13
14
15
16
17
18
19
...
29
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev October 2011