Merge branch 'master' into snyk-upgrade-b755cab69bb0e56a63305eaa4b2f9787

Author: mtrezza

CHANGELOG.md

@@ -1,3 +1,10 @@
+# [4.2.0](https://github.com/parse-community/parse-server-s3-adapter/compare/4.1.1...4.2.0) (2025-06-12)
+
+
+### Features
+
+* Add support for option `baseUrl` set to async function ([#285](https://github.com/parse-community/parse-server-s3-adapter/issues/285)) ([6eae9e1](https://github.com/parse-community/parse-server-s3-adapter/commit/6eae9e1f4f4e496e4a53983464639e6d2a303f93))
+
 ## [4.1.1](https://github.com/parse-community/parse-server-s3-adapter/compare/4.1.0...4.1.1) (2025-06-04)
 
 

README.md

@@ -126,7 +126,7 @@ The preferred method is to use the default AWS credentials pattern.  If no AWS c
       "bucketPrefix": '', // default value
       "directAccess": false, // default value
       "fileAcl": null, // default value
-      "baseUrl": null, // default value
+      "baseUrl": null, // string, function or async function
       "baseUrlDirect": false, // default value
       "signatureVersion": 'v4', // default value
       "globalCacheControl": null, // default value. Or 'public, max-age=86400' for 24 hrs Cache-Control
@@ -227,7 +227,7 @@ var s3Options = {
   "region": 'us-east-1', // default value
   "bucketPrefix": '', // default value
   "directAccess": false, // default value
-  "baseUrl": null // default value
+  "baseUrl": null // string, function or async function
   "signatureVersion": 'v4', // default value
   "globalCacheControl": null, // default value. Or 'public, max-age=86400' for 24 hrs Cache-Control
   "presignedUrl": false, // default value

index.js

@@ -31,13 +31,14 @@ const serialize = obj => {
   return str.join('&');
 };
 
-function buildDirectAccessUrl(baseUrl, baseUrlFileKey, presignedUrl, config, filename) {
-  let directAccessUrl;
+async function buildDirectAccessUrl(baseUrl, baseUrlFileKey, presignedUrl, config, filename) {
+  let urlBase;
   if (typeof baseUrl === 'function') {
-    directAccessUrl = `${baseUrl(config, filename)}/${baseUrlFileKey}`;
+    urlBase = await baseUrl(config, filename);
   } else {
-    directAccessUrl = `${baseUrl}/${baseUrlFileKey}`;
+    urlBase = baseUrl;
   }
+  let directAccessUrl = `${urlBase}/${baseUrlFileKey}`;
 
   if (presignedUrl) {
     directAccessUrl += presignedUrl.substring(presignedUrl.indexOf('?'));
@@ -246,7 +247,7 @@ class S3Adapter {
     }
 
     const baseUrlFileKey = this._baseUrlDirect ? fileName : fileKey;
-    return buildDirectAccessUrl(this._baseUrl, baseUrlFileKey, presignedUrl, config, filename);
+    return await buildDirectAccessUrl(this._baseUrl, baseUrlFileKey, presignedUrl, config, filename);
   }
 
   async handleFileStream(filename, req, res) {

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@parse/s3-files-adapter",
-  "version": "4.1.1",
+  "version": "4.2.0",
   "description": "AWS S3 adapter for parse-server",
   "main": "index.js",
   "repository": {

package.json

@@ -640,6 +640,39 @@ describe('S3Adapter tests', () => {
     });
   });
 
+  describe('getFileLocation with async baseUrl', () => {
+    const testConfig = {
+      mount: 'http://example.com/parse',
+      applicationId: 'xxxx',
+    };
+    let options;
+
+    beforeEach(() => {
+      options = {
+        directAccess: true,
+        bucketPrefix: 'foo/bar/',
+        baseUrl: async () => {
+          return await Promise.resolve('http://example.com/files');
+        },
+      };
+    });
+
+    it('should await async baseUrl', async () => {
+      const s3 = new S3Adapter('accessKey', 'secretKey', 'my-bucket', options);
+      await expectAsync(s3.getFileLocation(testConfig, 'test.png')).toBeResolvedTo(
+        'http://example.com/files/foo/bar/test.png'
+      );
+    });
+
+    it('should direct to async baseUrl when baseUrlDirect', async () => {
+      options.baseUrlDirect = true;
+      const s3 = new S3Adapter('accessKey', 'secretKey', 'my-bucket', options);
+      await expectAsync(s3.getFileLocation(testConfig, 'test.png')).toBeResolvedTo(
+        'http://example.com/files/test.png'
+      );
+    });
+  });
+
   describe('validateFilename', () => {
     let options;