From 72b21bc83872d30ef9915dfed8aa275bd4e0c366 Mon Sep 17 00:00:00 2001 From: Mike Geppert Date: Wed, 7 Jan 2026 19:58:04 -0600 Subject: [PATCH] Disable SetOption verification to prevent overwhelming devices - Verification was doubling the number of HTTP requests per setting - Devices were refusing connections due to request overload - Trust command success based on response, don't verify separately - Fixes connection refused and timeout errors --- console_settings.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/console_settings.py b/console_settings.py index 8f2b6a1..a2fb72f 100644 --- a/console_settings.py +++ b/console_settings.py @@ -179,11 +179,9 @@ class ConsoleSettingsManager: if not success: self.logger.error(f"{device_name}: Failed to set {param_name} after 3 attempts") return False - - # Verify the command was applied (if possible) - if not self._verify_command(device_ip, device_name, param_name, param_value): - self.logger.warning(f"{device_name}: Verification failed for {param_name}") - # Don't return False here - some commands can't be verified + + # Verification disabled - it overwhelms devices with extra queries + # Trust that the command was applied if no error was returned # Check if this is a rule definition - if so, enable it if param_name.lower().startswith('rule'):