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.

Sign up for GitHub

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

Jump to bottom

Vaulted doesn't properly show vaults with "/" in their name #196

Open
Mythra opened this issue Oct 5, 2020 · 0 comments · May be fixed by #197
Open

Vaulted doesn't properly show vaults with "/" in their name #196

Mythra opened this issue Oct 5, 2020 · 0 comments · May be fixed by #197

Comments

@Mythra
Copy link

Mythra commented Oct 5, 2020

Attempting to create a vault with / in it's name results in an error normally:

$ vaulted cp tetrate-hub tetrate/hub
Vault 'tetrate-hub'
   Password: 
Vault 'tetrate/hub'
   New password: 
   Confirm password: 
open /home/cynthia/.local/share/vaulted/tetrate/hub: no such file or directory

If you manually go ahead and create the directory everything seems to works fine:

$ mkdir -p /home/cynthia/.local/share/vaulted/tetrate/
$ vaulted cp tetrate-hub tetrate/hub
Vault 'tetrate-hub'
   Password: 
Vault 'tetrate/hub'
   New password: 
   Confirm password: 
$

Although it doesn't show up in vaulted ls:

$ vaulted ls
bs-root
cncf-envoy
tetrate-hub
$

However it is still possible to manually use it:

vaulted -n tetrate/hub
Vault 'tetrate/hub'
   Password: 
   MFA token: [snipped]
tetrate/hub — expires: 7 Oct 2020 04:56 UTC (35h59m58s remaining)

Would you be open to a PR allowing vaulted cp (and other creation mechanisms) to support creating directories like this, and letting vaulted ls show this? Or is this something that you don't want to support, and I shouldn't hope for?
@Mythra Mythra changed the title Vaulted doesn't properly create/show vaults with "/" in their name Vaulted doesn't properly show vaults with "/" in their name Oct 5, 2020
This was referenced Oct 5, 2020
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

support / inside of vault names Mythra/vaulted
1 participant
@Mythra