Skip to content

[stable33] fix: nightly release signature validation#6728

Merged
vitormattos merged 2 commits intostable33from
backport/6726/stable33
Feb 5, 2026
Merged

[stable33] fix: nightly release signature validation#6728
vitormattos merged 2 commits intostable33from
backport/6726/stable33

Conversation

@backportbot-libresign
Copy link

Backport of PR #6726

The occ integrity:sign-app command needs write access to the appinfo
directory to create signature.json. This adds chmod -R a+w after copying
appinfo to ensure the directory is writable regardless of file ownership.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
The 'Sign app' step was re-signing an already signed tarball created
by the Makefile. This caused signature validation failures on the
Nextcloud Appstore API because:

1. Makefile correctly signs at build/artifacts/sign/libresign
2. Duplicate step extracted, re-signed at wrong path, and recreated tarball
3. This corrupted the original valid signature
4. API rejected with 'invalid padding' or 'bad signature' errors

The Makefile already handles all signing correctly:
- Downloads certificate from GitHub
- Signs setup files with libresign:developer:sign-setup
- Signs app files with occ integrity:sign-app
- Creates properly signed tarball

Removing this redundant step fixes nightly releases not appearing
on apps.nextcloud.com.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@github-project-automation github-project-automation bot moved this to 0. Needs triage in Roadmap Feb 5, 2026
@backportbot-libresign backportbot-libresign bot added this to the Next RC (33) milestone Feb 5, 2026
@vitormattos vitormattos merged commit b743232 into stable33 Feb 5, 2026
59 checks passed
@vitormattos vitormattos deleted the backport/6726/stable33 branch February 5, 2026 16:34
@github-project-automation github-project-automation bot moved this from 0. Needs triage to 4. to release in Roadmap Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 4. to release

Development

Successfully merging this pull request may close these issues.

1 participant