Project

General

Profile

Actions
Copy link

Bug #8583

closed

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:
Xiangyu Lv
Target version:
-
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
Regression:
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.

Actions
Copy link
 #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.

Actions
Copy link
 #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

Actions
Copy link
 #3

Updated by Sage Weil over 9 years ago

  • Status changed from New to Fix Under Review
Actions
Copy link
 #4

Updated by Yehuda Sadeh over 9 years ago

  • Status changed from Fix Under Review to Resolved

Merged, commit:fdfe77f9e44be6153ead8bb082717762c747f7eb

Actions
Copy link

Also available in: Atom PDF