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

webhook: allow to pass data in write request #984

Merged
merged 8 commits into from
May 21, 2020
Merged

Conversation

bonifaido
Copy link
Member

@bonifaido bonifaido commented May 15, 2020

Signed-off-by: Nandor Kracser bonifaido@gmail.com

Q A
Bug fix? no
New feature? yes
API breaks? no
Deprecations? no
Related tickets fixes #977
License Apache 2.0

What's in this PR?

Design and implement the sending of write parameters and document the feature with examples.

Why?

It is already possible to request dynamic secrets from Vault with "write" requests.

However, this feature is undeservedly underdocumented and incomplete since it is not possible to add data to those write requests.

Additional context

Checklist

  • Error handling code meets the guideline
  • Logging code meets the guideline (TODO)
  • User guide and development docs updated (if needed)

@bonifaido bonifaido self-assigned this May 15, 2020
@bonifaido bonifaido marked this pull request as draft May 15, 2020 12:30
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
@bonifaido bonifaido marked this pull request as ready for review May 20, 2020 14:23
@bonifaido bonifaido added the kind/enhancement Categorizes issue or PR as related to an improvement. label May 20, 2020
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>
@bonifaido bonifaido requested a review from pbalogh-sa May 21, 2020 09:30
@bonifaido bonifaido merged commit bb5d8f1 into master May 21, 2020
@bonifaido bonifaido deleted the webhook-write-data branch May 21, 2020 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Categorizes issue or PR as related to an improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

webhook: complete and document the ">>vault:" write functionality
1 participant