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

Add Gopass driver #53

Closed
wants to merge 8 commits into from
Closed

Add Gopass driver #53

wants to merge 8 commits into from

Conversation

treussart
Copy link

No description provided.

@jkroepke
Copy link
Owner

jkroepke commented Nov 1, 2020

gopass supports yaml format directly.

this a vault like wrapper required for gopass?

@treussart
Copy link
Author

treussart commented Nov 1, 2020

Hi,
As far as I know, Gopass does not support the yaml format directly. https://github.com/gopasspw/gopass
Indeed, it's a vault like wrapper for Gopass, I'm using Gopass as my password storage.

@jkroepke
Copy link
Owner

jkroepke commented Nov 1, 2020

What about this?

https://github.com/gopasspw/gopass/blob/master/docs/features.md#directly-edit-structured-secrets-aka-yaml-support

or MIME Secrets (successor of yaml format)?

https://github.com/gopasspw/gopass/blob/master/docs/features.md#mime-secrets

Did you try that?

I guess it's might be better to have a sops like handling like the vault like handing. It will reduce a lot of shell code.

@treussart
Copy link
Author

Hi,
MIME secrets are not a good choice for helm secrets, I don't want to have multiple secrets in a file, just context (user, url, description, ...), no more.
I don't like sops because I like to have my secrets centralized in a repo and not in YAML files all over the place.

@jkroepke
Copy link
Owner

jkroepke commented Dec 23, 2020

Hi @treussart ,

in #63 I moved the logic to out of the vault driver. Now it's possible to re-use the logic by other drivers.

Could you take a look at a9fbe4a if this fits for you? It should reduce the amount of new code compared to this PR.

@treussart
Copy link
Author

Hard to say, we would have to test with gopass and helmfile.
We are now using summon for our project : https://github.com/cyberark/summon

@jkroepke
Copy link
Owner

All right, thanks for the feedback.

@jkroepke jkroepke closed this Dec 23, 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.

2 participants