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

VAULT-12144: add openapi responses for /sys/rotate endpoints #18624

Merged
merged 8 commits into from
Mar 28, 2023

Conversation

dhuckins
Copy link
Contributor

@dhuckins dhuckins commented Jan 6, 2023

Path Operation response docs
/sys/rotate UPDATE link
/sys/rotate/config READ link
/sys/rotate/config UPDATE link

added response structure tests to /rotate/config
(did not add to /rotate b/c it doesn't return anything)

tests added

Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins dhuckins changed the title add openapi responses for /sys/rotate endpoints VAULT-12144: add openapi responses for /sys/rotate endpoints Jan 6, 2023
@dhuckins dhuckins added this to the 1.13.0-rc1 milestone Jan 6, 2023
@dhuckins dhuckins requested a review from a team January 6, 2023 21:08
@dhuckins dhuckins marked this pull request as ready for review January 6, 2023 21:49
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@dhuckins dhuckins requested review from a team and AnPucel January 20, 2023 17:39
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
vault/logical_system_test.go Outdated Show resolved Hide resolved
vault/logical_system_test.go Outdated Show resolved Hide resolved
Signed-off-by: Daniel Huckins <dhuckins@users.noreply.github.com>
@averche averche modified the milestones: 1.13.0-rc1, 1.14 Feb 13, 2023
@dhuckins dhuckins closed this Mar 24, 2023
@dhuckins dhuckins reopened this Mar 24, 2023
@dhuckins dhuckins merged commit 1498560 into main Mar 28, 2023
@dhuckins dhuckins deleted the dh/response-structures-sys-3-rotate branch March 28, 2023 19:40
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.

3 participants