gopass: Copy with gopass -c somesite does not work
Summary
Copying a regular password gopass -c somesite
outputs the following:
flag provided but not defined: -c
Steps To Reproduce
gopass -c somesite
NOTE:
gopass otp -c somesite
DOES work as expected.
As a workaround I’m using "gopass show somesite | head -n 1 | tr -d '\n' | pbcopy"
but this is just too ugly.
Expected behavior
normal operation
Environment
- OS: [e.g. Mac OS X 10.15.5 (19F101)]
- OS version: Darwin macbook20.local 19.5.0 Darwin Kernel Version 19.5.0: Tue May 26 20:41:44 PDT 2020; root:xnu-6153.121.2~2/RELEASE_X86_64 x86_64
- gopass Version: gopass 1.10.0-git+HEAD go1.14.2 darwin amd64
- Installation method:
go get -u -v github.com/gopasspw/gopass
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Reactions: 1
- Comments: 17 (8 by maintainers)
Unfortunately I’m not completely sure if we intentionally or accidentally dropped -c from the default action.
We’re currently trying to clearly define each command.
I thought I was losing my mind. I knew
gopass -c somesite
worked earlier this week.There is a PR “Re-introduce gopass -c #1572” in the pipe, not merged yet though. So it looks like
-c
is on it’s way back along with-u
.We did never properly define how each command should behave. Feel free to contribute to the command definitions in
docs/commands
.I also was wondering where did
gopass -c
command go. It is the most common command that I use.The
-c
flag to the default command (i.e.gopass entry
) was deprecated in de194f214fbee6bd12c6fd344109d4c78bbed77b.Please use
gopass show
.We probably need to improve the help output to make that clear.
As
-c
is most used command, why would you even consider an option requireing to tapshow
every time? Ridiculous decision. Second, cheat-sheets and demos illustrate usage of -c, and… it does not work.@daraul Ok, works for me 😉
I just checked the changelog and we did not mention this, indeed. Sorry. The reason is that this feature didn’t exist intentional - it somehow creeped in. That’s something we’re trying to fix by requiring documentation for future changes.
If anyone else if craving for this feature please open a PR detailing the intended behaviour in
docs/commands/gopass.md
and we can bring it back.At least document the change (if you haven’t, I haven’t checked myself), and provide some deprecation notice when people use the old command. At least then people have a chance to adjust, as the command simply disappearing can be jarring to some.