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

enables the file name check also to match name of mountpoints #24703

Merged
merged 2 commits into from
Dec 18, 2020

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented Dec 14, 2020

When you have an external storage configured, you can assign tags to it as well. The file name checker of the workflow engine could not do so, because the filename is technically empty, instead we have a mountpoint name.

Thus, if we receive an empty file and identify an external storage, we return the name of the mountpoint, unless it is mounted as user home (/).

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz added this to the Nextcloud 21 milestone Dec 14, 2020
@rullzer rullzer mentioned this pull request Dec 15, 2020
59 tasks
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz merged commit 7a1b45a into master Dec 18, 2020
@blizzz blizzz deleted the fix/noid/wfe-filename-mountpoint-check branch December 18, 2020 10:43
@blizzz
Copy link
Member Author

blizzz commented Dec 18, 2020

/backport to stable20

@blizzz
Copy link
Member Author

blizzz commented Dec 18, 2020

/backport to stabl19

@blizzz
Copy link
Member Author

blizzz commented Dec 18, 2020

/backport to stable18

@backportbot-nextcloud
Copy link

The backport to stabl19 failed. Please do this backport manually.

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.

None yet

2 participants