rgw: fix root element for GetPublicAccessBlock

clwluvw · clwluvw · commit db74c10e008c · 2024-02-20T15:55:40.000+01:00
From the AWS doc (https://docs.aws.amazon.com/AmazonS3/latest/API/API_GetPublicAccessBlock.html) it needs to be PublicAccessBlockConfiguration. Fixes: https://tracker.ceph.com/issues/64492 Signed-off-by: Seena Fallah <seenafallah@gmail.com>
diff --git a/src/rgw/rgw_public_access.cc b/src/rgw/rgw_public_access.cc
@@ -9,7 +9,7 @@ void PublicAccessBlockConfiguration::decode_xml(XMLObj *obj) {
 }
 
 void PublicAccessBlockConfiguration::dump_xml(Formatter *f) const {
-  Formatter::ObjectSection os(*f, "BlockPublicAccessBlockConfiguration");
+  Formatter::ObjectSection os(*f, "PublicAccessBlockConfiguration");
   // Note: AWS spec mentions the values to be ALL CAPs, but clients seem to
   // require all small letters, and S3 itself doesn't seem to follow the API
   // spec here

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ void PublicAccessBlockConfiguration::decode_xml(XMLObj *obj) {`
`9`	`9`	`}`
`10`	`10`
`11`	`11`	`void PublicAccessBlockConfiguration::dump_xml(Formatter *f) const {`
`12`		`- Formatter::ObjectSection os(*f, "BlockPublicAccessBlockConfiguration");`
	`12`	`+ Formatter::ObjectSection os(*f, "PublicAccessBlockConfiguration");`
`13`	`13`	`// Note: AWS spec mentions the values to be ALL CAPs, but clients seem to`
`14`	`14`	`// require all small letters, and S3 itself doesn't seem to follow the API`
`15`	`15`	`// spec here`