rjgoyln commented on code in PR #65099:
URL: https://github.com/apache/airflow/pull/65099#discussion_r3070917425
##########
airflow-ctl/src/airflowctl/api/client.py:
##########
@@ -294,8 +294,9 @@ def load(self) -> Credentials:
raise AirflowCtlKeyringException("Keyring backend
is not available") from e
self.api_token = None
except FileNotFoundError:
- # This is expected during the auth login command
- if self.client_kind != ClientKind.AUTH:
+ # This is expected during the auth login command.
+ # Also allow token-only usage without local config (for commands
like `version --remote`).
+ if self.client_kind != ClientKind.AUTH and self.api_token is None:
Review Comment:
Thanks for the feedback, @bugraoz93! I agree that adding ClientKind.NO_AUTH
is a cleaner and more robust approach than extra conditional checks.
I’ve made the changes—let me know if anything needs adjustment.
Also, I couldn’t find your `NO_AUTH`implementation in #61882, sorry!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
