-
Notifications
You must be signed in to change notification settings - Fork 27
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
Web access to files #14
Comments
a UI for listing and revoking access tokens is indeed missing from reStore. for the files themselves you can use export.5apps.com, remotestorage-browser.5apps.com, or a similar app. |
Sorry I didn't have time to look at this when you posted it. I think access to files is out of scope, since that can be done using a client application with the Administering authorizations is certainly necessary but I think it should possibly be a separate application. Since that would involve using sessions, it would complicate the security considerations of reStore itself, which currently does not use sessions anywhere. And, whereas reStore is implemented on top of the bare Node Such an application could be built on top of the reStore backend storage engine abstraction as it stands, I believe. Are there any methods we'd need to add to that interface to facilitate an app like this? |
It'd be nice if reStore recorded a last-used date on authorization, so a potential interface could automatically revoke old tokens. I think this would be a great UI in addition to grouping authorizations by redirect URI. |
However, I don't think separating the administration interface from the storage engine is a good idea from a user's perspective. Perhaps I am misunderstanding this, but this new application would probably run on a new port? I suspect that would complicate matters for people who run restore behind a reverse proxy (eg myself) |
The admin interface would have to run on a separate port, or if on the same port then at least without CORS headers, to avoid exposing its cookie to applications on other origins, which are allowed to access the API but not the admin interface. |
Is there a security problem in just sending different/no CORS headers for the OAuth and admin paths? |
I think it should be ok, so then you don't need two separate ports! :) |
It would be nice if we could login in into restore and look at
The text was updated successfully, but these errors were encountered: