-
Notifications
You must be signed in to change notification settings - Fork 138
Prevent download on user shares #2280
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
Conversation
Raudius
left a comment
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 <[email protected]>
03d2819 to
6f904ca
Compare
Depends on nextcloud/server#32482
Implements #2267
Set the
hideDownloadattribute 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.