Merge pull request #3 from serverless-components/policy-statement-merge

fix merging of statements in policy
DaySmart · Sep 24, 2019 · a935dc6 · a935dc6
2 parents 9752f99 + f729364
commit a935dc6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@serverless/aws-sns-topic",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "main": "./serverless.js",
   "publishConfig": {
     "access": "public"

diff --git a/serverless.js b/serverless.js
@@ -75,7 +75,11 @@ class AwsSnsTopic extends Component {
       region: inputs.region || defaults.region
     })
 
-    const config = mergeDeepRight(getDefaults({ accountId, arn, defaults }), inputs)
+    const populatedDefaults = getDefaults({ accountId, arn, defaults })
+    const config = mergeDeepRight(populatedDefaults, inputs)
+    for (const i in config.policy.Statement) {
+      config.policy.Statement[i] = mergeDeepRight(populatedDefaults.policy.Statement[0], config.policy.Statement[i])
+    }
     config.arn = arn
 
     this.context.status(`Deploying`)