diff options
author | jim-p <jimp@pfsense.org> | 2013-10-22 10:51:28 -0400 |
---|---|---|
committer | jim-p <jimp@pfsense.org> | 2013-10-22 10:53:02 -0400 |
commit | afdf29d3a407d4d35bc16a32c01370bcf3197574 (patch) | |
tree | e1194e6773e2b1688906776bf3d1c49a6f516a71 /usr/local/www/system_authservers.php | |
parent | 2b6e6fee8076fb9b677c8bb3d7753c62081cb023 (diff) | |
download | pfsense-afdf29d3a407d4d35bc16a32c01370bcf3197574.zip pfsense-afdf29d3a407d4d35bc16a32c01370bcf3197574.tar.gz |
Make the RADIUS settings respect the description of the timeout field. If the timeout value is left blank, use 5 seconds, don't print an error.
Diffstat (limited to 'usr/local/www/system_authservers.php')
-rw-r--r-- | usr/local/www/system_authservers.php | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/usr/local/www/system_authservers.php b/usr/local/www/system_authservers.php index 62d0ed8..96c7216 100644 --- a/usr/local/www/system_authservers.php +++ b/usr/local/www/system_authservers.php @@ -215,7 +215,7 @@ if ($_POST) { if (auth_get_authserver($pconfig['name']) && !isset($id)) $input_errors[] = gettext("An authentication server with the same name already exists."); - if (($pconfig['type'] == "radius") && isset($_POST['radius_timeout']) && (!is_numeric($_POST['radius_timeout']) || (is_numeric($_POST['radius_timeout']) && ($_POST['radius_timeout'] <= 0)))) + if (($pconfig['type'] == "radius") && isset($_POST['radius_timeout']) && !empty($_POST['radius_timeout']) && (!is_numeric($_POST['radius_timeout']) || (is_numeric($_POST['radius_timeout']) && ($_POST['radius_timeout'] <= 0)))) $input_errors[] = gettext("RADIUS Timeout value must be numeric and positive."); /* if this is an AJAX caller then handle via JSON */ @@ -277,6 +277,8 @@ if ($_POST) { if ($pconfig['radius_timeout']) $server['radius_timeout'] = $pconfig['radius_timeout']; + else + $server['radius_timeout'] = 5; if ($pconfig['radius_srvcs'] == "both") { $server['radius_auth_port'] = $pconfig['radius_auth_port']; |