Skip to content

fix(query): skip Fargate local volumes in EFS transit encryption check - #8083

Open
l46983284-cpu wants to merge 5 commits into
Checkmarx:masterfrom
l46983284-cpu:fix/kics-8031-fargate-efs-volume-fp
Open

fix(query): skip Fargate local volumes in EFS transit encryption check#8083
l46983284-cpu wants to merge 5 commits into
Checkmarx:masterfrom
l46983284-cpu:fix/kics-8031-fargate-efs-volume-fp

Conversation

@l46983284-cpu

Copy link
Copy Markdown

Closes #8031

Reason for Proposed Changes

  • Query 4d46ff3b (EFS Volume With Disabled Transit Encryption) treats every plain volume block on aws_ecs_task_definition as a missing EFS transit encryption setting.
  • Fargate tasks use platform-managed local storage for plain volume blocks. Requiring efs_volume_configuration there is a false positive (reported for Fargate local volumes).

Proposed Changes

  • Skip the “missing efs_volume_configuration” branch when requires_compatibilities includes FARGATE.
  • Keep reporting DISABLED / missing transit_encryption whenever efs_volume_configuration is present, including Fargate + EFS.
  • Add negative fixtures for Fargate local volumes and a positive fixture for Fargate + EFS with transit encryption disabled.

How tested

  • Static review of query.rego guard placement and fixture set.
  • Host lacks full KICS/OPA suite; fixtures follow the existing query test layout.

I submit this contribution under the Apache-2.0 license.

Signed-off-by: Alex Chen <l46983284@gmail.com>
…tion

Signed-off-by: Alex Chen <l46983284@gmail.com>
…tion

Signed-off-by: Alex Chen <l46983284@gmail.com>
Signed-off-by: Alex Chen <l46983284@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(terraform): EFS Volume With Disabled Transit Encryption

1 participant