Skip to content
This repository has been archived by the owner on Jan 11, 2021. It is now read-only.

Add SNS permissions for Scout2 #8

Merged
merged 1 commit into from Feb 8, 2017
Merged

Add SNS permissions for Scout2 #8

merged 1 commit into from Feb 8, 2017

Conversation

edeca
Copy link

@edeca edeca commented Feb 7, 2017

Fixes #7

@l01cd3v l01cd3v merged commit 029145f into nccgroup:master Feb 8, 2017
@l01cd3v
Copy link
Collaborator

l01cd3v commented Feb 8, 2017

Thanks ! merged.

FYI, In a somewhat near future, I will probably do the following:

  • Modify list* into an explicit list of all APIs (just 2 for now)
  • Create a new policy (maybe "Scout2-All") that includes ListSubscriptions and remove it from the default policy in case people are not comfortable with exposing emails addresses/phone numbers/... to their auditors. This would happen after modification of Scout2 to handle the not authorized exception.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants