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

[stable28] fix(files): validate input when creating file/directory #46717

Merged
merged 4 commits into from
Jul 24, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jul 24, 2024

Backport of #46689

Warning, This backport's changes differ from the original

@backportbot backportbot bot requested review from artonge, susnux and Antreesy July 24, 2024 12:10
@backportbot backportbot bot added this to the Nextcloud 28.0.9 milestone Jul 24, 2024
If a mountpoint is returned from a `View` the name must match the view scoped name
and not the global name.
For example group folders have a name like `1` or `2` (the ID),
but the name of the FileInfo returned fro mthe View should be the mount point name,
like `First groupfolder` or similar.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@Antreesy Antreesy force-pushed the backport/46689/stable28 branch from b65a50c to 6d20832 Compare July 24, 2024 16:44
@Antreesy Antreesy marked this pull request as ready for review July 24, 2024 16:45
[stable28] fix: FileInfo from `View` should have the correct name of a mountpoint
@susnux susnux requested review from skjnldsv and nfebe and removed request for artonge July 24, 2024 18:43
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Copy link
Contributor

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux susnux merged commit 07b9068 into stable28 Jul 24, 2024
54 checks passed
@susnux susnux deleted the backport/46689/stable28 branch July 24, 2024 22:45
@Altahrim Altahrim mentioned this pull request Aug 8, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants