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

all: use OAuth2 refresh token in storage config #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vvanpo
Copy link

@vvanpo vvanpo commented Dec 23, 2024

Dropbox removed support for long-lived access tokens a few years ago:
https://dropbox.tech/developers/migrating-app-permissions-and-access-tokens

This storage implementation thus required re-running the server setup
every 4 hours. By replacing the stored access token with a refresh
token, and using the oauth2 package implementation of an HTTP client
that automatically refreshes the access token upon expiry, we can once
again have long-lived access to the Dropbox API. Currently refresh
tokens do not have a default expiry and are not rotated upon access.

Dropbox removed support for long-lived access tokens a few years ago:
https://dropbox.tech/developers/migrating-app-permissions-and-access-tokens

This storage implementation thus required re-running the server setup
every 4 hours. By replacing the stored access token with a refresh
token, and using the `oauth2` package implementation of an HTTP client
that automatically refreshes the access token upon expiry, we can once
again have long-lived access to the Dropbox API. Currently refresh
tokens do not have a default expiry and are not rotated upon access.
@vvanpo
Copy link
Author

vvanpo commented Jan 8, 2025

@palmamartin or @adg not sure who is maintaining this repository, is anyone available to review?

@palmamartin
Copy link
Contributor

@vvanpo: Sorry but I haven't used Dropbox and Upspin in years... maybe @adg is a better fit for reviewing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants