Skip to content
This repository has been archived by the owner on Nov 14, 2020. It is now read-only.

resource_user: Fix tags/password update. #31

Merged
merged 1 commit into from
Nov 25, 2019

Conversation

cyrilgdn
Copy link
Contributor

@cyrilgdn cyrilgdn commented May 7, 2019

Updating tags was resetting password (set to empty)

I had to update the rabbit-hole library which was in a very old version.
So it's quite a big change but this version did not check correctly RMQ API errors.

Copy link
Contributor

@heimweh heimweh left a comment

Choose a reason for hiding this comment

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

Nice work @cyrilgdn! This LGTM 👍

Updating tags was resetting password (set to empty)

I had to update the rabbit-hole library which was in a very old version.
So it's quite a big change but this version did not check correctly RMQ API errors.

Co-authored-by: Alexander Hellbom <[email protected]>
@ghost ghost added size/L and removed size/M labels Nov 25, 2019
@cyrilgdn
Copy link
Contributor Author

Thanks a lot @heimweh , I added you as a co-author to credit the work you've done on that.

@cyrilgdn cyrilgdn merged commit de1de7b into hashicorp:master Nov 25, 2019
@cyrilgdn cyrilgdn deleted the update_tags branch November 25, 2019 22:14
@heimweh
Copy link
Contributor

heimweh commented Dec 9, 2019

Thank you so much @cyrilgdn! 🙏

cyrilgdn added a commit that referenced this pull request Jan 6, 2020
We forgot in PR #31 to do it when changing the version of rabbit-hole
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants