Skip to content

Commit

Permalink
Minor logic update
Browse files Browse the repository at this point in the history
  • Loading branch information
iann0036 committed Apr 25, 2021
1 parent 9f51a32 commit 33e1911
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -165,4 +165,4 @@ This project requires Go 1.16 or above to be built correctly (due to embedding f

## Acknowledgements

This project makes heavy use of [Parliament](https://github.com/duo-labs/parliament) and was assisted by Scott Piper's [CSM explainer](https://summitroute.com/blog/2020/05/25/client_side_monitoring/). Thanks also to Noam Dahan's [research](https://ermetic.com/whats-new/blog/auditing-passrole-a-problematic-privilege-escalation-permission/) into missing `iam:PassRole` dependant actions.
This project makes use of [Parliament](https://github.com/duo-labs/parliament) and was assisted by Scott Piper's [CSM explainer](https://summitroute.com/blog/2020/05/25/client_side_monitoring/). Thanks also to Noam Dahan's [research](https://ermetic.com/whats-new/blog/auditing-passrole-a-problematic-privilege-escalation-permission/) into missing `iam:PassRole` dependant actions.
3 changes: 3 additions & 0 deletions logger.go
Original file line number Diff line number Diff line change
Expand Up @@ -590,6 +590,9 @@ func getStatementsForProxyCall(call Entry) (statements []Statement) {

// default (last ditch)
if len(resources) == 0 {
if len(mappedPriv.ResourceARNMappings) > 0 { // skip if resourcearn_mapping was specified and didn't hit
continue
}
resources = []string{"*"}
}

Expand Down

0 comments on commit 33e1911

Please sign in to comment.