Skip to content

Fix KeyError for long-running uploads - #336

Merged
Tooyosi merged 3 commits into
zooniverse:masterfrom
adammcmaster:fix/bearer-token-refresh-retry
Oct 15, 2026
Merged

Tooyosi merged 3 commits into
zooniverse:masterfrom
adammcmaster:fix/bearer-token-refresh-retry

Conversation

@adammcmaster

Copy link
Copy Markdown
Contributor

I keep getting KeyErrors during uploads with the CLI when the bearer token expires, because it's not being automatically refreshed correctly.

  • Adds a bearer-token lock for async/threaded uploads.
  • Validates token responses instead of blindly indexing access_token.
  • If token refresh fails, clears stale bearer/refresh token state, logs in again, and retries token acquisition once.

- Adds a bearer-token lock for async/threaded uploads.
- Validates token responses instead of blindly indexing access_token.
- If token refresh fails, clears stale bearer/refresh token state, logs in again, and retries token acquisition once.
@lcjohnso
lcjohnso requested a review from Tooyosi September 28, 2026 14:58
@Tooyosi
Tooyosi merged commit e8f337c into zooniverse:master Oct 15, 2026
2 checks passed
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