Project

General

Profile

Bug #1087

vouchers need to save to CF periodically

Added by Chris Buechler over 8 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Captive Portal
Target version:
Start date:
12/09/2010
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.0
Affected Architecture:

Description

Vouchers need to save to CF periodically the same as we have options for RRD and DHCP leases to avoid losing usage info in an unclean shut down.

Associated revisions

Revision 0d89a2fc (diff)
Added by Jim Pingle over 8 years ago

Add voucher backup, configurable from Diagnostics > NanoBSD. Fixes #1087

Revision d12003c9 (diff)
Added by Jim Pingle over 8 years ago

Revert "Add voucher backup, configurable from Diagnostics > NanoBSD. Fixes #1087" - voucher db backup already existed under a different name. Ticket is still fixed, just by different code that was already there.

This reverts commit 0d89a2fcac3deea06bdc4a481bbdfae4f18b1ff8.

Revision c1f9a4df (diff)
Added by Ermal Luçi over 8 years ago

Set default interval of syncing voucher db to config to 5 minutes instead of 300. Ticket #1087.

Revision 666bc4d1 (diff)
Added by Ermal Luçi over 8 years ago

Ticket #1087. Cleanup whitespace and also do not rely on having the db dirty flag set for backing up the dbs but always do this!

History

#1 Updated by Jim Pingle over 8 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

#3 Updated by Jim Pingle over 8 years ago

I backed out the changes I made. This was already present under the voucher config, there is an interval that can be set which saves the voucher data into the config periodically. Ermal has the details.

#4 Updated by Ermal Luçi over 8 years ago

Just test with a snapshot including all the changes.
Should be ok now.

#5 Updated by Chris Buechler over 8 years ago

oh, yeah it has a save interval there, guessing that wasn't working previously. Needs testing.

#6 Updated by Ermal Luçi over 8 years ago

I committed code to prevent that foreach error.

#7 Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF