From: Seena Fallah <seenafallah@gmail.com>
Date: Fri, 28 Mar 2025 23:00:02 +0000 (+0100)
Subject: qa/rgw: add perm check test for copy obj between zonegroups
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=e704129b4f5de4991e189c88c77b71851023c826;p=ceph.git

qa/rgw: add perm check test for copy obj between zonegroups

Make sure perms are evaluated properly for the source object.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 9523e15fb74e09718f5cc9c0bddf2492fc8d8128)
---

diff --git a/src/test/rgw/rgw_multi/tests.py b/src/test/rgw/rgw_multi/tests.py
index b7fd013a18952..616a41f2992ba 100644
--- a/src/test/rgw/rgw_multi/tests.py
+++ b/src/test/rgw/rgw_multi/tests.py
@@ -5764,7 +5764,7 @@ def test_bucket_replication_source_forbidden_getobjecttagging():
             'Version': '2012-10-17',
             'Statement': [{
                 'Effect': 'Deny',
-                'Principal': {'AWS': [f"arn:aws:iam:::user/{user.id}"]},
+                'Principal': {'AWS': [f"arn:aws:iam:::user/{non_account_user.id}"]},
                 'Action': 's3:GetObjectTagging',
                 'Resource': f'arn:aws:s3:::{source_bucket.name}/*',
             }]
@@ -5821,7 +5821,7 @@ def test_bucket_replication_source_forbidden_getobjectversiontagging():
             'Version': '2012-10-17',
             'Statement': [{
                 'Effect': 'Deny',
-                'Principal': {'AWS': [f"arn:aws:iam:::user/{user.id}"]},
+                'Principal': {'AWS': [f"arn:aws:iam:::user/{non_account_user.id}"]},
                 'Action': 's3:GetObjectVersionTagging',
                 'Resource': f'arn:aws:s3:::{source_bucket.name}/*',
             }]
@@ -5838,3 +5838,30 @@ def test_bucket_replication_source_forbidden_getobjectversiontagging():
     res = dest.s3_client.get_object(Bucket=dest_bucket.name, Key=objname)
     assert_equal(res['Body'].read().decode('utf-8'), 'foo')
     assert 'TagCount' not in res
+
+@run_per_zonegroup
+def test_copy_obj_perm_check_between_zonegroups(zonegroup):
+    if len(realm.current_period.zonegroups) < 2:
+        raise SkipTest('need at least 2 zonegroups to run this test')
+
+    source_zone = ZonegroupConns(zonegroup).rw_zones[0]
+    source_bucket = source_zone.create_bucket(gen_bucket_name())
+
+    objname = 'dummy'
+    k = new_key(source_zone, source_bucket.name, objname)
+    k.set_contents_from_string('foo')
+
+    for zg in realm.current_period.zonegroups:
+        if zg.name == zonegroup.name:
+            continue
+
+        dest_zone = ZonegroupConns(zg).non_account_alt_rw_zones[0]
+        dest_bucket = dest_zone.create_bucket(gen_bucket_name())
+        realm_meta_checkpoint(realm)
+
+        # copy object returns 403
+        e = assert_raises(ClientError, dest_zone.s3_client.copy_object,
+                          Bucket=dest_bucket.name,
+                          CopySource={'Bucket': source_bucket.name, 'Key': objname},
+                          Key=objname)
+        assert e.response['Error']['Code'] == 'AccessDenied'