From aad5eef2ab645d5da68d17b39daf7a33783eb8a7 Mon Sep 17 00:00:00 2001 From: jim-p Date: Tue, 22 Oct 2013 10:51:28 -0400 Subject: [PATCH] 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. --- usr/local/www/system_authservers.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/usr/local/www/system_authservers.php b/usr/local/www/system_authservers.php index 2984d8cd6f..66b188e354 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'];