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

Retrying a failed download from a SecureDrop results in a 4k empty file being saved to disk #6713

Open
nathandyer opened this issue Dec 22, 2022 · 1 comment

Comments

@nathandyer
Copy link
Contributor

Description

After a download from a SecureDrop fails when accessed from a Journalist Workstation, if you click the 'Retry' button, it "completes" the download by saving a 4kb empty text file, rather than the original submission.

Steps to Reproduce

  1. Fill the local filesystem with enough data to nearly be out of storage space.
  2. Download a relatively large submission from a SecureDrop instance.
  3. The download should fail once the filesystem runs out of space.
  4. Make some more room on the disk.
  5. Click "Retry" on the download from inside Tor browser.
  6. An empty file will be downloaded.

Expected Behavior

Clicking "Retry" downloads the original submission.

Actual Behavior

Clicking "Retry" downloads an empty file.

@eloquence
Copy link
Member

Per testing at least with the dev env this appears to be limited to downloading collections (even if said collection only contains one file), not individual files from the source page:

Peek 2022-12-22 13-42

I'm not sure client-side retry on collection downloads is viable since those are compiled via a POST request and subsequent server-side generation of a ZIP response.

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

No branches or pull requests

2 participants