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.

[github-actions]

You can check the documentation for this PR here -> Prowler Documentation

[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.

[github-actions]

You can check the documentation for this PR here -> Prowler Documentation

[github-actions]

You can check the documentation for this PR here -> Prowler Documentation

[github-actions]

You can check the documentation for this PR here -> Prowler Documentation

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.91%. Comparing base (c434181) to head (1688232).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5736      +/-   ##
==========================================
+ Coverage   89.88%   89.91%   +0.03%     
==========================================
  Files        1128     1127       -1     
  Lines       35182    35123      -59     
==========================================
- Hits        31622    31580      -42     
+ Misses       3560     3543      -17     

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