Skip to content
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

[stable30] fix: Filename validation should only forbid create and update #47583

Merged
merged 5 commits into from
Aug 28, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Aug 28, 2024

Backport of #47185

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@susnux
Copy link
Contributor

susnux commented Aug 28, 2024

/compile

@susnux susnux marked this pull request as ready for review August 28, 2024 17:19
@susnux susnux requested a review from skjnldsv as a code owner August 28, 2024 17:19
Renaming is basically copy + delete (a move), so no need to update permissions.
Especially if the node is in a invalid directory the node should be moveable but not editable.

Signed-off-by: Ferdinand Thiessen <[email protected]>
Needed to read files with the "Windows compatibility" feature.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux
Copy link
Contributor

susnux commented Aug 28, 2024

/compile

Signed-off-by: nextcloud-command <[email protected]>
@susnux
Copy link
Contributor

susnux commented Aug 28, 2024

Needs a force merge due to RC block.

@AndyScherzinger AndyScherzinger merged commit 318a6f7 into stable30 Aug 28, 2024
171 of 172 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/47185/stable30 branch August 28, 2024 20:07
@AndyScherzinger
Copy link
Member

DONE 👍

@blizzz blizzz mentioned this pull request Aug 29, 2024
24 tasks
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.

3 participants