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

Remove persistent attachment to sd-export-usb VM #320

Closed
eloquence opened this issue Nov 4, 2019 · 0 comments · Fixed by #323
Closed

Remove persistent attachment to sd-export-usb VM #320

eloquence opened this issue Nov 4, 2019 · 0 comments · Fixed by #323

Comments

@eloquence
Copy link
Member

eloquence commented Nov 4, 2019

In #259 we added support for exporting to USB drives which relies on a persistent attachment of a USB device to a disposable VM.

In practice we've encountered significant issues with this approach; notably, sys-usb will sometimes associate different device IDs with different storage devices attached to the same USB port (#307), and the device has to be plugged in at the time the VM starts.

We're therefore considering transitioning to an approach which relies on automatically attaching certain classes of devices (USB mass storage, printers) to the sd-export-usb VM. Even if that approach does not prove viable, the persistent attachment clearly isn't. Our fallback is to rely on the device manager in Qubes to handle the attachment manually.

For these reasons, we should remove the persistent attachment requirement from the code and configuration for the export VM.

@eloquence eloquence added this to Nominated for next sprint in SecureDrop Team Board Nov 5, 2019
@eloquence eloquence moved this from Nominated for next sprint to Current Sprint - 11/6-11/20 in SecureDrop Team Board Nov 6, 2019
@rmol rmol moved this from Current Sprint - 11/6-11/20 to Ready for Review in SecureDrop Team Board Nov 8, 2019
@emkll emkll removed this from Ready for Review in SecureDrop Team Board Nov 8, 2019
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 a pull request may close this issue.

1 participant