-
Notifications
You must be signed in to change notification settings - Fork 6
Description
Hey @landhb, remember me? I'm the maintainer of the keyring crate. I'm about to bring out keyring v4, now in alpha, and as part of that release I would like to move the keyutils credential store out of the keyring crate and into this crate (which was a dependency for it). My motivations for this are:
- I'm trying to get back to a very slim and simple keyring model, which just has one built-in credential store for each platform. (Keyring v3 introduced a requirement that clients think about what store they want to use, and that just confused them. They were a lot happier when keyring made that choice for them, so I'm trying to get back to that.)
- Keyring extensibility makes it super easy for people to bring their own stores, so there's no real advantage to having lots of stores built into the keyring crate itself. They can just use the keyring crate and this crate if they want to use the keyutils credential store.
I'm submitting a PR that adds support for use as a keyring credential store. The PR conditionalizes that support under a new "keystore" feature, so existing direct clients of this crate aren't forced into a fatter build if you release a new version that includes this PR.
Can you do me a favor and take a look and let me know if you'd be willing to accept this? I'd love to have this enhancement out when I do the final release of keyring v4.
Thanks!