Skip to content

Conversation

theburningmonk
Copy link
Collaborator

  • refactored the compile IAM role module to make it easier to support future integrations
  • generate appropriate roles for SNS, SQS, DynamoDB, ECS and Batch permissions to mirror the permissions that the AWS Console would generate when asked to create the roles
  • consolidate permissions by common actions and then resources, to minimise number of policy statements

- consolidate permissions by action and resource to minimise the no. of statements
- added tests for SNS, SQS, DynamoDB and handling for nested parallel states
- don't give me permissions if queueUrl is invalid, makes more sense
@horike37
Copy link
Collaborator

@theburningmonk
Thank you for working on this 👍 Works as expected 😄
Will release next version.

@horike37 horike37 merged commit 58da5e2 into serverless-operations:master Jan 22, 2019
@theburningmonk theburningmonk mentioned this pull request Jan 30, 2019
ss-betseqnzr pushed a commit to BetSEQNZR/serverless-step-functions that referenced this pull request Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants