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

FEATURE: allow external password caching #46

Merged

Conversation

skurfuerst
Copy link
Contributor

needs gopasspw/gopass#1469 to be merged first

@skurfuerst
Copy link
Contributor Author

skurfuerst commented Jul 23, 2020

Hey,
I would be interested on your thoughts on this one :)

Thanks and keep up the great work,
Sebastian

@fheinze
Copy link

fheinze commented Sep 4, 2020

Hey,
I would be interested on your thoughts on this one :)

Thanks and keep up the great work,
Sebastian

We are currently using the fork as it makes working with the YubiKey really fun ;) as we do not have to enter the PIN all the time and utilize the keychain of MacOS. Would be really nice to use the upstream repo directly. Have you considered merging the Pull request?

gopasspw/gopass#1469 seems to be merged ;)

@FiloSottile
Copy link
Owner

Oh, this is neat! Yeah, I like it, the PIN still protects the YubiKey when separated from the machine it's saved on, and if that machine is compromised the PIN can be keylogged anyway.

@FiloSottile FiloSottile force-pushed the allow-keychain-password-caching branch from e8fa6dc to 7e7fb4a Compare September 5, 2020 15:49
@FiloSottile FiloSottile marked this pull request as ready for review September 5, 2020 15:51
@FiloSottile FiloSottile merged commit 4998d8c into FiloSottile:master Sep 5, 2020
@philandstuff philandstuff mentioned this pull request Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants