Skip to content

Conversation

@mgallien
Copy link
Collaborator

if the folder has a fileid, it means that it was created on server

most probably we can encrypt it with success

previous code was skiping encryption if some ignored files have been reported (even if it was unrelated to the folder we want to encrypt)

@mgallien mgallien added this to the 4.1.0 milestone Nov 27, 2025
if the folder has a fileid, it means that it was created on server

most probably we can encrypt it with success

previous code was skiping encryption if some ignored files have been
reported (even if it was unrelated to the folder we want to encrypt)

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien enabled auto-merge November 27, 2025 09:16
@mgallien mgallien force-pushed the bugfix/doNotPreventEncryptionWithIgnoredFiles branch from d7ef7cb to 866864d Compare November 27, 2025 09:16
@mgallien
Copy link
Collaborator Author

/backport to stable-4.0

@github-actions
Copy link

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

Digest: sha256:17520c436dc3067666f90d58d43d77198c25e0aa1e726a6539cc3158b2c4f1a0

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.

@mgallien mgallien merged commit 9695762 into master Nov 27, 2025
20 of 21 checks passed
@mgallien mgallien deleted the bugfix/doNotPreventEncryptionWithIgnoredFiles branch November 27, 2025 09:55
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)
16 New Code Smells (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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants