Skip to content

Commit

Permalink
fix: update pinpoint IAM policy to use mobiletargeting instead of mob…
Browse files Browse the repository at this point in the history
…ileanalytics (#326)

* fix: update pinpoint IAM policy to use mobiletargeting instead of mobileanalytics

* chnages based on PR reviews
  • Loading branch information
kaustavghosh06 committed Oct 23, 2018
1 parent 31da634 commit 96fcde6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 27 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -208,15 +208,7 @@
{
"Effect": "Allow",
"Action": [
"mobileanalytics:PutEvents"
],
"Resource": [
"*"
]
},
{
"Effect": "Allow",
"Action": [
"mobiletargeting:PutEvents",
"mobiletargeting:UpdateEndpoint"
],
"Resource": [
Expand Down Expand Up @@ -284,15 +276,7 @@
{
"Effect": "Allow",
"Action": [
"mobileanalytics:PutEvents"
],
"Resource": [
"*"
]
},
{
"Effect": "Allow",
"Action": [
"mobiletargeting:PutEvents",
"mobiletargeting:UpdateEndpoint"
],
"Resource": [
Expand Down
10 changes: 1 addition & 9 deletions packages/amplify-category-notifications/lib/auth-helper.js
Original file line number Diff line number Diff line change
Expand Up @@ -96,15 +96,7 @@ function getPolicyDoc(context) {
{
Effect: 'Allow',
Action: [
'mobileanalytics:PutEvents',
],
Resource: [
'*',
],
},
{
Effect: 'Allow',
Action: [
'mobiletargeting:PutEvents',
'mobiletargeting:UpdateEndpoint',
],
Resource: [
Expand Down

0 comments on commit 96fcde6

Please sign in to comment.