Skip to content

Conversation

@icewind1991
Copy link
Member

this fixes external storages with / as mountpoint

it lessens the performance improvement a bit, in the future we can look at only disabling this optimization if there is an external storage with a / mountpoint.

Fixes #32367

this fixes external storages with '/' as mountpoint

Signed-off-by: Robin Appelman <[email protected]>
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Jun 9, 2022
@icewind1991 icewind1991 added this to the Nextcloud 25 milestone Jun 9, 2022
@icewind1991 icewind1991 requested review from a team, ArtificialOwl, blizzz and juliusknorr and removed request for a team June 9, 2022 17:24
@icewind1991
Copy link
Member Author

/backport to stable24

Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@blizzz blizzz merged commit b1616de into master Jun 9, 2022
@blizzz blizzz deleted the fix-external-home-dir branch June 9, 2022 22:44
@ShreshthTiwari
Copy link

ShreshthTiwari commented Jan 29, 2024

I have 3 hard disks mounted in /mnt directory other 2 are being shown inside the nextcloud but one of them is showing the same
Upload some content or sync with your devices

I even have changed the owner of all the files recusively
chwon -R www-data:www-data *
inside /mnt

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: files app bugs out when setting an external storage as root storage

5 participants