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.

Sign up for GitHub

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

Jump to bottom

kv2/metadata: ReadSecretPathsAsync allows empty path value to list all secrets on the mountPoint #337

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

kv2/metadata: ReadSecretPathsAsync allows empty path value to list all secrets on the mountPoint #337

wants to merge 3 commits into from

Commits on Nov 22, 2023

  1. remove null check for path, in list secrets API 

    the path is required in the API documentation of vault
but if you manually run the API like
curl --header "X-Vault-Token: …” --request LIST http://127.0.0.1:8200/v1/secret/metadata
it would work

therefore, removing the strict null check and adding a simple guard rail
    @konidev20
    konidev20 committed Nov 22, 2023
    Configuration menu
    Copy the full SHA
    11d68a9 View commit details
    Browse the repository at this point in the history

  2. update IKeyValueSecretsEngineV2 documentation

    @konidev20
    konidev20 committed Nov 22, 2023
    Configuration menu
    Copy the full SHA
    f0c6cb6 View commit details
    Browse the repository at this point in the history

  3. update CHANGELOG.md

    @konidev20
    konidev20 committed Nov 22, 2023
    Configuration menu
    Copy the full SHA
    1f06f00 View commit details
    Browse the repository at this point in the history