Skip to content

Commit

Permalink
Updated README.md
Browse files Browse the repository at this point in the history
Signed-off-by: muhamadto <muhamadto@gmail.com>
  • Loading branch information
muhamadto committed Aug 13, 2023
1 parent d3f66a6 commit a7285cf
Showing 1 changed file with 0 additions and 25 deletions.
25 changes: 0 additions & 25 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -237,24 +237,6 @@ and the following trust relationship
"arn:aws:sns:{aws-region}:{aws-account-number}:spring-native-aws-lambda-function-dead-letter-topic"
]
},
{
"Sid": "SQSPermissions",
"Effect": "Allow",
"Action": [
"sqs:GetQueueAttributes",
"sqs:CreateQueue",
"sqs:DeleteQueue",
"sqs:GetQueueUrl",
"sqs:SetQueueAttributes",
"sqs:ListQueues"
],
"Resource": [
"arn:aws:sqs:{aws-region}:{aws-account-number}:spring-native-aws-lambda-function-failure-queue-dlq",
"arn:aws:sqs:{aws-region}:{aws-account-number}:spring-native-aws-lambda-function-success-queue-dlq",
"arn:aws:sqs:{aws-region}:{aws-account-number}:spring-native-aws-lambda-function-failure-queue",
"arn:aws:sqs:{aws-region}:{aws-account-number}:spring-native-aws-lambda-function-success-queue"
]
},
{
"Sid": "LambdaPermissions",
"Effect": "Allow",
Expand Down Expand Up @@ -333,13 +315,6 @@ cdk bootstrap aws://{aws-account-number}/{aws-region} --profile cdk \
we created
in step 4

**NOTE 2**: I added queues to receive messages from the functions `onSuccess` and `onFailure`
topics, so I had to add the SQS permissions to the policy. However, in production scenario I would
stop at the topics and let the processor applications create and subscribe the queues to the topics.
This way, the processor applications can be deployed independently from the function and the
function can be deployed independently from the processor applications. This is a good practice to
follow in order to have a loosely coupled architecture.

#### Building AWS Lambda Function from Zip

Now that the setup is done you can deploy to AWS.
Expand Down

0 comments on commit a7285cf

Please sign in to comment.