Merge pull request #1862 from packagist/url-encode-copy-source-header

AsyncAws S3: URL-encode the CopySource argument

Author: frankdejonge

src/AsyncAwsS3/AsyncAwsS3Adapter.php

@@ -357,7 +357,7 @@ public function copy(string $source, string $destination, Config $config): void
             'ACL' => $this->visibility->visibilityToAcl($visibility ?: 'private'),
             'Bucket' => $this->bucket,
             'Key' => $this->prefixer->prefixPath($destination),
-            'CopySource' => $this->bucket . '/' . $this->prefixer->prefixPath($source),
+            'CopySource' => rawurlencode($this->bucket . '/' . $this->prefixer->prefixPath($source)),
         ];
 
         try {

src/AsyncAwsS3/AsyncAwsS3AdapterTest.php

@@ -386,6 +386,27 @@ public function copying_a_file_with_visibility(): void
         });
     }
 
+    /**
+     * @test
+     */
+    public function copying_a_file_with_non_ascii_characters(): void
+    {
+        $this->runScenario(function () {
+            $adapter = $this->adapter();
+            $adapter->write(
+                'ıÇöü🤔.txt',
+                'contents to be copied',
+                new Config()
+            );
+
+            $adapter->copy('ıÇöü🤔.txt', 'ıÇöü🤔_copy.txt', new Config());
+
+            $this->assertTrue($adapter->fileExists('ıÇöü🤔.txt'));
+            $this->assertTrue($adapter->fileExists('ıÇöü🤔_copy.txt'));
+            $this->assertEquals('contents to be copied', $adapter->read('ıÇöü🤔_copy.txt'));
+        });
+    }
+
     /**
      * @test
      */