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.

This commit is contained in:
jim-p 2013-10-22 10:51:28 -04:00
parent f144493c68
commit aad5eef2ab

View File

@ -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'];