rjgoyln opened a new pull request, #71195: URL: https://github.com/apache/airflow/pull/71195
## Summary `airflowctl auth login` could report success while the token never reached the keyring. `Credentials.save()` writes the API url config before storing the token, and when keyring rejected the token the `TypeError` handler only raised for `ClientKind.CLI` — a kind neither call site passes. Every other case, including a genuine keyring backend failure, returned as though the save had worked, leaving the user with a config file and no credentials. ## Behavior change A missing token now fails for every client kind, and a `TypeError` unrelated to a missing token propagates instead of being discarded. An unavailable keyring backend still reports the keyring error first, as before. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 5) Generated-by: Claude Code (Opus 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
