Skip to content

Commit

Permalink
Merge pull request #263 from cloud-gov/overhaul
Browse files Browse the repository at this point in the history
updated thresholds to only look for 300 days warn and 360 days violation
  • Loading branch information
rcgottlieb authored Dec 10, 2024
2 parents ff7ef6d + 1005597 commit e5405be
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
4 changes: 4 additions & 0 deletions ci/aws-iam-check-keys/find_stale_keys.py
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,10 @@ def main():
com_region = "us-east-1"
gov_region = "us-gov-west-1"

# Note that the thresholds in both thresholds.yml and other_iam_users.yml are now set
# to default to a warning of 300 days and a violation at 360 days
# This was decided based on a discussion with compliance over
# the finding related to stale keys
thresholds = load_thresholds(thresholds_filename)
com_users_list = load_system_users(com_users_filename, thresholds)
gov_users_list = load_system_users(gov_users_filename, thresholds)
Expand Down
16 changes: 8 additions & 8 deletions ci/aws-iam-check-keys/thresholds.yml
Original file line number Diff line number Diff line change
@@ -1,20 +1,20 @@
- account_type: "Operator"
is_wildcard: True
warn: 75
violation: 90
warn: 300
violation: 360
alert: True
- account_type: "Platform"
is_wildcard: True
warn: 80
violation: 165
warn: 300
violation: 360
alert: True
- account_type: "Application"
is_wildcard: True
warn: 80
violation: 165
warn: 360
violation: 360
alert: True
- account_type: "Customer"
is_wildcard: True
warn: 0
violation: 0
warn: 300
violation: 360
alert: False

0 comments on commit e5405be

Please sign in to comment.