bucket logging supports more parameters

Author: yangzong18

src/Models/LoggingEnabled.php

@@ -29,18 +29,28 @@ final class LoggingEnabled extends Model
     #[XmlElement(rename: 'TargetPrefix', type: 'string')]
     public ?string $targetPrefix;
 
+    /**
+     * Log transfer authorization role.
+     * @var string|null
+     */
+    #[XmlElement(rename: 'LoggingRole', type: 'string')]
+    public ?string $loggingRole;
+
 
     /**
      * LoggingEnabled constructor.
      * @param string|null $targetBucket The bucket that stores access logs.
      * @param string|null $targetPrefix The prefix of the log objects.
+     * @param string|null $loggingRole Log transfer authorization role.
      */
     public function __construct(
         ?string $targetBucket = null,
-        ?string $targetPrefix = null
+        ?string $targetPrefix = null,
+        ?string $loggingRole = null
     )
     {
         $this->targetBucket = $targetBucket;
         $this->targetPrefix = $targetPrefix;
+        $this->loggingRole = $loggingRole;
     }
 }

tests/IntegrationTests/ClientBucketLoggingTest.php

@@ -18,7 +18,7 @@ public function testBucketLogging()
             $bucketName,
             new Oss\Models\BucketLoggingStatus(
                 new Oss\Models\LoggingEnabled(
-                    targetBucket: $bucketName, targetPrefix: 'TargetPrefix'
+                    targetBucket: $bucketName, targetPrefix: 'TargetPrefix', loggingRole: 'AliyunOSSLoggingDefaultRole'
                 )
             )
         ));

tests/UnitTests/Transform/BucketLoggingTest.php

@@ -35,6 +35,16 @@ public function testFromPutBucketLogging()
         $this->assertEquals('bucket-123', $input->getBucket());
         $xml = <<<BBB
 <?xml version="1.0" encoding="UTF-8"?><BucketLoggingStatus><LoggingEnabled><TargetBucket>TargetBucket</TargetBucket><TargetPrefix>TargetPrefix</TargetPrefix></LoggingEnabled></BucketLoggingStatus>
+BBB;
+        $this->assertEquals($xml, $this->cleanXml($input->getBody()->getContents()));
+
+        $request = new Models\PutBucketLoggingRequest('bucket-123', new Models\BucketLoggingStatus(new Models\LoggingEnabled(
+            targetBucket: 'TargetBucket', targetPrefix: 'TargetPrefix', loggingRole: 'AliyunOSSLoggingDefaultRole'
+        )));
+        $input = BucketLogging::fromPutBucketLogging($request);
+        $this->assertEquals('bucket-123', $input->getBucket());
+        $xml = <<<BBB
+<?xml version="1.0" encoding="UTF-8"?><BucketLoggingStatus><LoggingEnabled><TargetBucket>TargetBucket</TargetBucket><TargetPrefix>TargetPrefix</TargetPrefix><LoggingRole>AliyunOSSLoggingDefaultRole</LoggingRole></LoggingEnabled></BucketLoggingStatus>
 BBB;
         $this->assertEquals($xml, $this->cleanXml($input->getBody()->getContents()));
     }
@@ -111,6 +121,39 @@ public function testToGetBucketLogging()
         $this->assertEquals('application/xml', $result->headers['content-type']);
         $this->assertEquals('bucket-log', $result->bucketLoggingStatus->loggingEnabled->targetBucket);
         $this->assertEquals('prefix-access_log', $result->bucketLoggingStatus->loggingEnabled->targetPrefix);
+
+
+        $output = new OperationOutput();
+        $result = BucketLogging::toGetBucketLogging($output);
+        $this->assertEquals('', $result->status);
+        $this->assertEquals(0, $result->statusCode);
+        $this->assertEquals('', $result->requestId);
+        $this->assertEquals(0, count($result->headers));
+
+        $body = '<?xml version="1.0" encoding="UTF-8"?>
+<BucketLoggingStatus>
+  <LoggingEnabled>
+        <TargetBucket>bucket-log</TargetBucket>
+        <TargetPrefix>prefix-access_log</TargetPrefix>
+        <LoggingRole>AliyunOSSLoggingDefaultRole</LoggingRole>
+    </LoggingEnabled>
+</BucketLoggingStatus>';
+        $output = new OperationOutput(
+            'OK',
+            200,
+            ['x-oss-request-id' => '123', 'content-type' => 'application/xml'],
+            Utils::streamFor($body)
+        );
+        $result = BucketLogging::toGetBucketLogging($output);
+        $this->assertEquals('OK', $result->status);
+        $this->assertEquals(200, $result->statusCode);
+        $this->assertEquals('123', $result->requestId);
+        $this->assertEquals(2, count($result->headers));
+        $this->assertEquals('123', $result->headers['x-oss-request-id']);
+        $this->assertEquals('application/xml', $result->headers['content-type']);
+        $this->assertEquals('bucket-log', $result->bucketLoggingStatus->loggingEnabled->targetBucket);
+        $this->assertEquals('prefix-access_log', $result->bucketLoggingStatus->loggingEnabled->targetPrefix);
+        $this->assertEquals('AliyunOSSLoggingDefaultRole', $result->bucketLoggingStatus->loggingEnabled->loggingRole);
     }
 
     public function testFromDeleteBucketLogging()