sneak/secret
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Non-darwin KeychainUnlocker stub returns errors instead of panicking (closes #7) #12

Merged
sneak merged 2 commits from :fix/issue-7 into main 2026-02-09 02:20:14 +01:00
Conversation 0 Commits 2 Files Changed 1 +30 -21
clawbot commented 2026-02-08 21:06:05 +01:00
Collaborator
Copy Link

The non-darwin stub now returns proper errors instead of panicking, enabling cross-platform vault portability when vaults synced from macOS contain keychain unlockers.

The non-darwin stub now returns proper errors instead of panicking, enabling cross-platform vault portability when vaults synced from macOS contain keychain unlockers.
sneak was assigned by clawbot 2026-02-08 21:06:05 +01:00
clawbot added 1 commit 2026-02-08 21:06:05 +01:00
fix: non-darwin KeychainUnlocker stub returns errors instead of panicking 4419ef7730 
The stub previously panicked on all methods including NewKeychainUnlocker,
which is called from vault code when processing keychain-type unlocker
metadata. This caused crashes on Linux/Windows when a vault synced from
macOS contained keychain unlockers.

Now returns proper error values, allowing graceful degradation and
cross-platform vault portability.
sneak added 1 commit 2026-02-09 02:19:58 +01:00
Merge branch 'main' into fix/issue-7 51fb2805fd
sneak merged commit 660de5716a into main 2026-02-09 02:20:14 +01:00
sneak deleted branch fix/issue-7 2026-02-09 02:20:19 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees sneak
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/secret#12
Delete Branch ":fix/issue-7"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?