8000 Add KeyChain support by boozook · Pull Request #813 · crev-dev/cargo-crev · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

boozook
Copy link
Contributor
@boozook boozook commented Mar 24, 2025

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
  • Modify CHANGELOG.md if applicable

dependabot bot and others added 3 commits March 11, 2025 04:19
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>
Copy link
@kana-sama kana-sama left a 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

@dpc
Copy link
Collaborator
dpc commented Mar 24, 2025

LGTM, please ping me in a day or two if doesn't get any response.

@eirnym
Copy link
eirnym commented Apr 7, 2025

@dpc any news?

@@ -10,6 +10,8 @@ use term::{
StderrTerminal, StdoutTerminal,
};

use crate::creds;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs a cfg?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, this needs cfg

Copy link

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

@eirnym eirnym force-pushed the mac-creds-keychain branch from 4f06cae to 1c146b1 Compare April 27, 2025 16:07
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.

6 participants
0