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

mimic: rgw: Admin OPS Api overwrites email when user is modified #22523

Merged
merged 1 commit into from Jul 17, 2018

Conversation

@smithfarm
Copy link
Contributor

commented Jun 12, 2018

rgw: Do not modify email if argument is not set
The RGW Admin OPS API overwrites the users email either it is not set via argument, e.g. when executing /{admin}/user?uid=test5&suspended=true.

Signed-off-by: Volker Theile <vtheile@suse.com>
(cherry picked from commit c6792a8)

@smithfarm smithfarm self-assigned this Jun 12, 2018

@smithfarm smithfarm added this to the mimic milestone Jun 12, 2018

@smithfarm smithfarm added bug fix core rgw and removed core labels Jun 12, 2018

@smithfarm smithfarm requested review from cbodley, oritwas and votdev Jun 12, 2018

@smithfarm smithfarm changed the title mimic: Admin OPS Api overwrites email when user is modified mimic: rgw: Admin OPS Api overwrites email when user is modified Jun 12, 2018

@smithfarm smithfarm requested a review from LenzGr Jun 12, 2018

@votdev

votdev approved these changes Jun 12, 2018

@yuriw

This comment has been minimized.

Copy link
Contributor

commented Jul 13, 2018

@yuriw yuriw merged commit 0cf0fd7 into ceph:mimic Jul 17, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm deleted the smithfarm:wip-24253-mimic branch Jul 17, 2018

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.