-
Notifications
You must be signed in to change notification settings - Fork 120
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
Prevent download on user shares #2280
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
would this also apply for documents accessible through public link share ? |
@PVince81 Yes it already exists for public link shares: The view-only mode is enabled according to the "hide download" share attribute. Will the download permission be relevant in public shares? I mean, do we need to adapt the way we enable the view-only mode for public shares so that we check the "hide download" value AND the download permission? |
@eneiluj you don't need to adapt it then. When resharing with a public link, the download permission is converted to "hide download" to be able to benefit from existing logic. the perm from the share attributes is only intended for local shares. |
8531d65
to
5f5d160
Compare
/backport to stable24 |
|
Server PR is merged for master, I'm in the progress of backporting it |
5f5d160
to
4ef5088
Compare
|
83f687c
to
03d2819
Compare
…ssion Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
03d2819
to
6f904ca
Compare
Depends on nextcloud/server#32482
Implements #2267
Set the
hideDownload
attribute of the Wopi token when it's created. Get this from the share permissions. If multiple shares give access to the file, only one with download disabled will lead to hiding it in Collabora.