Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
Wrede committed Sep 22, 2024
1 parent 404c69f commit 548833d
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion fedn/cli/client_cmd.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,12 @@ def validate_client_config(config):
"""
try:
if config["discover_host"] is None or config["discover_host"] == "":
raise InvalidClientConfig("Missing required configuration: discover_host")
if config["combiner"] is None or config["combiner"] == "":
raise InvalidClientConfig("Missing required configuration: discover_host or combiner")
if "discover_port" not in config.keys():
config["discover_port"] = None
if config["remote_compute_context"] and config["discover_host"] is None:
raise InvalidClientConfig("Remote compute context requires discover_host")
except Exception:
raise InvalidClientConfig("Could not load config from file. Check config")

Expand Down

0 comments on commit 548833d

Please sign in to comment.