pure icon indicating copy to clipboard operation
pure copied to clipboard

Make sure GPG-agent does not mess up the tty

Open sheeviZi opened this issue 1 year ago • 3 comments

When gpg-agent is used as an ssh-agent and configured with pinentry-ncurses, it messes up the terminal immediately after you cd into a git repository with an ssh remote.

Fixes issue #366.

I can help you reproduce it if you ask.


IssueHunt Summary

Referenced issues

This pull request has been submitted to:


sheeviZi avatar Aug 10 '22 14:08 sheeviZi

@sheeviZi that looks very simple, but what is the intended effect of this change?

~After applying this patch, I don't have the problem of the mangled input anymore (hurray! 🏆), but I also don't see any pinentry prompt. This is perfectly fine with me, I'm just trying to understand what's going on under the hood.~

Sorry I spoke too fast. Looks like my gpg-agent wasn't set up properly while I was trying to reproduce. See review comment.

antoineco avatar Aug 11 '22 13:08 antoineco

@antoineco Thanks for your time.

sheeviZi avatar Aug 11 '22 16:08 sheeviZi

@sheeviZi a note about the bounty on this issue.

If you want to claim the reward, you will have to

  1. Log in at https://issuehunt.io/r/sindresorhus/pure/issues/366 using your GitHub account
  2. Click "Submit a Pull Request" on the right hand side of the page
  3. Paste the URL of this PR

antoineco avatar Aug 11 '22 16:08 antoineco

@sheeviZi I saw the request for review but I am only providing feedback as a user, not a maintainer. You will have to wait for a maintainer to get their eyes on this in order to have it merged (if accepted as a solution to the problem).

antoineco avatar Aug 17 '22 04:08 antoineco

@antoineco Are you ok with the wiki page?

sheeviZi avatar Aug 17 '22 09:08 sheeviZi

Yes I think that looks good, and it's useful information paired with the fix from this PR.

antoineco avatar Aug 17 '22 09:08 antoineco

@mafredri is this ok for merge?

sheeviZi avatar Aug 30 '22 11:08 sheeviZi

@sindresorhus can you please take a look in order to close the linked issue; Otherwise I am not able to claim the bounty on issuehunt.

sheeviZi avatar Sep 10 '22 19:09 sheeviZi

Thank you @sindresorhus for merging this PR. Just mention me with @sheeviZi if you need anything in the future.

sheeviZi avatar Sep 11 '22 11:09 sheeviZi