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

don't put change password in log #237

Merged
merged 2 commits into from Mar 3, 2015

Conversation

Projects
None yet
3 participants
@kramvan1
Copy link
Contributor

commented Mar 3, 2015

Do the same as was done in the above add user case, remove the password from the resource name.

kramvan1 added some commits Mar 3, 2015

@jjasghar

This comment has been minimized.

Copy link
Collaborator

commented Mar 3, 2015

Oh neat work around. This looks like it'll resolve: #136

@cmluciano

This comment has been minimized.

Copy link
Collaborator

commented Mar 3, 2015

This LGTM. Testing locally.

jjasghar pushed a commit that referenced this pull request Mar 3, 2015

JJ Asghar
Merge pull request #237 from kramvan1/password_in_log
don't put change password in log

@jjasghar jjasghar merged commit 7dc4783 into rabbitmq:master Mar 3, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.