fix(kms): add symmetric condition to kms_cmk_rotation_enabled check (#1788)

Co-authored-by: sergargar <sergio@verica.io>
This commit is contained in:
Sergio Garcia
2023-01-27 10:49:40 +01:00
committed by GitHub
parent f6ea10db2d
commit 5d2b8bc8aa
2 changed files with 7 additions and 1 deletions

View File

@@ -9,7 +9,11 @@ class kms_cmk_rotation_enabled(Check):
report = Check_Report_AWS(self.metadata())
report.region = key.region
# Only check enabled CMKs keys
if key.manager == "CUSTOMER" and key.state == "Enabled":
if (
key.manager == "CUSTOMER"
and key.state == "Enabled"
and "SYMMETRIC" in key.spec
):
if key.rotation_enabled:
report.status = "PASS"
report.status_extended = (

View File

@@ -58,6 +58,7 @@ class KMS:
key.state = response["KeyMetadata"]["KeyState"]
key.origin = response["KeyMetadata"]["Origin"]
key.manager = response["KeyMetadata"]["KeyManager"]
key.spec = response["KeyMetadata"]["CustomerMasterKeySpec"]
except Exception as error:
logger.error(
f"{regional_client.region} -- {error.__class__.__name__}:{error.__traceback__.tb_lineno} -- {error}"
@@ -102,6 +103,7 @@ class Key:
manager: str
rotation_enabled: bool
policy: dict
spec: str
region: str
def __init__(