fix(terraform): Fix outdated CKV_AWS_312 for Elastic Beanstalk health reporting#7434
Open
AlekseiSokolov2TR wants to merge 5 commits intobridgecrewio:mainfrom
Open
fix(terraform): Fix outdated CKV_AWS_312 for Elastic Beanstalk health reporting#7434AlekseiSokolov2TR wants to merge 5 commits intobridgecrewio:mainfrom
AlekseiSokolov2TR wants to merge 5 commits intobridgecrewio:mainfrom
Conversation
a1805ff to
3188635
Compare
3188635 to
7457d8c
Compare
7457d8c to
0babac6
Compare
0babac6 to
ec11380
Compare
ec11380 to
d8b0ee8
Compare
d8b0ee8 to
f593aa6
Compare
f593aa6 to
7a9df8e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Description
CKV_AWS_312: Elastic Beanstalk environments do not have enhanced health reporting enabled
Checkov produces a false positive. The check references a setting that no longer exists.
The check expects
The actual configuration for the feature being checked should be
The relevant documentation can be found here
Fixes #6007
A related setting exists that controls streaming health data to CloudWatch:
However, the original wording of CKV_AWS_312 suggest that its intent is checking for the health streaming being enabled, not for health data being archived in CloudWatch. Therefore, either CKV_AWS_312 needs to be expanded to include this, or an additional check needs to be introduced, if the CloudWatch archival part is deemed necessary.
New/Edited policies (Delete if not relevant)
CKV_AWS_312
Description
The change addresses an update to the underlying API that makes the current check obsolete.
Checklist: