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

#179 fix path for latest-version v2 kv delete #213

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

Conversation

sfc-gh-dkarp
Copy link

@sfc-gh-dkarp sfc-gh-dkarp commented Feb 11, 2020

Current code attempts a DELETE on the metadata path to do kv delete of the latest version,
but Vault expects a DELETE on the data path.

From the Vault API docs (https://www.vaultproject.io/api/secret/kv/kv-v2.html#delete-latest-version-of-secret):

»Delete Latest Version of Secret

This endpoint issues a soft delete of the secret's latest version at the specified
location. This marks the version as deleted and will stop it from being returned
from reads, but the underlying data will not be removed. A delete can be undone
using the undelete path.

Method Path
DELETE /secret/data/:path

Current code attempts a DELETE on the metadta path, but Vault expects
a DELETE on the data path.

From the Vault API docs (https://www.vaultproject.io/api/secret/kv/kv-v2.html#delete-latest-version-of-secret):

»Delete Latest Version of Secret

This endpoint issues a soft delete of the secret's latest version at the specified
location. This marks the version as deleted and will stop it from being returned
from reads, but the underlying data will not be removed. A delete can be undone
using the undelete path.

Method	Path
DELETE	/secret/data/:path
@sfc-gh-dkarp sfc-gh-dkarp changed the title issue-179 fix path for latest-version v2 kv delete #179 fix path for latest-version v2 kv delete Feb 11, 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.

1 participant