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

Support uuid syntax #2

Closed
ben-grande opened this issue Mar 15, 2024 · 1 comment
Closed

Support uuid syntax #2

ben-grande opened this issue Mar 15, 2024 · 1 comment

Comments

@ben-grande
Copy link
Owner

After the following PR is merged:

Add support for uuid: syntax. It can be specified in the source, destination, target_uuid=uuid: and can be of @dispvm:uuid: in the previous fields.

The UUID is generated from Python's uuid library, as seen in qubes-core-admin/qubes/vm/init.py respects RFC 4122.

It is a 32 chars string without couting the - hyphens (+4=36). Composed with of 00000000-0000-0000-0000-000000000000 (8+4+4+4+12), being hexadecimal strings, [0-9a-fA-F]. Note that uppercase letters depends on the response of QubesOS/qubes-core-qrexec#135 (comment).

@ben-grande
Copy link
Owner Author

I asked for strict validation of the UUID syntax, I don't see if it blocked upper-case letters, although it would be easier at least with python on the parser to use .lower() before evaluation.

As lower-case was preferred, I am enforcing usage of lower-case on the policy.

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

No branches or pull requests

1 participant