From ec28bcc49a84415a235e6f12e0f69e6fc42fbc6e Mon Sep 17 00:00:00 2001 From: Omry Mendelovich <16597193+omryMen@users.noreply.github.com> Date: Thu, 10 Aug 2023 14:47:59 +0300 Subject: [PATCH] fix(arm): improve CKV_AZURE_24 check (#5427) improve CKV_AZURE_24 check --- .../arm/checks/resource/SQLServerAuditingRetention90Days.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/checkov/arm/checks/resource/SQLServerAuditingRetention90Days.py b/checkov/arm/checks/resource/SQLServerAuditingRetention90Days.py index b354e0ef21c..a02df540e57 100644 --- a/checkov/arm/checks/resource/SQLServerAuditingRetention90Days.py +++ b/checkov/arm/checks/resource/SQLServerAuditingRetention90Days.py @@ -50,6 +50,12 @@ def check_resource(resource: Dict[str, Any]) -> CheckResult: retention = properties.get("retentionDays") if isinstance(retention, int) and retention >= 90: return CheckResult.PASSED + if isinstance(retention, str): + try: + if int(retention) >= 90: + return CheckResult.PASSED + except ValueError: # not a valid number + return CheckResult.FAILED return CheckResult.FAILED