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

[cli] Different password when using -C #146

Closed
Rudloff opened this issue Jan 31, 2017 · 4 comments
Closed

[cli] Different password when using -C #146

Rudloff opened this issue Jan 31, 2017 · 4 comments
Labels
Milestone

Comments

@Rudloff
Copy link
Contributor

Rudloff commented Jan 31, 2017

With lesspass-cli 4.0.1, I get different results when using -C:

lesspass foo bar
master password: baz
=>~pof5X@YaVu4Qa
lesspass -C foo bar
master password: baz
p:8C_o'zA@4Gd1HX
@guillaumevincent
Copy link
Member

@Rudloff order matters

I use meow for parsing options and there is no booleanish option.

So lesspass -C foo bar set C option to foo.
For now you need to put -C at the end.

lesspass foo bar -C
master password: baz
Copied to clipboard
=>~pof5X@YaVu4Qa^C

I will try to fix lesspass-cli with another cli helper or create a PR to fix sindresorhus/meow#44

@Rudloff
Copy link
Contributor Author

Rudloff commented Jan 31, 2017

Oh right, sorry. I'm really used to having flags before arguments (like cp, chmod, etc.).

@guillaumevincent
Copy link
Member

@Rudloff don't be sorry, it's a bug and argument should work before and after

@guillaumevincent
Copy link
Member

guillaumevincent commented Feb 1, 2017

@Rudloff fixed in version 4.0.2

npm update -g lesspass-cli

lesspass -C foo bar
master password: baz
Copied to clipboard
=>~pof5X@YaVu4Qa

@guillaumevincent guillaumevincent modified the milestone: Harmony Feb 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants