From 51a2a9ad85bd6850c446290481fa550a7e23b339 Mon Sep 17 00:00:00 2001 From: Cameron LaPlante Date: Fri, 17 Jan 2025 11:56:23 -0800 Subject: [PATCH] lf_create_wanpath.py: LAN-1963 - added port verification logic Check that reserved ports (ie 4001 and 4002) are not used as mgr. Request alternative if attempted. Verified with: ./lf_create_wanpath.py --mgr 192.168.101.189 --mgr_port 4001\ --wp_name test_wp-B --wl_endp test_wl-B\ --log_level debug --debug ./lf_create_wanpath.py --mgr 192.168.101.189 --mgr_port 4002\ --wp_name test_wp-B --wl_endp test_wl-B\ --log_level debug --debug Signed-off-by: Cameron LaPlante --- py-scripts/lf_create_wanpath.py | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/py-scripts/lf_create_wanpath.py b/py-scripts/lf_create_wanpath.py index 154e41ea..1b597585 100755 --- a/py-scripts/lf_create_wanpath.py +++ b/py-scripts/lf_create_wanpath.py @@ -229,6 +229,21 @@ def add_wanpath(self, wanlink=wanlink) +def validate_port(port): + ''' Validate management port (not 4001 or 4002)''' + portsRestricted = ['4001', '4002'] + if port in portsRestricted: + logger.error('Ports 4001 and 4002 are reserved. Please choose another port. GUI is typically 8080.') + response = input("To continue enter non-reserved port: ") + if response != '': + return response + else: + logger.error("No new port selected. Exiting...") + exit(1) + else: + return port + + def main(): parser = argparse.ArgumentParser( @@ -286,6 +301,9 @@ def main(): args = parser.parse_args() + # validate port + args.mgr_port = validate_port(args.mgr_port) + logger_config = lf_logger_config.lf_logger_config() if args.log_level: logger_config.set_level(level=args.log_level)