dockersetup: allow updating if proxy config set in environment

Don't confuse proxy options being defaulted through from the environment
and the user explicitly specifying them. Also look at no_proxy option.

Fixes https://github.com/intel/clear-linux-dissector-web/issues/13

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
This commit is contained in:
Paul Eggleton 2019-09-10 14:47:40 +12:00
parent 050fbb0878
commit 16dc1eb933

View File

@ -69,7 +69,7 @@ def get_args():
args = parser.parse_args() args = parser.parse_args()
if args.update: if args.update:
if args.http_proxy or args.https_proxy or args.databasefile or args.no_https or args.cert or args.cert_key or args.letsencrypt: if args.http_proxy != default_http_proxy or args.https_proxy != default_https_proxy or args.no_proxy != default_no_proxy or args.databasefile or args.no_https or args.cert or args.cert_key or args.letsencrypt:
raise argparse.ArgumentTypeError("The -u/--update option will not update configuration or database content, and thus none of the other configuration options can be used in conjunction with it") raise argparse.ArgumentTypeError("The -u/--update option will not update configuration or database content, and thus none of the other configuration options can be used in conjunction with it")
if args.reinstall: if args.reinstall:
raise argparse.ArgumentTypeError("The -u/--update and -r/--reinstall options are mutually exclusive") raise argparse.ArgumentTypeError("The -u/--update and -r/--reinstall options are mutually exclusive")