Skip to content

fix(e2ee): do not loose encryption status for folders and files #8170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mgallien
Copy link
Collaborator

main issue is with child folders that were misbehaving once the initial upload was done

mgallien added 2 commits May 22, 2025 18:04
main issue is with child folders that were misbehaving once the initial
upload was done

Signed-off-by: Matthieu Gallien <[email protected]>
will stop failing to decode using private/publy key pair when
downloading encrypted content (i.e. symetric keys are encrypted using
asymetric encryption)

we improperly use a certificate hash to find the key while the private
key is the current one of the user downloading the files

Signed-off-by: Matthieu Gallien <[email protected]>
Copy link

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

SHA256 checksum: eb77db8a70a6f2b7577775aed36febeff4d2c9a8627f7d2b607788af2be959bc

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.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
8 Security Hotspots
39.4% Coverage on New Code (required ≥ 80%)
310 New Code Smells (required ≤ 0)
E Reliability Rating on New Code (required ≥ A)
E Maintainability Rating on New Code (required ≥ A)
1 New Bugs (required ≤ 0)

See analysis details on SonarQube Cloud

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

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