Project

General

Profile

Actions

Feature #1492

closed

Captive Portal Interim Updates

Added by Warren Baker almost 13 years ago. Updated almost 11 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Captive Portal
Target version:
Start date:
05/04/2011
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:

Description

Currently there is nothing in webGUI to configure how often Captive Portal pruning should be executed. So the default of 60seconds is used.
This has the side affect that if RADIUS Accounting is been used then interim updates are sent every 60seconds.

captiveportal.inc does cater for this by using $config['captiveportal']['croninterval'] as the variable to startup the minicron, so all that needs to happen is services_captiveportal.php needs a field added for these updates.

What would a reasonable time frame be? Minimum 60 seconds, maximum 3600 seconds? Not sure if the latter is used how this would impact on Captive Portal pruning?
If it does then possibly interim updates needs to be split out from the pruning process?

Actions #1

Updated by Chris Buechler almost 13 years ago

  • Tracker changed from Bug to Feature
Actions #2

Updated by Chris Buechler almost 13 years ago

  • Affected Version set to All

Not sure we want to expose this for 2.0, potential for fallout since that's never been exposed, though it's probably ok. Are there specific scenarios where this causes problems?

60-3600 is probably reasonable. Allowing too small of a number could lead to very large installs, or very slow boxes with quite a few users, to potentially have issues with running it multiple times simultaneously.

Actions #3

Updated by Ermal Luçi almost 13 years ago

  • Target version changed from 2.0 to 2.1
Actions #4

Updated by Michael Newton over 11 years ago

Ideally, pfSense should accept the Acct-Interim-Interval sent by the server to determine interim update interval, as other NAS devices do. I would agree this should be separate from the pruning interval, which needs to be a small value like 60 seconds.

Actions #5

Updated by Michael Newton over 11 years ago

Perhaps something like this, which is based on what we use in production for 2.0: https://github.com/bsdperimeter/pfsense/pull/237

Actions #6

Updated by Ermal Luçi about 11 years ago

Merged manually without the GUI option.

Actions #7

Updated by Ermal Luçi about 11 years ago

  • Status changed from New to Feedback
Actions #8

Updated by Ermal Luçi about 11 years ago

  • % Done changed from 0 to 100
Actions #9

Updated by Chris Buechler almost 11 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF