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 Fail: Multiple drives detected returns generic error #1021

Closed
ninavizz opened this issue Mar 26, 2020 · 3 comments · Fixed by #1777
Closed

Export Fail: Multiple drives detected returns generic error #1021

ninavizz opened this issue Mar 26, 2020 · 3 comments · Fixed by #1777
Labels
bug Something isn't working

Comments

@ninavizz
Copy link
Member

Description

As a Journalist user, I go to click "Export" on a file. I have 2 drives plugged into my laptop, one correctly provisioned and one not. One of two scenarios happens:

  1. If this is my first time trying that, I'll go to the "Insert Encrypted Drive" screen, click Continue, and then the below screen is returned. Or...
  2. If this is my second time trying that, I'll simply get this screen immediately after clicking "Export."

image

STR

  1. On a non-prod machine with the 25 March nightly running...
  2. Go to Export a file with 2 drives attached—one correctly encrypted, the other, not
  3. Continue until the first Error appears
  4. Try it again

Expected

There should be 2 distinct error messages for this screen...

  1. (speak to too many drives on the sd-devices VM) Something like... "Too many devices detected. Unplug whichever device you do not want to export your file to."
  2. (speak to incorrectly formatted drive) "Either that drive is not correctly encrypted or there is something else wrong with it." NOTE: Current error text does not include the word "correctly," which I feel is important—and shd maybe open a different issue for?
@ninavizz ninavizz added bug Something isn't working needs/reproducing labels Mar 26, 2020
@eloquence
Copy link
Member

eloquence commented Mar 26, 2020

This is a known issue until freedomofpress/securedrop-export#25 is fixed. See also discussion in #1736.

@ninavizz
Copy link
Member Author

freedomofpress/securedrop-export#25 speaks only to multiple luks-encrypted devices. I'm a Mac user, so have multiple "encrypted devices" tooting around.

If more than 1 device is detected, it was discussed a while ago seeing a simple error message thrown to the user, speaking to that. After reading through the detailed comments in freedomofpress/securedrop-export#25 and #1736 I'm still not understanding why a simple "Too many devices, pick one" message cannot be displayed.

My read on the comments is that a much smarter solution is being targeted, while overlooking the simpler intermediary option of the "Too many devices, pick one and unplug the others" option.

@eloquence
Copy link
Member

Not at all, I agree that having an error message for this case is a useful first iteration, freedomofpress/securedrop-export#25 describes the status code we need in order to do so (it may need re-scoping to also handle the case of non-LUKS block devices being attached).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants