Skip to content

feat: Bitwarden Secrets provider#513

Merged
zhaque44 merged 1 commit intohelmfile:mainfrom
carnei-ro:feat/bitwarden-secrets-manager
Sep 6, 2024
Merged

feat: Bitwarden Secrets provider#513
zhaque44 merged 1 commit intohelmfile:mainfrom
carnei-ro:feat/bitwarden-secrets-manager

Conversation

@carnei-ro
Copy link
Contributor

adds Bitwarden Secrets
closes #427

This provider uses the Bitwarden SDK Go, which needs GCO enabled and musl-tools to compile (linux), check this doc.

Use ref+bws://<PROJECT_NAME>/<SECRET_NAME> to fetch the secret value.

$ make build
CGO_ENABLED=1 CC=clang CXX=clang++ go build -ldflags '-X main.version=v0.37.3-56-g8a1c52c-dirty -X main.commit=8a1c52c880fe3a40e0c65035b6ab5f8e742b7534' -o bin/vals ./cmd/vals

$ cat myyaml.yaml 
baz:
  mykey:
    simple_secret: ref+bws://vals-test/fooKey
    yaml: |
      ref+bws://vals-test/myYAML
    parsing_yaml_secret: ref+bws://vals-test/myYAML#baz

$ cat myyaml.yaml | bin/vals eval
baz:
  mykey:
    parsing_yaml_secret: qux
    simple_secret: myValue
    yaml: |
      foo: bar
      baz: qux

$ env | grep BWS_
BWS_ORGANIZATION_ID=d2f88df6-REDA-CTED-0000-000000000000
BWS_ACCESS_TOKEN=0.b5a15cb4-REDA-CTED-0000-000000000000.abcdefghijklmnopqrstuvwxyzABCD:aSBsb3ZlIHZhbHMgISAhCg==

The test is passing:

$ go test -run TestValues_BitwardenSecrets_String .
ok      github.com/helmfile/vals        7.151s

image

@carnei-ro carnei-ro force-pushed the feat/bitwarden-secrets-manager branch 7 times, most recently from bbdc981 to c1f6e7a Compare September 6, 2024 18:17
@carnei-ro carnei-ro force-pushed the feat/bitwarden-secrets-manager branch 2 times, most recently from d69012f to bb7b813 Compare September 6, 2024 19:29
Signed-off-by: Leandro Carneiro <leandro@carnei.ro>
@carnei-ro carnei-ro force-pushed the feat/bitwarden-secrets-manager branch from fd5e146 to 2fea060 Compare September 6, 2024 19:45
@zhaque44 zhaque44 self-assigned this Sep 6, 2024
@zhaque44 zhaque44 merged commit 5378cfa into helmfile:main Sep 6, 2024
@carnei-ro carnei-ro deleted the feat/bitwarden-secrets-manager branch September 6, 2024 20:23
@yxxhero
Copy link
Member

yxxhero commented Sep 7, 2024

@zhaque44 this PR broken many build process.

zhaque44 added a commit to zhaque44/vals that referenced this pull request Sep 7, 2024
…-secrets-manager"

This reverts commit 5378cfa, reversing
changes made to 35d76ca.
zhaque44 added a commit to zhaque44/vals that referenced this pull request Sep 7, 2024
…-secrets-manager"

This reverts commit 5378cfa, reversing
changes made to 35d76ca.

Signed-off-by: zhaque44 <haque.zubair@gmail.com>
yxxhero pushed a commit that referenced this pull request Sep 7, 2024
…-manager" (#515)

This reverts commit 5378cfa, reversing
changes made to 35d76ca.

Signed-off-by: zhaque44 <haque.zubair@gmail.com>
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.

Feature: support Bitwarden Secrets Manager

3 participants