Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support aws iam user #991

Merged

Conversation

siddarthkay
Copy link
Contributor

Problem

Ability to see AWS IAM Users did not exist

Solution

This PR adds a new user.go file which queries the AWS v2 Go SDK and implements support for calculating AWS IAM Users.
An entry has also been made to aws.go to make sure iam.Users is included.

Changes Made

fixes #547

How to Test

Open Dashboard and apply relevant filters to see iam users

Screenshots

Screenshot 2023-09-23 at 10 48 31 PM

Checklist

  • Code follows the contributing guidelines
  • Changes have been thoroughly tested
  • Documentation has been updated, if necessary
  • Any dependencies have been added to the project, if necessary

Reviewers

@mlabouardy @jakepage91

Copy link
Collaborator

@mlabouardy mlabouardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mlabouardy mlabouardy added the aws label Sep 24, 2023
@mlabouardy mlabouardy added this to the v3.1.1 milestone Sep 24, 2023
@mlabouardy mlabouardy merged commit 5cd4c97 into tailwarden:develop Sep 24, 2023
3 checks passed
roblambell added a commit to roblambell/komiser that referenced this pull request Jan 8, 2024
Azanul pushed a commit that referenced this pull request Jan 9, 2024
* feat: add missing dynamodb action to policy.json

describe table has been needed since #1111

* feat: add missing lambda action to policy.json

list event source mappings has been needed since #1040

* feat: add missing iam action to policy.json

list users has been needed since #991

* fix: correct s3 permission for list buckets api call

listbucket iam action is for the contents of a bucket

* feat: add missing code suite actions to policy.json

needed since #1216, #1229, and #1228
Azanul pushed a commit that referenced this pull request Jan 9, 2024
* feat: add missing dynamodb action to policy.json

describe table has been needed since #1111

* feat: add missing lambda action to policy.json

list event source mappings has been needed since #1040

* feat: add missing iam action to policy.json

list users has been needed since #991

* fix: correct s3 permission for list buckets api call

listbucket iam action is for the contents of a bucket

* feat: add missing code suite actions to policy.json

needed since #1216, #1229, and #1228
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support AWS IAM user
2 participants