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

Allow dots in qube names #8199

Open
marmarek opened this issue May 12, 2023 · 2 comments · May be fixed by QubesOS/qubes-core-admin#540
Open

Allow dots in qube names #8199

marmarek opened this issue May 12, 2023 · 2 comments · May be fixed by QubesOS/qubes-core-admin#540
Labels
C: core P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality.

Comments

@marmarek
Copy link
Member

marmarek commented May 12, 2023

How to file a helpful issue

The problem you're addressing (if any)

Currently allowed characters in qube names are: a-zA-Z0-9_-. Before Qubes 4.0, dot used to be allowed and that was useful thing.

The solution you'd like

Allow dots in qube names again.

The value to a user, and who that user might be

Easier to apply own naming convention to qube names.

Related: #2422

@marmarek marmarek added T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality. C: core P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. labels May 12, 2023
@marmarek marmarek added this to the Release 4.2 milestone May 12, 2023
@DemiMarie
Copy link

Should a leading . be allowed?

@marmarek
Copy link
Member Author

Probably no.

DemiMarie added a commit to DemiMarie/qubes-gui-daemon that referenced this issue Jun 2, 2023
DemiMarie added a commit to DemiMarie/qubes-core-admin that referenced this issue Jun 2, 2023
@DemiMarie DemiMarie linked a pull request Jun 2, 2023 that will close this issue
DemiMarie added a commit to DemiMarie/qubes-core-admin that referenced this issue Jun 2, 2023
DemiMarie added a commit to DemiMarie/qubes-gui-daemon that referenced this issue Jun 3, 2023
This allows periods in qube names by using the function from
libqubes-pure to validate them instead of the built-in version.

Part of QubesOS/qubes-issues#8199.
DemiMarie added a commit to DemiMarie/qubes-gui-daemon that referenced this issue Jun 3, 2023
This uses the validation function from libqubes-pure (which permits
periods) instead of the old built-in version (which does not).

Part of QubesOS/qubes-issues#8199.
@andrewdavidwong andrewdavidwong removed this from the Release 4.2 milestone Aug 13, 2023
DemiMarie added a commit to DemiMarie/qubes-gui-daemon that referenced this issue Aug 14, 2023
This uses the validation function from libqubes-pure (which permits
periods) instead of the old built-in version (which does not).

Part of QubesOS/qubes-issues#8199.
DemiMarie added a commit to DemiMarie/qubes-gui-daemon that referenced this issue Aug 20, 2023
This uses the validation function from libqubes-pure (which permits
periods) instead of the old built-in version (which does not).

Part of QubesOS/qubes-issues#8199.
DemiMarie added a commit to DemiMarie/qubes-gui-daemon that referenced this issue Sep 17, 2023
This uses the validation function from libqubes-pure (which permits
periods) instead of the old built-in version (which does not).

Part of QubesOS/qubes-issues#8199.

This reverts commit 1ba8051.
DemiMarie added a commit to DemiMarie/qubes-gui-daemon that referenced this issue Sep 18, 2023
This uses the validation function from libqubes-pure (which permits
periods) instead of the old built-in version (which does not).

Part of QubesOS/qubes-issues#8199.

This reverts commit 1ba8051.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: core P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: enhancement Type: enhancement. A new feature that does not yet exist or improvement of existing functionality.
Projects
Status: In review
Development

Successfully merging a pull request may close this issue.

3 participants