Skip to content

rgw test-case to support x-expected-bucket-owner#610

Merged
ArbitCode merged 1 commit intoceph:masterfrom
ArbitCode:wip-raja-rgw-test-support-64526
Jan 28, 2025
Merged

rgw test-case to support x-expected-bucket-owner#610
ArbitCode merged 1 commit intoceph:masterfrom
ArbitCode:wip-raja-rgw-test-support-64526

Conversation

@ArbitCode
Copy link
Member

If the bucket is owned by a different account, the request fails with the HTTP status code 403 Forbidden (access denied).
PR : ceph/ceph#61215
Fixes : https://tracker.ceph.com/issues/64526

@ArbitCode ArbitCode requested review from cbodley and dang January 17, 2025 12:12
@ArbitCode ArbitCode self-assigned this Jan 17, 2025
@ArbitCode ArbitCode force-pushed the wip-raja-rgw-test-support-64526 branch 2 times, most recently from e969c21 to f977cf2 Compare January 17, 2025 12:17
Copy link
Contributor

@cbodley cbodley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this only covers one s3 op. since we're supporting x-amz-expected-bucket-owner for all bucket/object operations, can we at least cover one of each? list_objects_v2() is a bucket op, maybe add a put_object() test case too?

If the bucket is owned by a different account, the request fails with the HTTP status code 403 Forbidden (access denied).
PR: ceph/ceph#61215
Fixes: https://tracker.ceph.com/issues/64526

Signed-off-by: Raja Sharma <[email protected]>
@ArbitCode ArbitCode force-pushed the wip-raja-rgw-test-support-64526 branch from f977cf2 to 8f36772 Compare January 17, 2025 18:04
@ArbitCode ArbitCode requested a review from cbodley January 17, 2025 18:05
@ArbitCode
Copy link
Member Author

@ArbitCode ArbitCode merged commit 710dd45 into ceph:master Jan 28, 2025
@ArbitCode ArbitCode deleted the wip-raja-rgw-test-support-64526 branch January 28, 2025 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants