Actions
Bug #10106
closedrgw acl response should start with <?xml version="1.0" ?>
% Done:
0%
Source:
Community (user)
Tags:
Backport:
giant, firefly
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
I encountered some surprising behaviour when playing with radosgw and s3cmd.
You can probably make a convincing case that the fault lies with s3cmd. But other
applications could easily be wrong in the same way.
Anyway, when I perform GET /?acl or GET /ObjectName?acl on AWS S3, the response content starts like this:
<?xml version="1.0" ?>
<AccessControlPolicy xmlns="http://s3.amazonaws.com/doc/2006-03-01/">
With radosgw, the '<?xml' line is missing.
This confuses s3cmd, so that it does not understand the ACL. Any chance that radosgw could start sending
the missing line?
Updated by Sage Weil over 9 years ago
- Assignee set to Yehuda Sadeh
- Priority changed from Normal to High
- Source changed from other to Community (user)
Updated by Yehuda Sadeh over 9 years ago
- Assignee deleted (
Yehuda Sadeh) - Priority changed from High to Normal
- Source changed from Community (user) to other
Note that Amazon's API definition does not specify this:
http://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectGETacl.html
Updated by Yehuda Sadeh over 9 years ago
- Assignee set to Yehuda Sadeh
- Priority changed from Normal to High
- Source changed from other to Community (user)
Updated by Yehuda Sadeh about 9 years ago
- Status changed from New to Fix Under Review
- Backport set to giant, firefly
Updated by Sage Weil about 9 years ago
- Status changed from Fix Under Review to Pending Backport
Updated by Loïc Dachary about 9 years ago
- firefly backport https://github.com/ceph/ceph/pull/3961
Updated by Loïc Dachary about 9 years ago
- giant backport https://github.com/ceph/ceph/pull/4082 in sync with https://github.com/ceph/ceph-qa-suite/pull/370
Updated by Sage Weil about 9 years ago
- Status changed from Pending Backport to Resolved
Actions