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

Rename Vault->OpenBao in API field descriptions #72

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

cipherboy
Copy link
Member

@cipherboy cipherboy commented Jan 27, 2024

This starts to clean up the OpenAPI spec by renaming most instances of Vault with OpenBao, outside of references to environment variables or headers.

Note that this will need to be re-done if #64 is accepted and plugins are imported or forked. When the API has stabilized again, we can regenerate the OpenAPI spec and check it in.

You can run make openapi to check out the artifact generated by this pull request.

@naphelps naphelps self-requested a review February 5, 2024 15:35
@cipherboy cipherboy changed the title Rename Vault->OpenBao in API responses Rename Vault->OpenBao in API field descriptions Feb 7, 2024
@cipherboy cipherboy force-pushed the rename-openapi branch 2 times, most recently from 8f7b6f6 to fc3075a Compare February 7, 2024 12:35
This starts to clean up the OpenAPI spec by renaming most instances of
Vault with OpenBao, outside of references to environment variables or
headers.

Note that this will need to be re-done if openbao#64 is accepted and plugins
are imported or forked. When the API has stabilized again, we can
regenerate the OpenAPI spec and check it in.

Signed-off-by: Alexander Scheel <alexander.m.scheel@gmail.com>
@naphelps naphelps merged commit c575110 into openbao:main Feb 8, 2024
3 of 9 checks passed
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.

None yet

3 participants