sr-dev December 2011

sr-dev@lists.kamailio.org

32 participants
230 discussions

[tracker] Task opened: Error in generating Publush with state confirmed at recived 200OK package.
by sip-router 07 Dec '11

07 Dec '11

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Tomek Kotecki (koteckit) Attached to Project - sip-router Summary - Error in generating Publush with state confirmed at recived 200OK package. Task Type - Bug Report Category - Module Status - Unconfirmed Assigned To - Operating System - Linux Severity - Low Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - Hi, We have found a bug in version 3.2.0 kamailio (also occurs in 3.2.1). Pua_dialoginfo module generates an incorrect status for the package xml 200OK (method INVITE). In version 3.1.5 for package 200OK status in the body XML was correct and was: <state>confirmed </ state>. In version 3.2.x for package 200OK status in the body XML is: <state>terminated</ state>. In log from kamailio 3.2.0 we found ERROR: /usr/sbin/kamailio[1997]: ERROR: pua_dialoginfo [pua_dialoginfo.c:284]: unhandled dialog callback type 8 received, from=sip:username12@sip.mydomain.com /usr/sbin/kamailio[1997]: DEBUG: pua_dialoginfo [dialog_publish.c:242]: new_body: <?xml version="1.0"?> <dialog-info xmlns="urn:ietf:params:xml:ns:dialog-info" version="0" state="full" entity="sip:username12@sip.mydomain.com"> <dialog id="baeb1755-42d613e2(a)192.168.2.59" call-id="baeb1755-42d613e2(a)192.168.2.59" direction="initiator"> <state>terminated</state> <remote> <identity>sip:11@sip.mydomain.com</identity> <target uri="sip:11@sip.mydomain.com"/> </remote> <local> <identity>sip:username12@sip.mydomain.com</identity> <target uri="sip:username12@XX.XX.6.1:29727"/> </local> </dialog> </dialog-info> Correct in kamailio 3.1.5, without ERROR: /usr/sbin/kamailio[29551]: DEBUG: pua_dialoginfo [pua_dialoginfo.c:223]: dialog confirmed, from=sip:username12@sip.mydomain.com /usr/sbin/kamailio[29551]: DEBUG: pua_dialoginfo [dialog_publish.c:242]: new_body: <?xml version="1.0"?> <dialog-info xmlns="urn:ietf:params:xml:ns:dialog-info" version="0" state="full" entity="sip:username12@sip.mydomain.com"> <dialog id="16ee2c89-2e618509(a)192.168.2.59" call-id="16ee2c89-2e618509(a)192.168.2.59" direction="initiator"> <state>confirmed</state> <remote> <identity>sip:11@sip.mydomain.com</identity> <target uri="sip:11@sip.mydomain.com"/> </remote> <local> <identity>sip:username12@sip.mydomain.com</identity> <target uri="sip:username12@XX.XX.6.1:58340"/> </local> </dialog> </dialog-info> Best Regards, TK More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=187 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/msilo: removed debug statement
by Juha Heinanen 07 Dec '11

07 Dec '11

Module: sip-router Branch: master Commit: dfe367e8ddd51d28bb23265ad09169415ac2e145 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=dfe367e… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Wed Dec 7 09:16:20 2011 +0200 modules_k/msilo: removed debug statement --- modules_k/msilo/msilo.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/modules_k/msilo/msilo.c b/modules_k/msilo/msilo.c index 689b81a..e849fc7 100644 --- a/modules_k/msilo/msilo.c +++ b/modules_k/msilo/msilo.c @@ -539,8 +539,6 @@ static int get_non_mandatory_headers(struct sip_msg *msg, char *buf, int buf_len return -1; } memcpy(buf, avp_value.s.s, avp_value.s.len); - LM_INFO("copied '%.*s' to buf\n", avp_value.s.len, - avp_value.s.s); return avp_value.s.len; } }

1 0

git:master: modules_k/msilo: added storage of extra SIP headers from AVP
by Juha Heinanen 07 Dec '11

07 Dec '11

Module: sip-router Branch: master Commit: dc9f7a7fc8f5f5a7570f9e24c2f59ec77a7d6668 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=dc9f7a7… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Wed Dec 7 07:53:28 2011 +0200 modules_k/msilo: added storage of extra SIP headers from AVP --- modules_k/msilo/README | 32 +++++++++++++++++++++----- modules_k/msilo/doc/msilo_admin.xml | 29 ++++++++++++++++++++++++ modules_k/msilo/msilo.c | 42 +++++++++++++++++++++++++++++++++++ 3 files changed, 97 insertions(+), 6 deletions(-) diff --git a/modules_k/msilo/README b/modules_k/msilo/README index f9c44b0..e1fe9a3 100644 --- a/modules_k/msilo/README +++ b/modules_k/msilo/README @@ -65,6 +65,7 @@ Juha Heinanen 3.26. add_date (int) 3.27. max_messages (int) 3.28. add_contact (int) + 3.29. extra_hdrs_avp (str) 4. Functions @@ -113,9 +114,10 @@ Juha Heinanen 1.26. Set the “add_date” parameter 1.27. Set the “max_messages” parameter 1.28. Set the “add_contact” parameter - 1.29. m_store usage - 1.30. m_dump usage - 1.31. Kamailio config script - sample msilo usage + 1.29. Set the “extra_hdrs_avp” parameter + 1.30. m_store usage + 1.31. m_dump usage + 1.32. Kamailio config script - sample msilo usage Chapter 1. Admin Guide @@ -157,6 +159,7 @@ Chapter 1. Admin Guide 3.26. add_date (int) 3.27. max_messages (int) 3.28. add_contact (int) + 3.29. extra_hdrs_avp (str) 4. Functions @@ -254,6 +257,7 @@ Chapter 1. Admin Guide 3.26. add_date (int) 3.27. max_messages (int) 3.28. add_contact (int) + 3.29. extra_hdrs_avp (str) 3.1. db_url (string) @@ -596,6 +600,22 @@ modparam("msilo", "max_messages", 0) modparam("msilo", "add_contact", 1) ... +3.29. extra_hdrs_avp (str) + + Name of an AVP which may contain extra headers that are stored with the + message when m_store() is called. These extra headers (if any) are then + included in MESSAGE request generated by m_dump(). + + If the parameter is not set, m_store() does not look for extra headers + from any AVP. + + Default value is “null”. + + Example 1.29. Set the “extra_hdrs_avp” parameter +... +modparam("msilo", "extra_hdrs_avp", "$avp(msilo_extra_hdrs)") +... + 4. Functions 4.1. m_store([owner]) @@ -617,7 +637,7 @@ modparam("msilo", "add_contact", 1) This function can be used from REQUEST_ROUTE, FAILURE_ROUTE. - Example 1.29. m_store usage + Example 1.30. m_store usage ... m_store(); m_store("$tu"); @@ -637,7 +657,7 @@ m_store("$tu"); This function can be used from REQUEST_ROUTE. - Example 1.30. m_dump usage + Example 1.31. m_dump usage ... m_dump(); m_dump("$fu"); @@ -679,7 +699,7 @@ m_dump("$fu"); Next picture displays a sample usage of msilo. - Example 1.31. Kamailio config script - sample msilo usage + Example 1.32. Kamailio config script - sample msilo usage ... # $Id$ # diff --git a/modules_k/msilo/doc/msilo_admin.xml b/modules_k/msilo/doc/msilo_admin.xml index ee975f8..3d02670 100644 --- a/modules_k/msilo/doc/msilo_admin.xml +++ b/modules_k/msilo/doc/msilo_admin.xml @@ -566,6 +566,7 @@ modparam("msilo", "snd_time_avp", "$avp(i:123)") </programlisting> </example> </section> + <section> <title><varname>add_date</varname> (int)</title> <para> @@ -626,6 +627,34 @@ modparam("msilo", "add_contact", 1) </programlisting> </example> </section> + + <section> + <title><varname>extra_hdrs_avp</varname> (str)</title> + <para> + Name of an AVP which may contain extra headers that are + stored with the message when m_store() is called. These extra + headers (if any) are then included in MESSAGE request + generated by m_dump(). + </para> + <para> + If the parameter is not set, m_store() does not look + for extra headers from any AVP. + </para> + <para> + <emphasis> + Default value is <quote>null</quote>. + </emphasis> + </para> + <example> + <title>Set the <quote>extra_hdrs_avp</quote> parameter</title> +<programlisting format="linespecific"> +... +modparam("msilo", "extra_hdrs_avp", "$avp(msilo_extra_hdrs)") +... +</programlisting> + </example> + </section> + </section> <section> diff --git a/modules_k/msilo/msilo.c b/modules_k/msilo/msilo.c index cb23b2b..689b81a 100644 --- a/modules_k/msilo/msilo.c +++ b/modules_k/msilo/msilo.c @@ -39,6 +39,7 @@ * 2006-09-10 m_dump now checks if registering UA supports MESSAGE method (jh) * 2006-10-05 added max_messages module variable (jh) * 2011-10-19 added storage of extra SIP headers (hpw) + * 2011-12-07 added storage of extra SIP headers from AVP (jh) */ #include <stdio.h> @@ -154,6 +155,10 @@ static str ms_snd_time_avp_param = {NULL, 0}; int_str ms_snd_time_avp_name; unsigned short ms_snd_time_avp_type; +static str ms_extra_hdrs_avp_param = {NULL, 0}; +int_str ms_extra_hdrs_avp_name; +unsigned short ms_extra_hdrs_avp_type; + str msg_type = str_init("MESSAGE"); /** module functions */ @@ -220,6 +225,7 @@ static param_export_t params[]={ { "sc_snd_time", STR_PARAM, &sc_snd_time.s }, { "sc_stored_hdrs", STR_PARAM, &sc_stored_hdrs.s }, { "snd_time_avp", STR_PARAM, &ms_snd_time_avp_param.s }, + { "extra_hdrs_avp", STR_PARAM, &ms_extra_hdrs_avp_param.s }, { "add_date", INT_PARAM, &ms_add_date }, { "max_messages", INT_PARAM, &ms_max_messages }, { "add_contact", INT_PARAM, &ms_add_contact }, @@ -303,6 +309,8 @@ static int mod_init(void) sc_snd_time.len = strlen(sc_snd_time.s); if (ms_snd_time_avp_param.s) ms_snd_time_avp_param.len = strlen(ms_snd_time_avp_param.s); + if (ms_extra_hdrs_avp_param.s) + ms_extra_hdrs_avp_param.len = strlen(ms_extra_hdrs_avp_param.s); /* binding to mysql module */ if (db_bind_mod(&ms_db_url, &msilo_dbf)) @@ -334,6 +342,22 @@ static int mod_init(void) } } + if (ms_extra_hdrs_avp_param.s && ms_extra_hdrs_avp_param.len > 0) { + if (pv_parse_spec(&ms_extra_hdrs_avp_param, &avp_spec)==0 + || avp_spec.type!=PVT_AVP) { + LM_ERR("malformed or non AVP %.*s AVP definition\n", + ms_extra_hdrs_avp_param.len, ms_extra_hdrs_avp_param.s); + return -1; + } + + if (pv_get_avp_name(0, &(avp_spec.pvp), &ms_extra_hdrs_avp_name, + &ms_extra_hdrs_avp_type) != 0) { + LM_ERR("[%.*s]- invalid AVP definition\n", + ms_extra_hdrs_avp_param.len, ms_extra_hdrs_avp_param.s); + return -1; + } + } + db_con = msilo_dbf.init(&ms_db_url); if (!db_con) { @@ -502,6 +526,24 @@ static int get_non_mandatory_headers(struct sip_msg *msg, char *buf, int buf_len { struct hdr_field *hdrs; int len = 0; + int_str avp_value; + struct usr_avp *avp; + + if (ms_extra_hdrs_avp_name.n != 0) { + avp = NULL; + avp = search_first_avp(ms_extra_hdrs_avp_type, + ms_extra_hdrs_avp_name, &avp_value, 0); + if ((avp != NULL) && is_avp_str_val(avp)) { + if (buf_len <= avp_value.s.len) { + LM_ERR("insufficient space to store headers in silo\n"); + return -1; + } + memcpy(buf, avp_value.s.s, avp_value.s.len); + LM_INFO("copied '%.*s' to buf\n", avp_value.s.len, + avp_value.s.s); + return avp_value.s.len; + } + } for (hdrs = msg->headers; hdrs != NULL; hdrs = hdrs->next) {

1 0

Expiration in htable
by Alex Balashov 06 Dec '11

06 Dec '11

Question: how does the 'autoexpire' parameter to the 'htable' modparam interact with the 'expires' value now present in the database table definition for a DB-backed 'htable'? Which one has priority? -- Alex Balashov - Principal Evariste Systems LLC 260 Peachtree Street NW Suite 2200 Atlanta, GA 30303 Tel: +1-678-954-0670 Fax: +1-404-961-1892 Web: http://www.evaristesys.com/

2 2

debug make target
by Alex Balashov 06 Dec '11

06 Dec '11

Sorry for the stupid question, but is there a generic toplevel make target that will compile Kamailio/SR in ways maximally suitable for GDB analysis, e.g. - Without optimisations that strip out some symbols and values. - With -g, -ggdb, etc. Thanks! -- Alex Balashov - Principal Evariste Systems LLC 260 Peachtree Street NW Suite 2200 Atlanta, GA 30303 Tel: +1-678-954-0670 Fax: +1-404-961-1892 Web: http://www.evaristesys.com/

2 1

mi_xmlrpc warnings...
by Olle E. Johansson 06 Dec '11

06 Dec '11

Just a note for the archives or somebodys todo-list: When compiling mi_xmlrpc with the latest xmlrpc-c package in Centos 6 or FreeBSD 8 compilation fails. Seems like we're very depending on a particular library version. Which the build system warns me about... /O

2 2

kamailio fail to start when there are many records in DB in location or in pua table
by laura testi 06 Dec '11

06 Dec '11

Hi all, when there many records in DB tables like location(usrloc use write back mode), pua etc, kamailio fail to start by complaining "no pkg memory left" when try to load (restore) all records from DB to hashtable of location/pua. The share momory are allocated enough for the hashtable, but the default compilation of the PKG_MEMORY_SIZE is 4MB, which allow only restore several thousands records. I know I can increase the PKG_MEMORY_SIZE, but anyway there is a limitation which may fail to restore all records from DB during the startup of kamailio if it reach the limit of the number of records in DB. Is there anyway to reload the hashtable from table by doing query n rows a time and loop until non more result from DB? this will avoid the kamailio startup problem when there are many records in hashtable which are save also in DB. Thank you very much in advanced! Laura

2 4

Memcached module on FreeBSD
by Olle E. Johansson 05 Dec '11

05 Dec '11

Henning, Which memcached library did you create this for? I can't find the include file you're using in memcached from memcached.org Cheers, /O

3 3

TLS Variables in Makefile.def
by Olle E. Johansson 05 Dec '11

05 Dec '11

Are these variables valid? Are the explanations still up to date? Thanks, /O --------------- # -DUSE_TLS # compiles in tls support, requires -DUSE_TCP. Note: this is only # generic support (parsing a.s.o.), it does not include the actual # "tls engine". If you really want tls you need also either # -DCORE_TLS and a tls/ subdir with the tls code or -DTLS_HOOKS and # the tls module loaded. # -DCORE_TLS # compiles tls in-core support. Requires -DUSE_TLS, conflicts # -DTLS_HOOKS. Please use make CORE_TLS=1 instead (it will set all the # needed defines automatically and extra libraries needed for linking). # -DTLS_HOOKS # compile tls module support (support for having the "tls engine" in a # module). Requires -DUSE_TLS, conflicts -DCORE_TLS. # Please use make TLS_HOOKS=1 (or TLS_HOOKS=0 to for disabling) instead # of setting -DTLS_HOOKS (it will set all the needed defines # automatically)

2 1

FOSDEM 2012 Telephony/Communications Devroom Call for Presenters
by Bryan M. Johns 05 Dec '11

05 Dec '11

Greetings, This is a call for presenters (with presentations!) for the telephony and communications devroom at FOSDEM 2012 - http://www.fosdem.org/. We will be holding a day full of presentations on development topics in the area of open source telephony and communications on Sunday, February 5th. The schedule allows for presentations from 9:00 to 17:00. The room we will be using will have a projector, wifi, and 100 seats. Please submit all proposals no later than 2011-12-09. Notification of accepted speakers will be provided by 2011-12-16. We will then work to have a schedule finalized by 2012-01-06. Talks should be submitted by subscribing to and then posting to the telephony-devroom mailing list hosted on http://lists.fosdem.org/. If you would like to contact the devroom organizer directly, please contact Kevin P. Fleming <kpfleming at digium.com>. Feel free to forward this along to any people or mailing lists that you think would be interested in this event. Thank you! Bryan M. Johns Digium, Inc. | Community Director Direct: 256.428.6007 http://twitter.com/bryanmjohns Check us out at : www.asterisk.org or www.digium.com

1 0

← Newer
1
...
15
16
17
18
19
20
21
22
23
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev December 2011