sr-dev August 2014

sr-dev@lists.kamailio.org

25 participants
195 discussions

git:master: core: call onsend_route for replies

by Lucian Balaceanu

Module: sip-router Branch: master Commit: cd335addc40f358b6b80bfa5b9df8b468319838a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=cd335ad… Author: lucian balanceanu <lucian.balanceanu(a)1and1.ro> Committer: lucian balanceanu <lucian.balanceanu(a)1and1.ro> Date: Thu Aug 21 10:02:49 2014 +0300 core: call onsend_route for replies - this feature is aimed at tracing replies with sip_trace; e.g.: onsend_route { sip_trace("sip:10.84.1.49:5060"); } would now trace the requests passing Kamailio, as well as the replies; This patch has been proposed earlier: http://lists.sip-router.org/pipermail/sr-users/2012-May/073250.html --- forward.c | 11 +++++++++++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/forward.c b/forward.c index e4af439..849325f 100644 --- a/forward.c +++ b/forward.c @@ -856,6 +856,17 @@ static int do_forward_reply(struct sip_msg* msg, int mode) STATS_RPL_FWD_DROP(); goto error; } + /* call onsend_route */ + if(dst.send_sock == NULL) { + dst.send_sock=get_send_socket(msg, &dst.to, dst.proto); + if (dst.send_sock==0){ + LOG(L_ERR, "forward_reply: ERROR: cannot forward reply\n"); + goto done; + } + } + run_onsend(msg, &dst, new_buf, new_len); + + done: #ifdef STATS STATS_TX_RESPONSE( (msg->first_line.u.reply.statuscode/100) ); #endif

10 years, 3 months

sipcapture: Retry writing to homer database in case the first attempt fails

by Lucian Balaceanu

Hi, We would like to propose a patch for the sipcapture module where we retry a write to the homer database in case the first attempt fails. This proves useful under heavy loads. We add the insert_retries parameter to limit the maximum number of write retries and also the insert_retry_timeout as a time limit for the retry. Thank you, Lucian Balaceanu

10 years, 3 months

git:master: sipcapture: retry insert in DB on initial failure

by Lucian Balaceanu

Module: sip-router Branch: master Commit: e88b500f8fb6077ac17e5413ffe8770d72e5c6ac URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e88b500… Author: lucian balanceanu <lucian.balanceanu(a)1and1.ro> Committer: lucian balanceanu <lucian.balanceanu(a)1and1.ro> Date: Wed Aug 20 18:08:03 2014 +0300 sipcapture: retry insert in DB on initial failure - added insert_retries and insert_retry_timeout parameters, which control how many times and for how long Kamailio should try to rewrite to the Homer database in case the first attempt fails; trying to rewrite proves useful especially in heavy load scenarios. --- modules/sipcapture/README | 173 ++++++++++++++++++--------- modules/sipcapture/doc/sipcapture_admin.xml | 42 +++++++ modules/sipcapture/sipcapture.c | 64 +++++++--- 3 files changed, 204 insertions(+), 75 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=e88…

10 years, 3 months

next release?

by Juha Heinanen

is there any planned date for release or freeze of next version (4.2?) of kamailio? -- juha

10 years, 3 months

git:4.1: debugger: fix reset_msgid

by Victor Seva

Module: sip-router Branch: 4.1 Commit: 2fd085d21fb1a026d31d98541a9e33c568245771 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=2fd085d… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Tue Aug 19 17:21:57 2014 +0200 debugger: fix reset_msgid Don't change the msg->id when that message has been processed already (cherry picked from commit 6b2f7d0b9d405d8ffd2310f788254453d6748f0b) --- modules/debugger/debugger_api.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/modules/debugger/debugger_api.c b/modules/debugger/debugger_api.c index e6e5f54..39e599b 100644 --- a/modules/debugger/debugger_api.c +++ b/modules/debugger/debugger_api.c @@ -268,18 +268,18 @@ int dbg_msgid_filter(struct sip_msg *msg, unsigned int flags, void *bar) } msgid_base = _dbg_pid_list[indx].msgid_base; lock_release(_dbg_pid_list[indx].lock); - msgid_new = msg->id - msgid_base; - LM_DBG("msg->id:%d msgid_base:%d -> %d\n", msg->id, msgid_base, msgid_new); - if(msgid_new>0) + if(msg->id > msgid_base) { + msgid_new = msg->id - msgid_base; + LM_DBG("msg->id:%d msgid_base:%d -> %d\n", + msg->id, msgid_base, msgid_new); msg->id = msgid_new; - return 1; } else { - LM_WARN("msgid_new<=0??\n"); - return -1; + LM_DBG("msg->id:%d already processed\n", msg->id); } + return 1; } /**

10 years, 3 months

git:master: debugger: fix reset_msgid

by Victor Seva

Module: sip-router Branch: master Commit: 6b2f7d0b9d405d8ffd2310f788254453d6748f0b URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6b2f7d0… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Tue Aug 19 17:21:57 2014 +0200 debugger: fix reset_msgid Don't change the msg->id when that message has been processed already --- modules/debugger/debugger_api.c | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/modules/debugger/debugger_api.c b/modules/debugger/debugger_api.c index 6f4cbb8..05f904e 100644 --- a/modules/debugger/debugger_api.c +++ b/modules/debugger/debugger_api.c @@ -273,18 +273,18 @@ int dbg_msgid_filter(struct sip_msg *msg, unsigned int flags, void *bar) } msgid_base = _dbg_pid_list[indx].msgid_base; lock_release(_dbg_pid_list[indx].lock); - msgid_new = msg->id - msgid_base; - LM_DBG("msg->id:%d msgid_base:%d -> %d\n", msg->id, msgid_base, msgid_new); - if(msgid_new>0) + if(msg->id > msgid_base) { + msgid_new = msg->id - msgid_base; + LM_DBG("msg->id:%d msgid_base:%d -> %d\n", + msg->id, msgid_base, msgid_new); msg->id = msgid_new; - return 1; } else { - LM_WARN("msgid_new<=0??\n"); - return -1; + LM_DBG("msg->id:%d already processed\n", msg->id); } + return 1; } char* get_current_route_type_name()

10 years, 3 months

git:master: tmx: be sure From header is parsed before accessing the tag

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 92ff234d8b39c422aca683d4fbb0cfacc24a77b4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=92ff234… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Aug 19 11:10:14 2014 +0200 tmx: be sure From header is parsed before accessing the tag - reported by Juha Heinanen --- modules/tmx/tmx_pretran.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/modules/tmx/tmx_pretran.c b/modules/tmx/tmx_pretran.c index 3483c32..653f64c 100644 --- a/modules/tmx/tmx_pretran.c +++ b/modules/tmx/tmx_pretran.c @@ -213,6 +213,10 @@ int tmx_check_pretran(sip_msg_t *msg) LM_ERR("failed to get Via header\n"); return -1; } + if (parse_from_header(msg)<0 || get_from(msg)->tag_value.len==0) { + LM_ERR("failed to get From header\n"); + return -1; + } vbr = msg->via1->branch;

10 years, 3 months

git:master: tmx: mark pretran struct as linked when adding to a non-empty slot

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: cb04eea7b4e2a382a0315529d38d2f4ec5a60d93 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=cb04eea… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Aug 19 18:19:46 2014 +0200 tmx: mark pretran struct as linked when adding to a non-empty slot - otherwise is not going to be unlinked properly, with high potential of ending in an infinite loop - reported by Juha Heinanen --- modules/tmx/tmx_pretran.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/modules/tmx/tmx_pretran.c b/modules/tmx/tmx_pretran.c index 653f64c..a9c6f2a 100644 --- a/modules/tmx/tmx_pretran.c +++ b/modules/tmx/tmx_pretran.c @@ -121,6 +121,7 @@ void tmx_pretran_link_safe(int slotid) _tmx_proc_ptran->next = _tmx_ptran_table[slotid].plist; _tmx_ptran_table[slotid].plist->prev = _tmx_proc_ptran; _tmx_ptran_table[slotid].plist = _tmx_proc_ptran; + _tmx_proc_ptran->linked = 1; return; } @@ -250,9 +251,7 @@ int tmx_check_pretran(sip_msg_t *msg) } if(dsize<256) dsize = 256; - lock_get(&_tmx_ptran_table[slotid].lock); - tmx_pretran_unlink_safe(slotid); - lock_release(&_tmx_ptran_table[slotid].lock); + tmx_pretran_unlink(); if(dsize > _tmx_proc_ptran->dbuf.len) { if(_tmx_proc_ptran->dbuf.s) shm_free(_tmx_proc_ptran->dbuf.s);

10 years, 3 months

[tracker] Task opened: textopsx : hf_value and hf_value2 can not parse authorize header

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Kamrul Khan (dodullive.com) Attached to Project - sip-router Summary - textopsx : hf_value and hf_value2 can not parse authorize header Task Type - Bug Report Category - Module Status - Unconfirmed Assigned To - Operating System - Linux Severity - Low Priority - Normal Reported Version - 4.1 Due in Version - Undecided Due Date - Undecided Details - I tried using hf_value and hf_value2 for getting username, uri and response from my authorize header. But, each time it keeps returning Digest as result. For @hf_value2.Authorization.username returns Digest. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=461 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.

10 years, 3 months

git:master: tmx: documentation for t_precheck_trans()

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 392becb3e592f06f4ff843b303f1ddc765fa1a9f URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=392becb… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Mon Aug 18 22:05:30 2014 +0200 tmx: documentation for t_precheck_trans() --- modules/tmx/README | 28 +++++++++++++++++++++++++++- modules/tmx/doc/tmx_admin.xml | 35 +++++++++++++++++++++++++++++++++-- 2 files changed, 60 insertions(+), 3 deletions(-) diff --git a/modules/tmx/README b/modules/tmx/README index 73eba24..80e6a45 100644 --- a/modules/tmx/README +++ b/modules/tmx/README @@ -47,6 +47,7 @@ Juha Heinanen 3.8. t_suspend() 3.9. t_continue(tindex, tlabel, rtname) 3.10. t_reuse_branch() + 3.11. t_precheck_trans() 4. Exported pseudo-variables 5. MI Commands @@ -83,6 +84,7 @@ Juha Heinanen 1.8. t_suspend usage 1.9. t_continue usage 1.10. t_reuse_branch usage + 1.11. t_precheck_trans usage Chapter 1. Admin Guide @@ -106,6 +108,7 @@ Chapter 1. Admin Guide 3.8. t_suspend() 3.9. t_continue(tindex, tlabel, rtname) 3.10. t_reuse_branch() + 3.11. t_precheck_trans() 4. Exported pseudo-variables 5. MI Commands @@ -135,7 +138,7 @@ Chapter 1. Admin Guide This module collects extensions from Kamailio TM module. Kamailio TM (Transaction Management) module documentation is available - at: http://www.kamailio.org/docs/modules/1.5.x/tm.html + at: http://www.kamailio.org/docs/modules/stable/tm.html 2. Dependencies @@ -165,6 +168,7 @@ Chapter 1. Admin Guide 3.8. t_suspend() 3.9. t_continue(tindex, tlabel, rtname) 3.10. t_reuse_branch() + 3.11. t_precheck_trans() 3.1. t_cancel_branches(which) @@ -350,6 +354,28 @@ event_route [tm:branch-failure:contact] { } ... +3.11. t_precheck_trans() + + Check if current processed message is handled by other process. This + function is useful to catch retransmissions before transaction is + created. The function returns true if the request is handled by another + process. + + Note that ACK and CANCEL requests are not tracked by this function, the + return code being false for them. + + This function can be used from REQUEST_ROUTE . + + Example 1.11. t_precheck_trans usage +... + # handle retransmissions + if(t_precheck_trans()) { + t_check_trans(); + exit; + } + t_check_trans(); +... + 4. Exported pseudo-variables * $T_branch_idx diff --git a/modules/tmx/doc/tmx_admin.xml b/modules/tmx/doc/tmx_admin.xml index 643281b..029c943 100644 --- a/modules/tmx/doc/tmx_admin.xml +++ b/modules/tmx/doc/tmx_admin.xml @@ -20,8 +20,8 @@ </para> <para> &kamailio; TM (Transaction Management) module documentation is available at: - <ulink url="http://www.kamailio.org/docs/modules/1.5.x/tm.html"> - http://www.kamailio.org/docs/modules/1.5.x/tm.html</ulink> + <ulink url="http://www.kamailio.org/docs/modules/stable/tm.html"> + http://www.kamailio.org/docs/modules/stable/tm.html</ulink> </para> </section> @@ -370,6 +370,37 @@ event_route [tm:branch-failure:contact] { </programlisting> </example> </section> + <section> + <title> + <function moreinfo="none">t_precheck_trans()</function> + </title> + <para> + Check if current processed message is handled by other process. + This function is useful to catch retransmissions before transaction + is created. The function returns true if the request is handled by + another process. + </para> + <para> + Note that ACK and CANCEL requests are not tracked by this function, + the return code being false for them. + </para> + <para> + This function can be used from REQUEST_ROUTE . + </para> + <example> + <title><function>t_precheck_trans</function> usage</title> + <programlisting format="linespecific"> +... + # handle retransmissions + if(t_precheck_trans()) { + t_check_trans(); + exit; + } + t_check_trans(); +... +</programlisting> + </example> + </section> </section>

10 years, 3 months

← Newer
1
...
4
5
6
7
8
9
10
...
20
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2014