Skip to content

Conversation

@nfebe
Copy link
Contributor

@nfebe nfebe commented Apr 29, 2025

File shares can't support create or delete permissions. This change ensures those permissions are stripped or ignored when the shared item is not a folder.

@nfebe nfebe added 3. to review Waiting for reviews feature: sharing labels Apr 29, 2025
@nfebe nfebe requested a review from a team as a code owner April 29, 2025 10:36
@github-project-automation github-project-automation bot moved this to 🏗️ In progress in 📁 Files team Apr 29, 2025
Copy link
Contributor Author

@nfebe nfebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests pending

@nfebe nfebe force-pushed the fix/no-issue/prevent-create-delete-perms-on-file-shares branch from dd40338 to a124da4 Compare April 29, 2025 11:29
@nfebe
Copy link
Contributor Author

nfebe commented Apr 29, 2025

/compile

@nfebe nfebe enabled auto-merge April 29, 2025 11:30
@nfebe
Copy link
Contributor Author

nfebe commented Apr 29, 2025

/backport to stable31

@nfebe
Copy link
Contributor Author

nfebe commented Apr 29, 2025

/backport to stable30

@nfebe
Copy link
Contributor Author

nfebe commented Apr 29, 2025

/backport to stable29

@nfebe
Copy link
Contributor Author

nfebe commented Apr 29, 2025

/compile

File shares can't support create or delete permissions. This change ensures those permissions are
 stripped or ignored when the shared item is not a folder.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe nfebe force-pushed the fix/no-issue/prevent-create-delete-perms-on-file-shares branch from a124da4 to 99ae04d Compare April 29, 2025 11:42
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nfebe nfebe merged commit 008e9f2 into master Apr 29, 2025
131 of 133 checks passed
@nfebe nfebe deleted the fix/no-issue/prevent-create-delete-perms-on-file-shares branch April 29, 2025 12:14
@AndyScherzinger AndyScherzinger added this to the Nextcloud 32 milestone Apr 29, 2025
@AndyScherzinger AndyScherzinger moved this from 🏗️ In progress to ☑️ Done in 📁 Files team Apr 29, 2025
@AndyScherzinger
Copy link
Member

Thanks a lot for the swift fix 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

[Bug]: Error "Failed to update share" when using custom permissions during file sharing via email

7 participants