Skip to content
This repository was archived by the owner on Feb 8, 2024. It is now read-only.

Commit fbf40ba

Browse files
Added cosmetic change in etag field of CopyObjectResult (#388)
Problem: Observed discrepancy with AWS for CopyObject API's response. In MGW, "etag" field was not included in ("/ /") Solution: Included etag field of CopyObject API's response in ("/ /") to make it AWS compliant Signed-off-by: shraddhaghatol <[email protected]>
1 parent 1e124eb commit fbf40ba

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/rgw/rgw_rest_s3.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3330,7 +3330,7 @@ void RGWCopyObj_ObjStore_S3::send_response()
33303330
if (op_ret == 0) {
33313331
dump_time(s, "LastModified", mtime);
33323332
if (!etag.empty()) {
3333-
s->formatter->dump_string("ETag", std::move(etag));
3333+
s->formatter->dump_format("ETag", "\"%s\"",etag.c_str());
33343334
}
33353335
s->formatter->close_section();
33363336
rgw_flush_formatter_and_reset(s, s->formatter);

0 commit comments

Comments
 (0)