Merge pull request #54837 from imtzw/urlencode

rgw: fix list obj v1 nextmarker

Reviewed-by: Matt Benjamin <mbenjamin@redhat.com>
Reviewed-by: Casey Bodley <cbodley@redhat.com>

diff --cc src/rgw/rgw_rest_s3.cc
index b3d3891b0ea971827e053c3994a1268a136478f9,511adb990ad126a3ba90795a76108a05533fa72d..f76ac80cf8e6d57502e63b141844260f4ffa1339
--- 1/src/rgw/rgw_rest_s3.cc
--- 2/src/rgw/rgw_rest_s3.cc
+++ b/src/rgw/rgw_rest_s3.cc
@@@ -4151,15 -4103,11 +4107,11 @@@ void RGWListBucketMultiparts_ObjStore_S
      for (iter = uploads.begin(); iter != uploads.end(); ++iter) {
        rgw::sal::MultipartUpload* upload = iter->get();
        s->formatter->open_array_section("Upload");
-       if (encode_url) {
-         s->formatter->dump_string("Key", url_encode(upload->get_key(), false));
-       } else {
-         s->formatter->dump_string("Key", upload->get_key());
-       }
+       dump_urlsafe(s, encode_url, "Key", upload->get_key(), false);
        s->formatter->dump_string("UploadId", upload->get_upload_id());
        const ACLOwner& owner = upload->get_owner();
 -      dump_owner(s, owner.get_id(), owner.get_display_name(), "Initiator");
 -      dump_owner(s, owner.get_id(), owner.get_display_name()); // Owner
 +      dump_owner(s, owner.id, owner.display_name, "Initiator");
 +      dump_owner(s, owner.id, owner.display_name); // Owner
        s->formatter->dump_string("StorageClass", "STANDARD");
        dump_time(s, "Initiated", upload->get_mtime());
        s->formatter->close_section();