Skip to content

Commit 65271d8

Browse files
authored
Merge pull request ceph#59960 from smanjara/wip-fix-missing-http-data
rgw/multisite: include request body when CreateBucket op is forwarded to master Reviewed-by: Casey Bodley <[email protected]>
2 parents 83d7437 + 43a6f12 commit 65271d8

File tree

3 files changed

+5
-1
lines changed

3 files changed

+5
-1
lines changed

src/rgw/rgw_op.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3703,7 +3703,6 @@ void RGWCreateBucket::execute(optional_yield y)
37033703

37043704
if (!driver->is_meta_master()) {
37053705
// apply bucket creation on the master zone first
3706-
bufferlist in_data;
37073706
JSONParser jp;
37083707
op_ret = rgw_forward_request_to_master(this, *s->penv.site, s->owner.id,
37093708
&in_data, &jp, s->info, y);

src/rgw/rgw_op.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1111,6 +1111,7 @@ class RGWCreateBucket : public RGWOp {
11111111
bool relaxed_region_enforcement = false;
11121112
RGWCORSConfiguration cors_config;
11131113
std::set<std::string> rmattr_names;
1114+
bufferlist in_data;
11141115

11151116
virtual bool need_metadata_upload() const { return false; }
11161117

src/rgw/rgw_rest_s3.cc

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2533,6 +2533,10 @@ int RGWCreateBucket_ObjStore_S3::get_params(optional_yield y)
25332533
if ((op_ret < 0) && (op_ret != -ERR_LENGTH_REQUIRED))
25342534
return op_ret;
25352535

2536+
if (!driver->is_meta_master()) {
2537+
in_data.append(data);
2538+
}
2539+
25362540
if (data.length()) {
25372541
RGWCreateBucketParser parser;
25382542

0 commit comments

Comments
 (0)