-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Labels
GlobusGlobusGlobussemver: small improvementSmall improvement (will increment patch version)Small improvement (will increment patch version)
Description
How will this affect the next version number?
New feature (increment MINOR version)
Is your feature request related to a problem?
- The token file is only set up to store one token at a time. This causes the following problem: If a user uses
zstashto transfer to Globus Endpoint A and then to Globus Endpoint B, they will get aTransferAPIError, as described in case 1 of Update auth tests #397 (comment), unless they first delete the token file to start fresh (as in case 3 of that link) - Since
TOKEN_FILEis hard-coded asTOKEN_FILE = os.path.expanduser("~/.zstash_globus_tokens.json")inzstash/zstash/globus_utils.py, it is not currently possible to use different token files for different things (testing, for example, as noted in Update auth tests #397 (comment)).
Describe the solution you'd like
- Either allow the token file to store multiple tokens or allow the use of multiple token files. Either way, this will require either a way for
zstashto accurately determine which token to use. With inadequate information there, it may have to resort to looping through tokens to check if they work. - Allow the use of multiple token files or for users to specify which token file they want
zstashto check.
Describe alternatives you've considered
No response
Additional context
No response
Metadata
Metadata
Assignees
Labels
GlobusGlobusGlobussemver: small improvementSmall improvement (will increment patch version)Small improvement (will increment patch version)