checkov icon indicating copy to clipboard operation
checkov copied to clipboard

CKV_AZURE_12: Bicep reports always the retention is lower than 90 days

Open sbe-arg opened this issue 3 years ago • 3 comments

Describe the issue CKV_AZURE_12 should accept values of 91 or higher in bicep and not report any errors.

Examples image

Version (please complete the following information):

  • latest v12

sbe-arg avatar Jan 24 '23 21:01 sbe-arg

hey @sbe-arg thanks for reaching out. I tested it and I think it is related to the argument enabled: enableLogs, if this clearly not set to true then this check fails.

gruebel avatar Jan 24 '23 22:01 gruebel

hahaha great point @gruebel I had default to false that parameter I have done the opposite now default to true and send a false when required and the errors cleared up so is not that the information is not being captured.

Although if the logs are disabled regardless of the int value the check should be skipped right?

sbe-arg avatar Jan 25 '23 00:01 sbe-arg

or should analyzed also if set to false either way does seem something is wrong or missing

sbe-arg avatar Jan 25 '23 23:01 sbe-arg

Thanks for contributing to Checkov! We've automatically marked this issue as stale to keep our issues list tidy, because it has not had any activity for 6 months. It will be closed in 14 days if no further activity occurs. Commenting on this issue will remove the stale tag. If you want to talk through the issue or help us understand the priority and context, feel free to add a comment or join us in the Checkov slack channel at https://slack.bridgecrew.io Thanks!

stale[bot] avatar Jul 25 '23 12:07 stale[bot]

Closing issue due to inactivity. If you feel this is in error, please re-open, or reach out to the community via slack: https://slack.bridgecrew.io Thanks!

stale[bot] avatar Aug 08 '23 20:08 stale[bot]