peopledoc / vault-cli

A configurable command-line interface tool (and python library) to interact with Hashicorp Vault
https://vault-cli.readthedocs.io/
Other
81 stars 21 forks source link

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

Closed jason-riddle closed 3 years ago

jason-riddle commented 4 years ago

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 commented 4 years ago

Hi, thanks for the issue. vault was the original name of the command and only very recently was it changed to vault-cli (https://github.com/peopledoc/vault-cli/issues/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

I'll leave the ticket open until I:

ewjoachim commented 4 years ago

(https://pypi.org/project/vault-cli/1.3.0/)

abebars commented 3 years ago

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

ewjoachim commented 3 years ago

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 commented 3 years ago

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