(charmhub) Provide a better error to the user on a login attempt when already having credentials (CRAFT-1269)
Metadata
Current evaluation
Merged PR #916 in commit 3b54c459, improving the double-login error to explain that stored credentials may be invalid and recommend logging out; fixes #833.
Resolution: closed resolved
Reason: PR #916 was merged as commit 3b54c459, with the implementation in 5e656275. Its improved Charmhub login error advises users to log out before logging in again and closes issue #833.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes #833.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #916 in commit 3b54c459, improving the double-login error to explain that stored credentials may be invalid and recommend logging out; fixes #833. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #916 (merge commit 3b54c459), improving Charmhub's error when credentials already exist and fixing #833 (CRAFT-1269). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged changes that improve Charmhub login error messaging when credentials already exist, resolving issue #833 and ticket CRAFT-1269. Approved by one reviewer with minimal code modifications. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix improving Charmhub login error messaging when credentials already exist, resolving issue #833 (CRAFT-1269). |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/charmcraft#833
(confidence 100%)
The original double-login error report explicitly fixed by PR #916.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #833 Provide a better error description and suggest solution for the "double login" case | charmcraft | closed | Improved the double-login error to warn that stored credentials may be invalid and recommend logging out first; merged in PR #916 via commit 3b54c459. | |
| #759 charmcraft login produces an error | charmcraft | closed | Closed as superseded by #833, which improved the double-login error and recommended `charmcraft logout`; the follow-up was resolved by merged PR #916. | |
| #725 craft-store error when using `charmcraft login` command | charmcraft | closed | Confusing craft-store "Credentials found" error on `charmcraft login` (double login). Closed by facundobatista as tracked in the more specific #833, whose improved error landed via PR #916. | |
| #810 Confusing output from `charmcraft login` | charmcraft | closed | Superseded by #833 and resolved by merged PR #916, which clarified that stored credentials may be invalid and advised users to log out before logging in again. | |
| #609 store: handle store errors in whoami command (CRAFT-701) | charmcraft | merged | Merged in canonical/charmcraft#609 via 3d7474b1 (merge 69df5a9a), adding allow_login so whoami reports logged-out and expired-credential errors without automatic login. | |
| #599 Better UX when retrying store ops if alternate auth is used (CRAFT-653). | charmcraft | merged | Merged in canonical/charmcraft#599 (c6ec9c0d), improving store retries with alternate auth by prompting users to regenerate invalid credentials instead of re-login. | |
| #597 Forbid login/logout when using alternate auth (CRAFT-646). | charmcraft | merged | Merged PR #597 (CRAFT-646) into main via commit fb8a160b, merge 13a79cf9: charmcraft's store client now raises CraftError to forbid `login`/`logout` when alternate auth is set via the env var. Follow-up PR #599 improved retry UX. |