-
Notifications
You must be signed in to change notification settings - Fork 548
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
Add new resource vault_pki_secret_backend_acme_eab to manage ACME EAB tokens #2367
Add new resource vault_pki_secret_backend_acme_eab to manage ACME EAB tokens #2367
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very neatly done.
I think the documentation needs one fix, but I'll mark the PR as approved now since the fix doesn't require a re-review.
page_title: "Vault: vault_pki_secret_backend_acme_eab resource" | ||
sidebar_current: "docs-vault-resource-pki-secret-backend-acme-eab" | ||
description: |- | ||
Sets the EST configuration on a PKI Secret Backend for Vault. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may have overlooked updating this description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
|
||
* `role` - (Optional) Create an EAB token that is specific to a role's ACME directory. | ||
|
||
**NOTE**: EAB tokens are specific to an ACME directory. Within Vault ACME there are different |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not very familiar with the ACME terminology, but maybe the sentence can be simplified to
Within Vault there are different ACME directories.
And maybe end the sentence with semicolons?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tweaked it a bit.
…2587-add-support-for-acme-eab-tokens
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Description
Add a new resource called
vault_pki_secret_backend_acme_eab
that allows TFVP to create and delete EAB tokens (External Account Bindings). These tokens allow someone to restrict who can create a new ACME account within a particular ACME directory within Vault.Relates #1947
Checklist
Output from acceptance testing:
Community Note