Skip to content

Only check path for being accessible when the storage is a object home#24103

Merged
MorrisJobke merged 1 commit intomasterfrom
bugfix/noid/groupfolder-share-object-storage
Nov 19, 2020
Merged

Only check path for being accessible when the storage is a object home#24103
MorrisJobke merged 1 commit intomasterfrom
bugfix/noid/groupfolder-share-object-storage

Conversation

@juliusknorr
Copy link
Member

This fixes an issue where shares originating from groupfolders were not showing up at the recipient due to the check for a valid "files" path being applied to all object storages instead of just the user ones.

For groupfolders the storage would be something like object::store:amazon::nc-65a6889e4cd6 where the previous check caused a false positive.

@MorrisJobke MorrisJobke merged commit c2510ec into master Nov 19, 2020
@MorrisJobke MorrisJobke deleted the bugfix/noid/groupfolder-share-object-storage branch November 19, 2020 21:37
@MorrisJobke
Copy link
Member

@juliushaertl Backport?

@juliusknorr
Copy link
Member Author

/backport to stable20

@juliusknorr
Copy link
Member Author

/backport to stable19

@juliusknorr
Copy link
Member Author

/backport to stable18

@backportbot-nextcloud

This comment has been minimized.

mejo- added a commit to nextcloud/deck that referenced this pull request Jul 2, 2024
mejo- added a commit to nextcloud/spreed that referenced this pull request Jul 2, 2024
mejo- added a commit to nextcloud/spreed that referenced this pull request Jul 2, 2024
backportbot bot pushed a commit to nextcloud/deck that referenced this pull request Jul 2, 2024
backportbot bot pushed a commit to nextcloud/deck that referenced this pull request Jul 2, 2024
backportbot bot pushed a commit to nextcloud/spreed that referenced this pull request Jul 2, 2024
backportbot bot pushed a commit to nextcloud/spreed that referenced this pull request Jul 2, 2024
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