fix: workaround for keyring 25 incompatibility
Metadata
Current evaluation
Merged a one-line workaround for keyring 25 incompatibility in commit 6ef080b, addressing canonical/craft-store#162.
Resolution: closed resolved
Reason: PR #163 was approved and merged without unresolved review comments. Commit 6ef080b applied the workaround; PR #206 later added the full compatibility fix in bdcbf0e.
Evaluated by: mr-cal-bot
No scores available.
Issue body
https://github.com/canonical/craft-store/issues/162
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged a one-line workaround for keyring 25 incompatibility in commit 6ef080b, addressing canonical/craft-store#162. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged the keyring 25 compatibility workaround in 6ef080b; a fuller fix later landed in #206 (bdcbf0e). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged a one-line workaround for keyring 25 incompatibility in commit 6ef080b, addressing canonical/craft-store#162. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a one-line workaround to resolve keyring 25 incompatibility. Approved by two reviewers and passed CI checks, addressing issue #162. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-store#162
(confidence 100%)
Linked keyring 25 incompatibility report addressed by this workaround.
-
Superseded By:
canonical/craft-store#206
(confidence 95%)
Later merged full keyring 25 compatibility fix in commit bdcbf0e.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #206 fix: make compatible with keyring 25 | craft-store | merged | Merged canonical/craft-store#206 in commit bdcbf0e, adding a two-file compatibility fix for keyring 25 and resolving issue #162. | |
| #162 craft-store is not compatible with keyring 25.x | craft-store | closed | Fixed keyring 25 compatibility with workaround 6ef080b and full fix PR #206, merged as bdcbf0e. | |
| #1388 build(deps): bump craft-store to 2.5.0 | charmcraft | merged | Merged PR #1388 as commit 70066d49, upgrading craft-store to 2.5.0 to add a file-based keyring fallback on headless systems and fix #719. | |
| #75 auth: handle keyring unlocking errors | craft-store | merged | Merged PR #75 as commit 74a2ed4, adding keyring unlock error handling that raises a Craft-store error instead of exposing the backend exception. | |
| #390 fix(auth): error more nicely on keyring errors | craft-store | merged | Merged PR #390 (commit d87f1bd) broadens keyring error handling, wrapping more keyring failures as KeyringUnlockError. Fixes #58 and ships in craft-store 3.4.0. | |
| #38 auth: avoid MemoryKeyring in keyring and raise appropriately (CRAFT-771) | craft-store | merged | Merged in canonical/craft-store#38 via d7ee75f, preventing automatic MemoryKeyring selection and raising NoKeyringError during StoreClient initialization when no keyring exists. | |
| #170 fix: catch keyring locked error | craft-store | merged | Merged PR #170 added handling for locked keyring errors, fixing #166; merged as commit ee19448. | |
| #4455 build(deps): bump craft-store to 2.5.0 | snapcraft | merged | Merged PR #4455 (commit 9b5c2f4cf) bumped craft-store to 2.5.0, enabling headless systems to use a file-based keyring fallback for login. | |
| #1741 fix(store): fall back to ephemeral keyring | charmcraft | merged | Merged PR #1741 as commit 5fa2a47c, adding an ephemeral keyring fallback when none is available and fixing #1717. |