admin-valentin/arcade-mcp
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: No output from arcade login when already logged in (#143)

Browse source 
I ran into this and was scratching my head why `arcade login` was
exiting with no output.

Fix: Get config data from the new top-level object we introduced.
This commit is contained in:
Nate Barbettini 2024-11-01 16:49:31 -07:00 committed by GitHub
parent aff7350eec
commit 1fdf982238
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
arcade/arcade/cli/authn.py
View file

@ -133,14 +133,12 @@ def check_existing_login() -> bool:
try: try:
with open(config_file_path) as f: with open(config_file_path) as f:
config: dict[str, Any] = yaml.safe_load(f) config: dict[str, Any] = yaml.safe_load(f)
api_key = config.get("api", {}).get("key") cloud_config = config.get("cloud", {})
email = config.get("user", {}).get("email") api_key = cloud_config.get("api", {}).get("key")
email = cloud_config.get("user", {}).get("email")
if api_key and email: if api_key and email:
console.print( console.print(f"You're already logged in as {email}. ", style="bold green")
f"You're already logged in as {email}. "
f"Delete {config_file_path} to log in as a different user."
)
return True return True
except yaml.YAMLError: except yaml.YAMLError:
console.print( console.print(

1
arcade/arcade/cli/main.py
View file

@ -64,6 +64,7 @@ def login(
""" """
if check_existing_login(): if check_existing_login():
console.print("Delete ~/.arcade/credentials.yaml to log in as a different user.\n")
return return
# Start the HTTP server in a new thread # Start the HTTP server in a new thread