Skip to content

fix(View): Catch exceptions when executing mkdir for non-existent parents - #49880

Merged
provokateurin merged 1 commit into
masterfrom
fix/view/catch-mkdir-exception-non-existent-parents
Dec 17, 2024
Merged

fix(View): Catch exceptions when executing mkdir for non-existent parents#49880
provokateurin merged 1 commit into
masterfrom
fix/view/catch-mkdir-exception-non-existent-parents

Conversation

@provokateurin

Copy link
Copy Markdown
Member

Summary

Follow-up to #47417.
We can not simply assume that the user is allowed to create the parent folder, for example groupfolders ACL or files_accesscontrol rules could prevent it.

Checklist

@provokateurinprovokateurin added bug 3. to review Waiting for reviews labels Dec 16, 2024
@provokateurinprovokateurin added this to the Nextcloud 31 milestone Dec 16, 2024
@provokateurin
provokateurin requested review from a team, Altahrim, artonge and sorbaugh and removed request for a teamDecember 16, 2024 11:53
@provokateurin

Copy link
Copy Markdown
MemberAuthor

/backport to stable30

@provokateurin

Copy link
Copy Markdown
MemberAuthor

/backport to stable29

@provokateurin

Copy link
Copy Markdown
MemberAuthor

/backport to stable28

Comment threadlib/private/Files/View.php
@provokateurin
provokateurinforce-pushed the fix/view/catch-mkdir-exception-non-existent-parents branch from afef56a to 0e00d84CompareDecember 16, 2024 13:02
…ents
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin
provokateurinforce-pushed the fix/view/catch-mkdir-exception-non-existent-parents branch from 0e00d84 to 4bc0b58CompareDecember 16, 2024 15:00
@provokateurin
provokateurin merged commit f16d047 into masterDec 17, 2024
@provokateurin
provokateurin deleted the fix/view/catch-mkdir-exception-non-existent-parents branch December 17, 2024 08:30
@skjnldsv

Copy link
Copy Markdown
Member

/backport to stable29

@skjnldsv

Copy link
Copy Markdown
Member

/backport to stable30

@skjnldsvskjnldsv mentioned this pull request Jan 7, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@provokateurin@skjnldsv@nfebe@come-nc