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

clarify that init recovery options are only available for auto unseal #17862

Merged
merged 3 commits into from Nov 10, 2022

Conversation

ccapurso
Copy link
Contributor

@ccapurso ccapurso commented Nov 9, 2022

#17764 noted that the recent validation changes to the sys/init endpoint around the recovery options were not clearly documented. This PR clarifies that the recovery options are only available if auto unseal is in use.

Copy link
Contributor

@averche averche left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tiny nit to keep things consistent, LGTM otherwise

website/content/api-docs/system/init.mdx Outdated Show resolved Hide resolved
Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com>
@ccapurso ccapurso merged commit 3a7ce59 into main Nov 10, 2022
@ccapurso ccapurso deleted the update-init-recovery-flag-docs branch November 10, 2022 20:43
jayant07-yb pushed a commit to jayant07-yb/hashicorp-vault-integrations that referenced this pull request Mar 15, 2023
…hashicorp#17862)

* clarify that init recovery options are only available for auto unseal

* add some language consistency

Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com>

Co-authored-by: Anton Averchenkov <84287187+averche@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants