Skip to content

Say what a release is missing before spending six minutes on it - #3

Merged
thorstenalpers merged 1 commit into
mainfrom
fix/release-preflight
Aug 31, 2026
Merged

Say what a release is missing before spending six minutes on it#3
thorstenalpers merged 1 commit into
mainfrom
fix/release-preflight

Conversation

@thorstenalpers

Copy link
Copy Markdown
Owner

The release run on main fails at the last step:

failed to decode secret key: incorrect updater private key password: Missing comment in secret key

That reads as a broken key. It is not — there is no key. The repository has zero Actions secrets, so TAURI_SIGNING_PRIVATE_KEY expands to an empty string, the bundler tries to parse it as a minisign secret, and the parser complains about the comment line an empty string does not have. It got that far only after compiling the whole app and producing the installer.

This checks first, in seconds, and names both halves — because the public one is missing too:

state on main
secret TAURI_SIGNING_PRIVATE_KEY not set
secret TAURI_SIGNING_PRIVATE_KEY_PASSWORD not set
plugins.updater.pubkey ""

The public key matters as much as the private one: with it empty, a signed installer would still ship an app that cannot verify anything it downloads.

What still has to happen outside this PR

npx tauri signer generate -w openeventviewer.key

The public half goes into plugins.updater.pubkey in src-tauri/tauri.conf.json as its own commit; the private half and its password become the two secrets. The private half never enters the repository.

🤖 Generated with Claude Code

With no signing key the bundler is handed an empty string, tries to read
it as a minisign secret and reports "Missing comment in secret key" —
which reads as a broken key when the truth is that there is no key. The
run got that far only after building the whole app and the installer.

Checked first now, in seconds, naming both halves: the private one as a
secret and the public one in tauri.conf.json, which is just as absent and
would leave the shipped app unable to verify anything it downloaded.
@thorstenalpers
thorstenalpers merged commit fc7e281 into main Aug 31, 2026
2 checks passed
@thorstenalpers
thorstenalpers deleted the fix/release-preflight branch August 31, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant