Project

General

Profile

Bug #8583

rgw: admin ops create user API can not determine existing user

Added by Xiangyu Lv almost 10 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Version: Firefly 0.8.0

Reproduce Steps:
1) Failed to create a user only with uid
PUT /admin/user?format=json&uid=foo
400 Bad Request {"Code":"InvalidArgument"}

2) Succeeded to create a user with uid and display-name
PUT /admin/user?format=json&uid=foo&display-name=foo
200 OK

3) Succeeded to create a user again with uid and display-name
PUT /admin/user?format=json&uid=foo&display-name=foo
200 OK

The change proposal here is to make create a user only with uid succeed and there after fail.

Associated revisions

Revision 3e85041a (diff)
Added by Ray Lv over 9 years ago

rgw: admin ops create user API can not determine existing user

Fixes: #8583

Signed-off-by: Ray Lv <>

History

#1 Updated by Xiangyu Lv almost 10 years ago

Sent a pull request to fix it: https://github.com/ceph/ceph/pull/1950

Please help to review it.

#2 Updated by Xiangyu Lv over 9 years ago

Added new commit per Yehuda's comments, please check it at: https://github.com/ceph/ceph/pull/1950

#3 Updated by Sage Weil over 9 years ago

  • Status changed from New to Fix Under Review

#4 Updated by Yehuda Sadeh over 9 years ago

  • Status changed from Fix Under Review to Resolved

Merged, commit:fdfe77f9e44be6153ead8bb082717762c747f7eb

Also available in: Atom PDF