freedomofpress / dangerzone

Take potentially dangerous PDFs, office documents, or images and convert them to safe PDFs
https://dangerzone.rocks/
GNU Affero General Public License v3.0
3.74k stars 172 forks source link

qubes: Do not close stderr #967

Closed apyrgio closed 3 weeks ago

apyrgio commented 1 month ago

Do not close stderr as part of the Qubes termination logic, since we need to read the debug logs. This shouldn't affect typical termination scenarios, since we expect our disposable qube to be either busy reading from stdin, or writing to stdout. If this is not the case, then forcefully killing the qrexec-client-vm process should unblock the qube.

With this change, the Qubes CI tests should pass once more on main.