fix(aws): remove cloudwatch_log_group_no_critical_pii_in_logs check

[MrCloudSec]

Description

Remove check cloudwatch_log_group_no_critical_pii_in_logs since presidio-analyzer is downloading a package during Prowler execution.

Checklist

• Are there new checks included in this PR? Yes / No
  • If so, do we need to update permissions for the provider? Please review this carefully.
• Review if the code is being covered by tests.
• Review if code is being documented following this specification https://github.com/google/styleguide/blob/gh-pages/pyguide.md#38-comments-and-docstrings
• Review if backport is needed.

License

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

[jfagoagas]

We are removing the check cloudwatch_log_group_no_critical_pii_in_logs for the following reasons:

• It uses the presidio-analyzer library which by default loads during runtime the NLP module and other PII recognizers from external sources not defined in the Prowler dependencies. That's something we could not do as a security product taking also into account that Prowler can run in environments with little to zero access to the Internet.
• Those dependencies cannot be installed via poetry since they are not available in PyPI and would require adding more steps to our installation procedure to get them.
• The NLP module installed, en-core-web-lg is a large vector table with ~500k entries with a size of 400MiB. This is a huge package to be donwloaded during runtime.
• The installation procedure of the above dependencies adds several lines to our terminal UI with the download process and some logs.
• presidio-analyzer requires a ton of new libraries that we need to review first.

We will work to reintroduce the cloudwatch_log_group_no_critical_pii_in_logs in a way that all dependencies are explicitly defined, required and verified.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.85%. Comparing base (f8e0db4) to head (214e12c).
Report is 1 commits behind head on v4.5.

Additional details and impacted files

@@            Coverage Diff             @@
##             v4.5    #5735      +/-   ##
==========================================
- Coverage   89.89%   89.85%   -0.04%     
==========================================
  Files        1115     1114       -1     
  Lines       34560    34501      -59     
==========================================
- Hits        31067    31002      -65     
- Misses       3493     3499       +6     

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