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-3008] Update RabbitMQ dependency and fix regression in UserInfo.Tags in v3.9 #12877

Merged
merged 2 commits into from
Oct 20, 2021

Conversation

vinay-gopalan
Copy link
Contributor

This PR updates the go.mod to use github.com/michaelklishin/rabbit-hole/v2 which is the latest version of the package. This was done to resolve a RabbitMQ regression in v3.9 with UserInfo.Tags; they now return a string array instead of a string.

With this update, we can now use the image 3-management in backend_test.go and do not need to pin the docker image versions as done previously in this PR. The only change in Vault logic is passing role.Tags as a string array when trying to set the UserInfo.tags field in path_role_create.go.

@vinay-gopalan vinay-gopalan requested a review from a team October 20, 2021 01:13
@vercel vercel bot temporarily deployed to Preview – vault-storybook October 20, 2021 01:16 Inactive
@vercel vercel bot temporarily deployed to Preview – vault October 20, 2021 01:16 Inactive
Copy link
Contributor

@calvn calvn left a comment

Choose a reason for hiding this comment

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

Nice find, this looks good!

@vinay-gopalan vinay-gopalan merged commit 1baa173 into main Oct 20, 2021
@vinay-gopalan vinay-gopalan deleted the rabbitmq-secrets-3.9 branch October 20, 2021 16:46
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.

2 participants