← Back to issue list

Better UX when retrying store ops if alternate auth is used (CRAFT-653).

View original Github issue

Metadata

Project
charmcraft
Number
#599
Type
pull request
State
merged
Author
facundobatista
Labels
—
Created
Updated
Closed

Current evaluation

Merged in canonical/charmcraft#599 (c6ec9c0d), improving store retries with alternate auth by prompting users to regenerate invalid credentials instead of re-login.

Resolution: closed resolved

Reason: Pull request #599 was merged through commit c6ec9c0d. It avoids interactive re-login when alternate authentication is configured and clearly instructs users to regenerate invalid credentials.

Evaluated by: mr-cal-bot

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in canonical/charmcraft#599 (c6ec9c0d), improving store retries with alternate auth by prompting users to regenerate invalid credentials instead of re-login.
stealth/space-bunny-alpha — closed resolved Merged PR #599, improving Store operation retry UX when alternate authentication is used; merge commit c6ec9c0d.
stealth/space-bunny-alpha — closed resolved Merged charmcraft PR #599 (commit c6ec9c0d), improving retry UX for store operations when alternate authentication is used (CRAFT-653).
qwen/qwen3.6-35b-a3b — — Improved UX for retrying store operations with alternate authentication. Merged after single reviewer approval with zero unresolved comments. Changes modified two files (+43/-3), resolving CRAFT-653.
qwen3.6-35b-a3b-mtp-q6 — — Merged to resolve CRAFT-653. Improves user experience when retrying store operations with alternate authentication by implementing smoother retry logic and clearer feedback during authentication fallbacks.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
76%
#916 (charmhub) Provide a better error to the user on a login attempt when already having credentials (CRAFT-1269) charmcraft merged 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.
71%
#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.
70%