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

"Allow Download" permission for shared folders removed but download or save as via Onlyoffice still possible #687

Closed
timm2k opened this issue Sep 6, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@timm2k
Copy link

timm2k commented Sep 6, 2022

Hello Onlyoffice team,

If I remove Nextcloud's share permission Allow download form a folder containing documents ONLYOFFICE still let me download or use save as for the protected content.

Nextcloud PR#32482: Add share attributes + prevent download permission

This permission is not respected by the secure view function yet.

For the collabora plugin this was fixed several days ago.
Are you able to put the deselected Allow download under secure view too?

PR for fix in richdocuments plugin

Kind regards
Timm

@Antipkin-A Antipkin-A added the bug Something isn't working label Sep 9, 2022
@timm2k
Copy link
Author

timm2k commented Oct 17, 2022

Hi @Antipkin-A,

I've applied your patch 2fac82e to my test system. I'm still able to use file menu -> download as or save copy as in ONLYOFFICE webinterface for shared files without Allow Download permission.

I think this is not the expected behavior of the patch?
Should the patch deny access to a document completely?

My intention was to have a view only and remove the export functions from ONLYOFFICE UI is Allow Download is set to false (like described on https://nextcloud.com/de/blog/secure-view-prevent-your-shared-files-from-getting-downloaded/).

Kind Regards
Timm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants