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

vault-cli installed as vault, causes collision with official hashicorp vault cli #158

Closed
jason-riddle opened this issue Apr 8, 2020 · 5 comments · Fixed by #194
Closed

Comments

@jason-riddle
Copy link

The documentation says to install vault-cli with pip install vault-cli. However, the script is available as vault and because ~/.local/bin comes first in my path, it is overriding the vault CLI binary, making it impossible to use both tools. Steps to reproduce (using pipx):

# pipx install vault-cli

# pipx list
venvs are in ~/.local/pipx/venvs
apps are exposed on your $PATH at ~/.local/bin
   package vault-cli 1.2.0, Python 3.8.2
    - vault

# ls -al ~/.local/bin
total 0
drwxr-xr-x  3 user  staff   96 Apr  8 10:51 .
drwx------  5 user  staff  160 Apr  8 10:51 ..
lrwxr-xr-x  1 user  staff   51 Apr  8 10:51 vault -> /Users/user/.local/pipx/venvs/vault-cli/bin/vault

Why is this the case? Also, can this be reverted?

@ewjoachim
Copy link
Contributor

ewjoachim commented Apr 9, 2020

Hi, thanks for the issue. vault was the original name of the command and only very recently was it changed to vault-cli (#89) (EDIT I just realized the doc describes master, but I haven't released it yet into a proper version ! Will do this asap!), but we kept the old name for compatibility's sake. That's for the why. We plan to remove the old alias someday but doing it right now would likely break all existing users :/

Now to the "can this be reverted", I'm afraid the best I can suggest is to

  • Use the official vault command by full path
  • rm /.local/bin/vault which will leave you with vault-cli for this tool and vault for the official tool.

I'll leave the ticket open until I:

  • Find a better solution
  • Release a new version of vault-cli that supports vault-cli as the command name, which should be in a few minutes

@ewjoachim
Copy link
Contributor

@abebars
Copy link

abebars commented May 31, 2021

any updates on this issue? I think users can be notified of the breaking change?

@ewjoachim
Copy link
Contributor

Yep, I've started making sure our internal users won't suffer from the change, and I should be able to make a new release once that is done.

@ewjoachim
Copy link
Contributor

Ok, we should be good to proceed. Do you want to do the PR @abebars ?

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

Successfully merging a pull request may close this issue.

3 participants