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

update docs related to OCI alias changes #10952

Merged
merged 2 commits into from
Feb 23, 2021
Merged

update docs related to OCI alias changes #10952

merged 2 commits into from
Feb 23, 2021

Conversation

catsby
Copy link
Contributor

@catsby catsby commented Feb 19, 2021

hashicorp/vault-plugin-auth-oci#10 fixed an issue where the OCI login would use the alias name literally. It now will use the role name.

This PR updates the documentation and changelog. It does not update go.mod to pull in the OCI changes; that comes in the next release.

@catsby catsby marked this pull request as ready for review February 19, 2021 22:09
@catsby catsby requested review from kalafut and mgritter February 19, 2021 22:12
CHANGELOG.md Outdated
@@ -56,6 +56,7 @@ BUG FIXES:
* api/sys/config/ui: Fixes issue where multiple UI custom header values are ignored and only the first given value is used [[GH-10490](https://github.com/hashicorp/vault/pull/10490)]
* api: Fixes CORS API methods that were outdated and invalid [[GH-10444](https://github.com/hashicorp/vault/pull/10444)]
* auth/jwt: Fixes `bound_claims` validation for provider-specific group and user info fetching. [[GH-10546](https://github.com/hashicorp/vault/pull/10546)]
* auth/oci: Fixes alias name to use the role name, and not the literal string `name` [[GH-10](https://github.com/hashicorp/vault-plugin-auth-oci/pull/10)]
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this need to be in a changelog file to avoid being overwritten?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Probably so? I updated the PR to use a changelog/ entry in 6f3164d

@vercel vercel bot temporarily deployed to Preview – vault-storybook February 22, 2021 15:51 Inactive
@vercel vercel bot temporarily deployed to Preview – vault February 22, 2021 15:51 Inactive
@catsby catsby merged commit 76584d2 into master Feb 23, 2021
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