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

[Feature]: Add refresh tokens for better Globus functionality #338

Open
forsyth2 opened this issue Jul 29, 2024 · 1 comment · May be fixed by #349
Open

[Feature]: Add refresh tokens for better Globus functionality #338

forsyth2 opened this issue Jul 29, 2024 · 1 comment · May be fixed by #349
Labels
semver: small improvement Small improvement (will increment patch version)

Comments

@forsyth2
Copy link
Collaborator

How will this affect the next version number?

New feature (increment MINOR version)

Is your feature request related to a problem?

zstash users may find themselves needing to re-authenticate into Globus every couple days, when they do not need to do this on the command line.

Describe the solution you'd like

https://globus-sdk-python.readthedocs.io/en/stable/examples/native_app.html#with-refresh-tokens

Describe alternatives you've considered

No response

Additional context

No response

@forsyth2 forsyth2 added the semver: small improvement Small improvement (will increment patch version) label Jul 29, 2024
@mahf708
Copy link
Contributor

mahf708 commented Jul 29, 2024

I think we likely want a slightly rework of the whole auth logic, it seems to be quite different from what they have on that page above. I would try to mimic it as closely as possible (to avoid running into bugs on their side, etc.)

@forsyth2 forsyth2 linked a pull request Sep 4, 2024 that will close this issue
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: small improvement Small improvement (will increment patch version)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants