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

Export does not handle empty blobs response #8404

Closed
1 task done
charlag opened this issue Jan 28, 2025 · 0 comments · Fixed by #8407
Closed
1 task done

Export does not handle empty blobs response #8404

charlag opened this issue Jan 28, 2025 · 0 comments · Fixed by #8407
Assignees
Labels
dev bug unpublished bugs, found during our development/test cycle (excluded from release notes) state:tested We tested it and are about to release it

Comments

@charlag
Copy link
Contributor

charlag commented Jan 28, 2025

When the blob is not found the get multiple for blobs will return just 0 0 0 0 (as in: 0 blobs returned). Currently we throw an error but this is a valid response.

Feedback v264.250127.0 - Error - PAID - linux
Client: linux
Type: PAID
Tutanota version: 264.250127.0
Timestamp (UTC): Tue, 28 Jan 2025 12:52:39 GMT
User agent:
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) tutanota-desktop-debug/264.250127.0 Chrome/132.0.6834.83 Electron/34.0.1 Safari/537.36
Error
Error message: Invalid blob count: 0
Stacktrace:
Error: Invalid blob count: 0
at parseMultipleBlobsResponse (asset://app/BlobFacade-chunk.js:319:28)
at BlobFacade.downloadBlobsOfOneArchive (asset://app/BlobFacade-chunk.js:276:10)
at async BlobFacade.downloadAndDecryptBlobsOfMultipleInstances (asset://app/BlobFacade-chunk.js:138:41)
at async MailExportTokenFacade.loadWithToken (asset://app/MailExportTokenFacade-chunk.js:33:11)
at async MailExportFacade.loadAttachmentData (asset://app/MailExportFacade-chunk.js:45:21)

Test notes

  • Export ivkfortest10 on staging system
@charlag charlag added the dev bug unpublished bugs, found during our development/test cycle (excluded from release notes) label Jan 28, 2025
@charlag charlag added this to the Mailbox Export milestone Jan 28, 2025
charlag added a commit that referenced this issue Jan 28, 2025
@hrb-hub hrb-hub self-assigned this Jan 29, 2025
@hrb-hub hrb-hub added the state:tested We tested it and are about to release it label Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev bug unpublished bugs, found during our development/test cycle (excluded from release notes) state:tested We tested it and are about to release it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants