Skip to content

Comments

Bugfix/e2ee fix certificate migration#9514

Draft
mgallien wants to merge 4 commits intomasterfrom
bugfix/e2eeFixCertificateMigration
Draft

Bugfix/e2ee fix certificate migration#9514
mgallien wants to merge 4 commits intomasterfrom
bugfix/e2eeFixCertificateMigration

Conversation

@mgallien
Copy link
Collaborator

No description provided.

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots
240 New Code Smells (required ≤ 0)
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

we need to use the certificate that was used to encrypt folder metadata
when reading and decrypting them

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
we also want to get the fingerprint of a software certificate not only
for hardware ones

should enable us to have correct behavior in all cases (and maybe be
able to migrate software certificates if there is a need)

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien force-pushed the bugfix/e2eeFixCertificateMigration branch 2 times, most recently from 38ec924 to 90afdf0 Compare February 25, 2026 13:36
during discovery of all certificates, the private key handle can become
invalid

to avoid having invalid references, rather store the certificate handle
and use it to get the cached key

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien force-pushed the bugfix/e2eeFixCertificateMigration branch from 90afdf0 to efc1737 Compare February 25, 2026 14:44
@github-actions
Copy link

Artifact containing the AppImage: nextcloud-appimage-pr-9514.zip

Digest: sha256:b3dfd68ce7f4827d02520991daac1f840470bac1472c99665f8dfa91aa86fff6

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

ensure the proper owner user certificate is used after a migration

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant