summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJ. Eric Ivancich <ivancich@redhat.com>2025-01-17 15:31:31 +0100
committerGitHub <noreply@github.com>2025-01-17 15:31:31 +0100
commit2d416465c24a68ab5342ac100f5b9a47d9b3a305 (patch)
treeb470630671ed9d68fb6a1764b199217db4d921a8
parentMerge pull request #61370 from yehudasa/wip-69536 (diff)
parentrgw: S3 Delete Bucket Policy should return 204 on success (diff)
downloadceph-2d416465c24a68ab5342ac100f5b9a47d9b3a305.tar.xz
ceph-2d416465c24a68ab5342ac100f5b9a47d9b3a305.zip
Merge pull request #61384 from sj14/sj/delete-policy-204
rgw: S3 Delete Bucket Policy should return 204 on success Reviewed-by: Casey Bodley <cbodley@redhat.com>
-rw-r--r--src/rgw/rgw_op.cc5
1 files changed, 4 insertions, 1 deletions
diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc
index 7b0ca3134a3..1793c0b8065 100644
--- a/src/rgw/rgw_op.cc
+++ b/src/rgw/rgw_op.cc
@@ -8582,6 +8582,10 @@ void RGWGetBucketPolicy::execute(optional_yield y)
void RGWDeleteBucketPolicy::send_response()
{
+ if (!op_ret) {
+ /* A successful Delete Bucket Policy should return a 204 on success */
+ op_ret = STATUS_NO_CONTENT;
+ }
if (op_ret) {
set_req_state_err(s, op_ret);
}
@@ -9257,4 +9261,3 @@ void rgw_slo_entry::decode_json(JSONObj *obj)
JSONDecoder::decode_json("etag", etag, obj);
JSONDecoder::decode_json("size_bytes", size_bytes, obj);
};
-