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

Rudimentary support for KV version 2 #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Rudimentary support for KV version 2 #35

wants to merge 1 commit into from

Conversation

broamski
Copy link
Collaborator

  • Adds rudimentary support for KV version 2 secret fetching
  • Update some upstream packages via go mod tidy; go mod vendor
  • NEW Tests for KV v2 fetching backed by a full-featured test cluster

The vault go api client doesn't discern between kv 1 and kv 2 as the vault binary is responsible for assembling the proper api paths to be called, so we must rely on DAYTONA consumers to do the same.

For example:
vault kv list secret/whatever is actually performing a list on secret/metadata/whatever. So a KV version 2 path walk in DAYTONA would be supplied as:
VAULT_SECRETS_MANY=secret/metadata/stuff, while a single secret is defined as VAULT_SECRET_SINGLE=secret/data/mysecret

@broamski broamski requested a review from robison March 11, 2020 22:45
@broamski broamski added this to the 1.1.1 milestone Mar 11, 2020
- Update some upstream packages via go mod tidy; go mod vendor
@broamski broamski linked an issue Mar 12, 2020 that may be closed by this pull request
@broamski broamski modified the milestones: 1.1.1, 1.1.2 Mar 25, 2020
@broamski broamski changed the base branch from 1.1.1 to master March 25, 2020 21:37
@broamski broamski added the notready Don't merge yet label Mar 25, 2020
@broamski broamski modified the milestones: 1.1.X, 2.0.0 Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notready Don't merge yet
Development

Successfully merging this pull request may close these issues.

Can't fetching secrets
1 participant