Fix setFileMetada and add tests

.gitignore

@@ -121,4 +121,5 @@ dist
 .DS_Store
 
 test.js
-*.odt
+*.odt
+cqueue.js

CHANGELOG.md

@@ -1,3 +1,6 @@
+### v2.2.3
+- Patched `setFileMetadata`: Bucket aliases are now supported and are not generating a 403 error anymore.
+
 ### v2.2.2
 - Patched rock-req options initialisation
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tiny-storage-client",
-  "version": "2.2.2",
+  "version": "2.2.3",
   "description": "Tiny node client to request distributed AWS S3 or the OpenStack Swift Object Storage.",
   "main": "index.js",
   "scripts": {

s3.js

@@ -312,6 +312,10 @@ module.exports = (config) => {
     let _path = path;
     if (options?.alias !== _activeBucket) {
       _path = _path.replace(options?.alias, _activeBucket);
+      /** For copy-ing object with metadatas, the alias must be applied on the URL */
+      if (options?.headers?.['x-amz-copy-source']) {
+        options.headers['x-amz-copy-source'] = options?.headers?.['x-amz-copy-source'].replace(options?.alias, _activeBucket);
+      }
     }
 
     const _urlParams = getUrlParameters(options?.queries ?? '', options?.defaultQueries ?? '');

tests/s3.test.js

@@ -2201,8 +2201,14 @@ describe('S3 SDK', function () {
 
         const nockRequestS2 = nock(url1S3, {
             reqheaders: {
-              'x-amz-copy-source': () => true,
-              'x-amz-metadata-directive': () => true
+              'x-amz-copy-source': (value) => {
+                assert.strictEqual(value, '/invoices-gra-1234/file.pdf');
+                return true
+              },
+              'x-amz-metadata-directive': (value) => {
+                assert.strictEqual(value, "REPLACE");
+                return true
+              }
             }
           })
           .defaultReplyHeaders(_headers2)