fix S3 Route: aggregate result promises to one promise

Author: connystrecker

lib/s3.js

@@ -61,7 +61,7 @@ function process(s3Config, event, context) {
         return null;
     }
 
-    const results = [];
+    const resultPromises = [];
     for (const record of event.Records) {
         for (const routeConfig of s3Config.routes) {
             const bucketNameMatched = matchConfigToEventValue(routeConfig.bucketName, record.s3.bucket.name);
@@ -74,16 +74,15 @@ function process(s3Config, event, context) {
             }
 
             if (bucketNameMatched && eventNameMatched && objectKeyPrefixMatched) {
-                const result = routeConfig.action(record, context);
-                if (result) {
-                    results.push(result);
+                const resultPromise = routeConfig.action(record, context);
+                if (resultPromise) {
+                    resultPromises.push(resultPromise);
                 }
                 break;
             }
         }
     }
-
-    return results;
+    return Promise.all(resultPromises);
 }
 
 module.exports = process;