Project

General

Profile

Fix #6892

rgw: ignore empty owner in set acl api

Added by Josh Durgin over 6 years ago. Updated about 6 years ago.

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

0%

Source:
Community (user)
Tags:
Backport:
dumpling
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature:

Description

s3cmd sends an empty owner field when setting acls. Currently radosgw rejects this since it expects the owner to match the existing owner of the bucket.

See http://www.mail-archive.com/ceph-users@lists.ceph.com/msg05503.html for details.

Associated revisions

Revision 14cf4caf (diff)
Added by Yehuda Sadeh over 6 years ago

rgw: don't error out on empty owner when setting acls

Fixes: #6892
Backport: dumpling, emperor
s3cmd specifies empty owner field when trying to set acls on object
/ bucket. We errored out as it didn't match the current owner name, but
with this change we ignore it.

Signed-off-by: Yehuda Sadeh <>

Revision 8812c4e9 (diff)
Added by Yehuda Sadeh over 6 years ago

rgw: don't error out on empty owner when setting acls

Fixes: #6892
Backport: dumpling, emperor
s3cmd specifies empty owner field when trying to set acls on object
/ bucket. We errored out as it didn't match the current owner name, but
with this change we ignore it.

Signed-off-by: Yehuda Sadeh <>
(cherry picked from commit 14cf4caff58cc2c535101d48c53afd54d8632104)

Revision fced0562 (diff)
Added by Yehuda Sadeh about 6 years ago

rgw: don't error out on empty owner when setting acls

Fixes: #6892
Backport: dumpling, emperor
s3cmd specifies empty owner field when trying to set acls on object
/ bucket. We errored out as it didn't match the current owner name, but
with this change we ignore it.

Signed-off-by: Yehuda Sadeh <>
(cherry picked from commit 14cf4caff58cc2c535101d48c53afd54d8632104)

History

#1 Updated by Yehuda Sadeh over 6 years ago

  • Status changed from New to Fix Under Review
  • Backport set to dumpling, emperor

#2 Updated by Sage Weil over 6 years ago

  • Status changed from Fix Under Review to Resolved

#3 Updated by Ian Colle about 6 years ago

  • Status changed from Resolved to Pending Backport
  • Backport changed from dumpling, emperor to dumpling

#4 Updated by Yehuda Sadeh about 6 years ago

  • Status changed from Pending Backport to Resolved

cherry picked into dumpling

Also available in: Atom PDF