-
Notifications
You must be signed in to change notification settings - Fork 2.2k
docs(admin): restructure and update server-side encryption documentation #13754
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
base: master
Are you sure you want to change the base?
Conversation
Updated the server-side encryption documentation. - Reorganized and streamlined - Improved clarity throughout - Fixed a few outdated comments/notes - Additional reference links - Added context of "encryption" to help reader understand SSE in the context of other encryption solutions (both within Nextcloud and provided by their OS/etc) - Expanded/clarified key management modes and encryption methods a bit Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Added a tip for further encryption command examples and details. Signed-off-by: Josh <josh.t.richards@gmail.com>
Moved them all to the Encryption chapter rather than having them split between there and the general troubleshooting chapter. And cleaned up some grammar and typos. Signed-off-by: Josh <josh.t.richards@gmail.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
|
It would be cool if you could mention that |
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.
Thank you for working on this. Sorry, I don't have time to look at this at the moment.
@Toorero Can you clarify? The "version", in the context of SSE, is the encryption metadata version (not File Versions). The command just tries a bunch of different possible encryption versions. It doesn't touch file content or roll back to any prior file versions. |
|
@joshtrichards thanks for working on it. One suggestion from my side. We should mention that user keys have some draw backs which can easily irritate users. A example I use all the time: Think about a group share, the moment you share a file with a group Nextcloud will take the public keys of all group members and encrypt the file-key with it. With large groups this can have a (huge) performance impact. If a admin adds later additional users to the group they will see the file shared with the group but they will not be able to decrypt it because they where not part of the group when the file-key was encrypted. This can lead to a lot of confusion and 1st level support request like "i can't open a file shared with me", "nextcloud is broken", etc. |
This PR updates and improves the server-side encryption documentation.
Note: In-depth implementation details remain in the dedicated "Details" chapter, which is not affected by this PR.. Same goes for occ encryption section in the occ command reference chapter.
☑️ Resolves
Follow-up items:
🖼️ Screenshots
(works better if you open the screenshot image in a new tab then clicked on again to view it full-size)