Skip to content

Commit 63ad0a6

Browse files
authored
Merge pull request #129 from cloudgraphdev/fix/CG-1330-aws-cis-140-38
fix(CG-1330): AWS CIS 1.4.0 rule 3.8 fix
2 parents 28e436c + d4f0421 commit 63ad0a6

2 files changed

Lines changed: 6 additions & 14 deletions

File tree

src/aws/cis-1.4.0/rules/aws-cis-1.4.0-3.8.ts

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ export default {
1212
audit: `Via the Management Console:
1313
1414
1. Sign in to the AWS Management Console and open the IAM console at https://console.aws.amazon.com/iam.
15-
2. In the left navigation pane, choose E*ncryption Keys*.
15+
2. In the left navigation pane, choose *Encryption Keys*.
1616
3. Select a customer created master key (CMK)
1717
4. Under the *Key Policy* section, move down to *Key Rotation*.
1818
5. Ensure the *Rotate this key every year* checkbox is checked.
@@ -52,7 +52,7 @@ export default {
5252
id
5353
arn
5454
accountId
55-
__typename
55+
__typename
5656
keyManager
5757
keyRotationEnabled
5858
}
@@ -62,16 +62,8 @@ export default {
6262
conditions: {
6363
or: [
6464
{
65-
and: [
66-
{
67-
path: '@.keyManager',
68-
equal: 'AWS',
69-
},
70-
{
71-
path: '@.keyRotationEnabled',
72-
equal: true,
73-
},
74-
],
65+
path: '@.keyManager',
66+
equal: 'AWS',
7567
},
7668
{
7769
and: [

src/aws/cis-1.4.0/tests/aws-cis-1.4.0-3.x.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -551,9 +551,9 @@ describe('CIS Amazon Web Services Foundations: 1.4.0', () => {
551551
await testRule(data, Result.FAIL)
552552
})
553553

554-
test('Security Issue when rotation is disabled with AWS as a manager', async () => {
554+
test('No Security Issue when rotation is disabled with AWS as a manager', async () => {
555555
const data: CIS3xQueryResponse = getTestRuleFixture('AWS', false)
556-
await testRule(data, Result.FAIL)
556+
await testRule(data, Result.PASS)
557557
})
558558
})
559559

0 commit comments

Comments
 (0)