-
Notifications
You must be signed in to change notification settings - Fork 35
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
MissingIdentityError on token refresh #1050
Comments
Hi there! I think you've uncovered a bug in the implementation. Every login flow needs to have the |
One minor note on this, as we're working on it right now. I was incorrect about the cause -- there isn't a likely case in which the We have a series of changes to apply, which will make it simple for the two cases to be distinguished, and will add the appropriate handling for token refreshes. |
Just checking in and it seems like this will be fixed by #1055 in the next release, correct? For my own planning, is there an expected date for the next release? |
Yep, this was fixed by #1055. We're expecting to cut a release this week (no specific date within the week as of yet) which includes this bugfix. |
Excellent! Thank you all for your help and the quick fix. |
Of course, thanks for reaching out when you did! |
Last week, I transitioned to using the experimental
UserApp
class withJSONTokenStorage
:This was working pretty consistently in my tests, so I set up some script runs on timers to do some transfers. The first of these timed script executions completed successfully, but after a while, everything started failing with a
MissingIdentityError
at the point that I test the login withoperation_lc
:After debugging this a bit, it seems that the token has expired and the app is trying to refresh the token, but the new token for the
transfer.api.globus.org
resource server does not include theidentity_id
field. Despite the suggestion in the exception message, I cannot see how to add theopenid
scope to the transfer client scopes (my attempts to do so produce anUnmetScopeRequirementsError
).The only thing I can do to get this working again is to delete my token store
.json
file.app.login(force=True)
does force me to login again, but does not fix theMissingIdentityError
. I do understand that I'm using an experimental API.The text was updated successfully, but these errors were encountered: