-
Notifications
You must be signed in to change notification settings - Fork 77
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
StorageException: Resumable upload is already complete. #666
Comments
Hi @n12c! Thanks for filing the issue. Do you have an example use case that causes this issue? |
I don't have a self-contained reproducer; this seems quite intermittent. Generally we just call I can try to write a reproducer but it would basically consist of uploading random objects using this API in a tight loop until (hopefully) encountering the same issue. |
Thanks! I'll try to reproduce the issue, it might be a retry an accidental retry when there shouldn't be one. |
Update: I just saw this happen without CSEK (no encryption of any kind, just the default Google managed keys). So, CSEK is not a factor. |
Hi |
Hi @richiesgr I'm working to release a fix today. Apologies for the delay. |
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly: - [x] Ensure the tests and linter pass - [x] Appropriate docs were updated (if necessary) Fixes #666 ☕️
Following #647, we no longer sometimes see:
This appeared in 1.113.4 but no longer appears in 1.113.6. Thanks for the fix.
Instead (in 1.113.6) we now sometimes see:
What do you think? More tweaks to #604 required?
Thanks for your time.
The text was updated successfully, but these errors were encountered: