-
Notifications
You must be signed in to change notification settings - Fork 158
Open
Labels
Type:EnhancementType:MaintenanceE.g. technical debt, packaging, etc.E.g. technical debt, packaging, etc.
Description
With #1191 we're getting one step closer to being able to remove the need to client side created signed urls.
Problem
- The
signing-keyendpoint is the last remaining endpoint offered by theocsservice in OpenCloud (not to be confused with theocsservice in reva). Removing it would mean we could delete the whole service - The concept of client side signed urls has a few drawbacks
- The secret used for signing the url leaks the server
- The client is able to generate any signed url it likes
- The client is in control of the lifetime of the signature
- per-user secrets need to be managed and stored on the server side
Solution
Switch all remaining cases where signed urls are needed to be able to download things to use server generated signed urls. Currently client signed urls are used by web at least in these cases:
- Downloading files using the Download button in the web ui. This could be solved by just using the oc:downloadURL property added by proxy(sign_url_auth): Allow to verify server signed URLs #1191
- Downloading a directory using the archiver
- Downloading versions of files Deprecate and remove
/cloud/user/signing-keyendpoint #1197 (comment) - ???
The other clients (desktop, ios, android) don't seem to be using the signing-key anywhere.
Metadata
Metadata
Assignees
Labels
Type:EnhancementType:MaintenanceE.g. technical debt, packaging, etc.E.g. technical debt, packaging, etc.