-
-
Notifications
You must be signed in to change notification settings - Fork 117
Deserialize device from qrexec arg properly #759
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
Conversation
26aa02b to
c15b16d
Compare
Avoid ':', '*' and '?' in the argument, as those are not allowed over qrexec. Fixes QubesOS/qubes-issues#10040
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025122104-4.3&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025111104-4.3&flavor=update
Failed tests57 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/158999#dependencies 14 fixed
Unstable testsDetailsPerformance TestsPerformance degradation:32 performance degradations
Remaining performance tests:119 tests
|
|
PipelineRetry |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #759 +/- ##
==========================================
- Coverage 70.74% 70.25% -0.49%
==========================================
Files 61 61
Lines 13809 14074 +265
==========================================
+ Hits 9769 9888 +119
- Misses 4040 4186 +146
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Avoid ':' and '*' in the argument, as those are not allowed over qrexec.
Fixes QubesOS/qubes-issues#10040