Skip to content
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

External storage global OAuth app ID #4442

Closed
zauguin opened this issue Apr 22, 2017 · 3 comments
Closed

External storage global OAuth app ID #4442

zauguin opened this issue Apr 22, 2017 · 3 comments

Comments

@zauguin
Copy link

zauguin commented Apr 22, 2017

When users are allowed to configure external storage for example with a Dropbox backend, each user has to enter app credentials.
It would be very useful to add a global configurarion value for server wide app credentials, so that every user can individually login with their own Dropbox accounts, but reuse the same global app ID/secret.
The same applies to Google Drive.

@MorrisJobke
Copy link
Member

It would be very useful to add a global configurarion value for server wide app credentials, so that every user can individually login with their own Dropbox accounts, but reuse the same global app ID/secret.
The same applies to Google Drive.

This would not really work well, because on both Google Drive and Dropbox are quite low rate limits and quite fast the external storage would not be read-/writable anymore.

cc @icewind1991

@zauguin
Copy link
Author

zauguin commented Apr 23, 2017

@MorrisJobke According to the documentation, on Dropbox at least the rate limiting only applies on a per-user basis, so it shouldn't have different effects if we share an app ID for multiple users.
Or do you refer to the user count limit for non-production apps?

@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@skjnldsv skjnldsv added the 1. to develop Accepted and waiting to be taken care of label Jun 12, 2019
@ghost ghost removed the stale Ticket or PR with no recent activity label Jun 12, 2019
@szaimen
Copy link
Contributor

szaimen commented May 21, 2021

I am going to close this since there doesn't seem to be a lot of interest (no upvotes) and no respond since around 2 years. Please reopen if you still want to implement this.

@szaimen szaimen closed this as completed May 21, 2021
@szaimen szaimen added Nice to have and removed 1. to develop Accepted and waiting to be taken care of labels Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants