Module: sip-router
Branch: master
Commit: 355ab783a99e413eb96a385ab067336316aaafda
URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=355ab78…
Author: Richard Good <richard.good(a)smilecoms.com>
Committer: Richard Good <richard.good(a)smilecoms.com>
Date: Tue Oct 29 12:40:42 2013 +0200
modules/cdp/acctstatemachine.c: fixed incorrect reservation expiry warning
First check if acc state is OPEN before sending reservation expiry warning
---
modules/cdp/session.c | 17 ++++++++++-------
1 files changed, 10 insertions(+), 7 deletions(-)
diff --git a/modules/cdp/session.c b/modules/cdp/session.c
index dc41e1f..77a1f45 100644
--- a/modules/cdp/session.c
+++ b/modules/cdp/session.c
@@ -448,13 +448,16 @@ int cdp_sessions_timer(time_t now, void* ptr)
int res_valid_for = x->u.cc_acc.reserved_units_validity_time;
int last_reservation = x->u.cc_acc.reserved_units;
int buffer_time = 15; //15 seconds - TODO: add as config parameter
- if (last_res_timestamp) {
- //we have obv already started reservations
- if ((last_res_timestamp + res_valid_for) < (time(0) + last_reservation + buffer_time)) {
- LM_DBG("reservation about to expire, sending callback\n");
- cc_acc_client_stateful_sm_process(x, ACC_CC_EV_RSVN_WARNING, 0);
- }
-
+ //we should check for reservation expiries if the state is open
+ if(x->u.cc_acc.state==ACC_CC_ST_OPEN){
+ if (last_res_timestamp) {
+ //we have obv already started reservations
+ if ((last_res_timestamp + res_valid_for) < (time(0) + last_reservation + buffer_time)) {
+ LM_DBG("reservation about to expire, sending callback\n");
+ cc_acc_client_stateful_sm_process(x, ACC_CC_EV_RSVN_WARNING, 0);
+ }
+
+ }
}
/* TODO: if reservation has expired we need to tear down the session. Ideally
* the client application (module) should do this but for completeness we should
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
A new Flyspray task has been opened. Details are below.
User who did this - Víctor Seva (linuxmaniac)
Attached to Project - sip-router
Summary - avpops: avp_copy add xavp
Task Type - Feature Request
Category - Module
Status - New
Assigned To - Víctor Seva
Operating System - All
Severity - Low
Priority - Normal
Reported Version - Development
Due in Version - Undecided
Due Date - Undecided
Details - We have avp_copy on avpops module but we don't have anything to copy xavps.
$xavp(a[0]=>b) = avp(x);
$avp(x) = $xavp(a[0]=>b);
$xavp(a[0]) = $xavp(b[1]);
$xavp(a) = $xavp(b);
More information can be found at the following URL:
https://sip-router.org/tracker/index.php?do=details&task_id=359
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.
Module: sip-router
Branch: master
Commit: ecc6e35b31d59e6e9995a03c6ab00f21a23b1be9
URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ecc6e35…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: Tue Oct 29 11:07:03 2013 +0100
srdb1: drop unique constrant on alias user+domain in dbaliases
- it doesn't work for multiple branches when use_domain is set, reported
by Vassilis Radis
---
lib/srdb1/schema/dbaliases.xml | 11 +++++++++--
1 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/lib/srdb1/schema/dbaliases.xml b/lib/srdb1/schema/dbaliases.xml
index 04c2c03..12078e3 100644
--- a/lib/srdb1/schema/dbaliases.xml
+++ b/lib/srdb1/schema/dbaliases.xml
@@ -12,7 +12,10 @@
<version>1</version>
<type db="mysql">&MYSQL_TABLE_TYPE;</type>
<description>
- <db:para>This table us used by the alias_db module as an alternative for user aliases via userloc. More information about the alias_db module can be found at: &KAMAILIO_MOD_DOC;alias_db.html
+ <db:para>This table us used by the alias_db module as an
+ alternative for user aliases via userloc. More information
+ about the alias_db module can be found at:
+ &KAMAILIO_MOD_DOC;alias_db.html
</db:para>
</description>
@@ -61,10 +64,14 @@
</column>
<index>
+ <name>alias_user_idx</name>
+ <colref linkend="alias_username"/>
+ </index>
+
+ <index>
<name>alias_idx</name>
<colref linkend="alias_username"/>
<colref linkend="alias_domain"/>
- <unique/>
</index>
<index>
Module: sip-router
Branch: master
Commit: da3d48de7d52df83fcb568530441e7ac6475baae
URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=da3d48d…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: Tue Oct 29 10:08:04 2013 +0100
pua_publish: more verbose output for error case when sending publish
---
modules/pua_usrloc/ul_publish.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/modules/pua_usrloc/ul_publish.c b/modules/pua_usrloc/ul_publish.c
index 4eaf7a9..24dd3c9 100644
--- a/modules/pua_usrloc/ul_publish.c
+++ b/modules/pua_usrloc/ul_publish.c
@@ -323,8 +323,8 @@ void ul_publish(ucontact_t* c, int type, void* param)
print_publ(publ);
if((error=pua_send_publish(publ))< 0)
{
- LM_ERR("while sending publish\n");
- if(type & UL_CONTACT_UPDATE && error == ERR_PUBLISH_NO_BODY) {
+ LM_ERR("while sending publish for ul event %d\n", type);
+ if((type & UL_CONTACT_UPDATE) && error == ERR_PUBLISH_NO_BODY) {
/* This error can occur if Kamailio was restarted/stopped and for any reason couldn't store a pua
* entry in 'pua' DB table. It can also occur if 'pua' table is cleaned externally while Kamailio
* is stopped so cannot retrieve these entries from DB when restarting.
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task is now closed:
FS#332 - add arbitrary params in Path header
User who did this - Daniel-Constantin Mierla (miconda)
Reason for closing: Implemented
More information can be found at the following URL:
https://sip-router.org/tracker/index.php?do=details&task_id=332
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.