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)

Commits related to this issue

Most upvoted comments

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 tap show 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.