Project

General

Profile

Bug #13096

rgw: The type of caps can be any value.

Added by Kongming Wu over 8 years ago. Updated over 8 years ago.

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

0%

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

Description

When executing the command of [radosgw-admin caps add --uid=wkm --caps="122=write"], the results show we add caps successfully.But I read source files and documents of ceph 0.94.2, and found that the “types” of caps are certain values, which means the number 122 in the above is incorrect. But I am not sure how many,because there are 5 types according to the document, but i found more in the source code.

History

#1 Updated by Loïc Dachary over 8 years ago

  • Project changed from Ceph to rgw

#2 Updated by Yehuda Sadeh over 8 years ago

The solution would require mapping out the different types of caps that exist, and then to restrict that to those existing types. This is an easy project to work on for anyone who wants to start looking at the code!

#3 Updated by Nathan Cutler over 8 years ago

  • Tags set to new-dev

#4 Updated by Kongming Wu over 8 years ago

Yehuda Sadeh wrote:

The solution would require mapping out the different types of caps that exist, and then to restrict that to those existing types. This is an easy project to work on for anyone who wants to start looking at the code!

About 2 months ago, I added a pull request on github with the No.#6051. Among the modifying I listed All existed types of caps that can be found in the source code.

#5 Updated by Yehuda Sadeh over 8 years ago

Ah, right! Ok, added a couple of comments in the PR, but looks good otherwise.

#6 Updated by Kongming Wu over 8 years ago

Yehuda Sadeh wrote:

Ah, right! Ok, added a couple of comments in the PR, but looks good otherwise.

OK!I see it and I am going to fix the PR.

#7 Updated by Kongming Wu over 8 years ago

Yehuda Sadeh wrote:

Ah, right! Ok, added a couple of comments in the PR, but looks good otherwise.

Hi,Yehuda,I repushed a PR by overwriting, Accoding to what you said.

#8 Updated by Yehuda Sadeh over 8 years ago

  • Status changed from New to Resolved

Merge commit:7ce223ec0f4906a9dee688af1d3c16ecf6a52be9

Also available in: Atom PDF