Project

General

Profile

Bug #26835

RGW returns MalformedACLError when user create too many ACL.

Added by Chang Liu over 4 years ago. Updated almost 3 years ago.

Status:
Fix Under Review
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

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

History

#1 Updated by Chang Liu over 4 years ago

see

  int grants_num = req_grant_map.size();
  if (grants_num > max_num) {
    ldpp_dout(this, 4) << "An acl can have up to " << max_num
        << " grants, request acl grants num: " << grants_num << dendl;
    op_ret = -ERR_MALFORMED_ACL_ERROR;
    s->err.message = "The request is rejected, because the acl grants number you requested is larger than the maximum " 
                     + std::to_string(max_num)
                     + " grants allowed in an acl.";
    return;
  }

#2 Updated by Chang Liu over 4 years ago

  • Subject changed from RGW returns MalformedACLError when user create too many ACL. S3 returns LimitExceeded indeed. to RGW returns MalformedACLError when user create too many ACL.

should we return LimitExceeded here?

#3 Updated by Casey Bodley over 4 years ago

Returning LimitExceeded does sound reasonable here. Would you care to open a pull request and link it here?

#5 Updated by Casey Bodley almost 4 years ago

  • Status changed from In Progress to 7

#6 Updated by Patrick Donnelly almost 3 years ago

  • Status changed from 7 to Fix Under Review

Also available in: Atom PDF