Skip to content

[stable9] Fallback to share link owner when no owner found (#26587)#26620

Merged
PVince81 merged 1 commit into
stable9from
stable9-0b91f8d976b04fef24992792c485f728925c2469
Nov 14, 2016
Merged

[stable9] Fallback to share link owner when no owner found (#26587)#26620
PVince81 merged 1 commit into
stable9from
stable9-0b91f8d976b04fef24992792c485f728925c2469

Conversation

@DeepDiver1975

Copy link
Copy Markdown
Member

backport of #26587

When creating link shares from external storage, the filesystem cannot
find an owner in some scenarios (ex: system-wide mounts). In such
cases, fall back to using the current user's trashbin which happens to
also be the user who created the link share.

Fixes an issue where this scenario made deletion impossible due to
missing user information.

When creating link shares from external storage, the filesystem cannot
find an owner in some scenarios (ex: system-wide mounts). In such
cases, fall back to using the current user's trashbin which happens to
also be the user who created the link share.

Fixes an issue where this scenario made deletion impossible due to
missing user information.
@DeepDiver1975 DeepDiver1975 added this to the 9.0.7 milestone Nov 14, 2016
@mention-bot

Copy link
Copy Markdown

@DeepDiver1975, thanks for your PR! By analyzing the history of the files in this pull request, we identified @schiessle, @icewind1991 and @Xenopathic to be potential reviewers.

@PVince81

Copy link
Copy Markdown
Contributor

Tested, works 👍

@PVince81
PVince81 merged commit f31082b into stable9 Nov 14, 2016
@PVince81
PVince81 deleted the stable9-0b91f8d976b04fef24992792c485f728925c2469 branch November 14, 2016 10:35
@lock

lock Bot commented Aug 4, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants