]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw/s3: dump Message field in Error response even if empty 51482/head
authorCasey Bodley <cbodley@redhat.com>
Wed, 12 Apr 2023 15:18:15 +0000 (11:18 -0400)
committerCasey Bodley <cbodley@redhat.com>
Mon, 15 May 2023 13:40:23 +0000 (09:40 -0400)
some clients like veeam fail to decode the response if Message is missing

Fixes: https://tracker.ceph.com/issues/59433
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 6de9e376b61fc11be9033a03d027dd0fa7f200ad)

src/rgw/rgw_common.cc

index c9a1c0fcdc7811c92d3844fee11aa37eceba9459..5868268c5e1b174a4e1b5a5349b581e0f7208ae1 100644 (file)
@@ -368,8 +368,7 @@ void dump(req_state* s)
     s->formatter->open_object_section("Error");
   if (!s->err.err_code.empty())
     s->formatter->dump_string("Code", s->err.err_code);
-  if (!s->err.message.empty())
-    s->formatter->dump_string("Message", s->err.message);
+  s->formatter->dump_string("Message", s->err.message);
   if (!s->bucket_name.empty()) // TODO: connect to expose_bucket
     s->formatter->dump_string("BucketName", s->bucket_name);
   if (!s->trans_id.empty())    // TODO: connect to expose_bucket or another toggle