Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(aws): update EKS check in compliance frameworks #5672

Merged
merged 3 commits into from
Nov 7, 2024

Conversation

MrCloudSec
Copy link
Member

@MrCloudSec MrCloudSec commented Nov 7, 2024

Context

The check eks_control_plane_endpoint_access_restricted was recently renamed to eks_cluster_not_publicly_accessible. This update ensures that all compliance frameworks are consistent with the latest names.

Description

This PR updates the check name eks_control_plane_endpoint_access_restricted and eks_endpoints_not_publicly_accessible to eks_cluster_not_publicly_accessible in the following compliance frameworks:

  • aws_well_architected_framework_security_pillar_aws
  • kisa_isms_p_2023_korean_aws
  • kisa_isms_p_2023_aws
  • nist_csf_1.1_aws
  • nist_800_171_revision_2_aws

Checklist

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@MrCloudSec MrCloudSec requested review from a team as code owners November 7, 2024 13:56
Copy link

codecov bot commented Nov 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.86%. Comparing base (bf4db86) to head (5636e5c).
Report is 785 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5672      +/-   ##
==========================================
- Coverage   89.87%   89.86%   -0.02%     
==========================================
  Files        1119     1119              
  Lines       34724    34724              
==========================================
- Hits        31209    31204       -5     
- Misses       3515     3520       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jfagoagas jfagoagas merged commit aa79a28 into master Nov 7, 2024
11 of 12 checks passed
@jfagoagas jfagoagas deleted the PRWLR-5223-update-renamed-eks-check-in-frameworks branch November 7, 2024 14:56
@prowler-bot prowler-bot added the was-backported The PR was successfully backported to the target branch label Nov 7, 2024
@prowler-bot
Copy link
Collaborator

💚 All backports created successfully

Status Branch Result
v4.5

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@jfagoagas jfagoagas added the backport-to-v4.5 Backport PR to the v4.5 branch label Dec 5, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
backport-to-v4.5 Backport PR to the v4.5 branch was-backported The PR was successfully backported to the target branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants