Skip to content

Keep the display-helper signing keychain on the search list - #457

Merged
theogravity merged 1 commit into
mainfrom
fix/desktop-helper-keychain
Oct 26, 2026
Merged

theogravity merged 1 commit into
mainfrom
fix/desktop-helper-keychain

Conversation

@theogravity

Copy link
Copy Markdown
Contributor

The cut dispatch on #454 (run 40306776766) died on all darwin desktop shards with the same The specified item could not be found in the keychain. #454 imported the helper's signing identity into a dedicated keychain and then took it back off the user's search list, relying on codesign --keychain alone; identity resolution walks the search list, so the import was invisible to the lookup. Keep the keychain listed for the build's lifetime (same single identity as Tauri's later bundle keychain, so no ambiguity), restore the baseline in the step trap. Docs paragraph follows.

#454 created and imported a dedicated keychain for the display helper's
build-script signature, then took it back off the user's search list and
relied on codesign's --keychain alone. The dispatch on that commit
(run 40306776766) died with the same "The specified item could not be
found in the keychain": codesign resolves a named identity through the
search list, not the --keychain argument, so an unlisted keychain is
invisible to the lookup. Leave the keychain where create-keychain put
it for the build's lifetime; it carries the same single identity as
Tauri's bundle-time keychain, so there is no ambiguity, and the step
trap restores the baseline. Both duplicated build-job bodies edited
identically; docs paragraph follows.
@theogravity
theogravity merged commit f775281 into main Oct 26, 2026
14 checks passed
@theogravity
theogravity deleted the fix/desktop-helper-keychain branch October 26, 2026 11:35
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant