rgw/logging: source and log bucket must be different

Signed-off-by: Yuval Lifshitz <[email protected]>

Author: yuvalif

doc/radosgw/bucket_logging.rst

@@ -15,6 +15,12 @@ The log bucket can accumulate logs from multiple buckets. It is recommended to c
 a different "prefix" for each bucket, so that the logs of different buckets will be stored
 in different objects in the log bucket.
 
+.. note::
+
+    - The log bucket must be created before enabling logging on a bucket
+    - The log bucket cannot be the same as the bucket being logged
+    - The log bucket cannot have logging enabled on it
+
 
 .. toctree::
    :maxdepth: 1

src/rgw/rgw_rest_bucket_logging.cc

@@ -181,6 +181,11 @@ class RGWPutBucketLoggingOp : public RGWDefaultResponseOp {
         return;
       }
       const rgw_bucket target_bucket_id(target_tenant_name, target_bucket_name);
+      if (target_bucket_id == src_bucket_id) {
+        ldpp_dout(this, 1) << "ERROR: target bucket '" << target_bucket_id << "' must be different from source bucket" << dendl;
+        op_ret = -EINVAL;
+        return;
+      }
       std::unique_ptr<rgw::sal::Bucket> target_bucket;
       op_ret = driver->load_bucket(this, target_bucket_id,
                                    &target_bucket, y);