Skip to content

Commit

Permalink
Merge branch 'bridgecrewio-bugfix/check_11_check_access_keys_usage'
Browse files Browse the repository at this point in the history
t push origin master:wq
  • Loading branch information
toniblyx committed Feb 19, 2020
2 parents 44716cf + ca34590 commit 0210c43
Showing 1 changed file with 14 additions and 12 deletions.
26 changes: 14 additions & 12 deletions checks/check11
Original file line number Diff line number Diff line change
Expand Up @@ -17,19 +17,21 @@ CHECK_ALTERNATE_check101="check11"
check11(){
# "Avoid the use of the root account (Scored)."
MAX_DAYS=-1
last_login_date=$(cat $TEMP_REPORT_FILE|awk -F, '{ print $1,$5 }' |grep '<root_account>' | awk '{ print $2 }')
arn=$(cat $TEMP_REPORT_FILE|awk -F, '{ print $1,$2 }' |grep '<root_account>' | awk '{ print $2 }')
last_login_dates=$(cat $TEMP_REPORT_FILE | awk -F, '{ print $1,$5,$11,$16 }' | grep '<root_account>' | cut -d' ' -f2,3,4)

#check if last_login_date date is a valid date, if not, its a pass.
if [[ ${last_login_date%T*} =~ ^[0-9]{4}-[0-9]{2}-[0-9]{2}$ ]];then
days_not_in_use=$(how_many_days_from_today ${last_login_date%T*})
if [ "$days_not_in_use" -gt "$MAX_DAYS" ];then
textFail "Root Account $arn last accessed was less then ${MAX_DAYS#-} day ago"
else
textPass "Root Account $arn last accessed was more then ${MAX_DAYS#-} day ago"
failures=0
for date in $last_login_dates; do
if [[ ${date%T*} =~ ^[0-9]{4}-[0-9]{2}-[0-9]{2}$ ]];then
days_not_in_use=$(how_many_days_from_today ${date%T*})
if [ "$days_not_in_use" -gt "$MAX_DAYS" ];then
failures=1
textFail "Root user in the account was last accessed ${MAX_DAYS#-} day ago"
break
fi
fi
else
textPass "Root Account $arn last accessed was more then ${MAX_DAYS#-} day ago"
fi
done

if [[ $failures == 0 ]]; then
textPass "Root user in the account wasn't accessed in the last ${MAX_DAYS#-} days"
fi
}

0 comments on commit 0210c43

Please sign in to comment.