From: Yehuda Sadeh <yehuda@redhat.com>
Date: Wed, 7 Jun 2017 00:18:39 +0000 (-0700)
Subject: Merge pull request #14117 from zhangsw/fix-rgw-getobjectmetadata-bug
X-Git-Tag: v12.1.0~254
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=7bd8773f7ad1f6f02f8a4c9ddfdeca3df65357e1;p=ceph.git

Merge pull request #14117 from zhangsw/fix-rgw-getobjectmetadata-bug

rgw: return the version id in get object and object metadata request.

Reviewed-by: Yehuda Sadeh <yehuda@redhat.com>
---

7bd8773f7ad1f6f02f8a4c9ddfdeca3df65357e1
diff --cc src/rgw/rgw_op.cc
index daee3f2cdb7d,038a5ec84c80..0118aca0c323
--- a/src/rgw/rgw_op.cc
+++ b/src/rgw/rgw_op.cc
@@@ -1516,12 -1395,8 +1516,13 @@@ void RGWGetObj::execute(
    op_ret = read_op.prepare();
    if (op_ret < 0)
      goto done_err;
+   version_id = read_op.state.obj.key.instance;
  
 +  /* STAT ops don't need data, and do no i/o */
 +  if (get_type() == RGW_OP_STAT_OBJ) {
 +    return;
 +  }
 +
    /* start gettorrent */
    if (torrent.get_flag())
    {
diff --cc src/rgw/rgw_rest_s3.cc
index eef63a57ce91,b330f88ca0fc..0b2550d704b2
--- a/src/rgw/rgw_rest_s3.cc
+++ b/src/rgw/rgw_rest_s3.cc
@@@ -231,11 -216,12 +231,15 @@@ int RGWGetObj_ObjStore_S3::send_respons
      }
    }
  
 +  for (auto &it : crypt_http_responses)
 +    dump_header(s, it.first, it.second);
 +
    dump_content_length(s, total_len);
    dump_last_modified(s, lastmod);
+   if (!version_id.empty()) {
+     dump_header(s, "x-amz-version-id", version_id);
+   }
+   
  
    if (! op_ret) {
      if (! lo_etag.empty()) {