-
Notifications
You must be signed in to change notification settings - Fork 94
Add KeyChain support #813
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
base: main
Are you sure you want to change the base?
Add KeyChain support #813
Conversation
Bumps [cachix/cachix-action](https://github.com/cachix/cachix-action) from 15 to 16. - [Release notes](https://github.com/cachix/cachix-action/releases) - [Commits](cachix/cachix-action@v15...v16) --- updated-dependencies: - dependency-name: cachix/cachix-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
empty password id is confusing
LGTM, please ping me in a day or two if doesn't get any response. |
@dpc any news? |
@@ -10,6 +10,8 @@ use term::{ | |||
StderrTerminal, StdoutTerminal, | |||
}; | |||
|
|||
use crate::creds; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this needs a cfg?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, this needs cfg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've fixed the problem. Sorry for the delay
4f06cae
to
1c146b1
Compare
This PR adds mac keychain basic support - without unlocked session, just ordinary get/set.
It needed to mode safely (and friendly!) store and retrieve passphrases then from env or shell.
Checklist:
cargo +nightly fmt --all
CHANGELOG.md
if applicable