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

Create New Qube tool: Vague error when putting a space in Qube name #9558

Closed
Geblaat opened this issue Nov 3, 2024 · 1 comment · Fixed by QubesOS/qubes-core-admin-client#317
Assignees
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: core C: manager/widget diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@Geblaat
Copy link

Geblaat commented Nov 3, 2024

How to file a helpful issue

Qubes OS release

4.2.3

Brief summary

The Create New Qube tool has succeeded the Create Qubes VM tool. The old tool did not allow a space in the name of a Qube name. It just doesn't enter a space at all when you press Space. The new tool does enter the space, but then gives a vague error when creating the Qube. I assume a space in a Qube name is still unsupported, but now this is unclear to new users.

Steps to reproduce

  1. Open Create New Qube
  2. Enter a name with a space in it
  3. Click Create

Expected behavior

  1. Throw an error that explains to the user a space is not allowed in a Qube name
  2. Or don't enter a space at all at step 2 when the user presses Space bar, like the old tool

Actual behavior

Error message:
Could not create qube
An error occurred: Got empty response from qubesd. See journalctl in dom0 for details

@Geblaat Geblaat added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels Nov 3, 2024
@alimirjamali
Copy link

alimirjamali commented Nov 3, 2024

The linked #9303 issue makes it clear. Every AdminAPI call from qubesadmin to qubesd with key=value pairs should assure that there is no space in values. Namely admin.pool.Add, admin.vm.Create and admin.vm.CreateInPool. Since the qubesd side uses .split(' ')

p.s.:
Interestingly QubesBase.add_pool is using \n to join the key=value pairs payload and qubesd side is using .splitlines().

alimirjamali added a commit to alimirjamali/qubes-core-admin-client that referenced this issue Nov 4, 2024
alimirjamali added a commit to alimirjamali/qubes-core-admin-client that referenced this issue Nov 4, 2024
@andrewdavidwong andrewdavidwong added C: core C: manager/widget diagnosed Technical diagnosis has been performed (see issue comments). pr submitted A pull request has been submitted for this issue. affects-4.2 This issue affects Qubes OS 4.2. labels Nov 5, 2024
marmarek pushed a commit to QubesOS/qubes-core-admin-client that referenced this issue Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: core C: manager/widget diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants