Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat(compliance): add ens for azure and gcp #5746

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

pedrooot
Copy link
Member

Description

This pull request introduces support for Azure ENS compliance output in the Prowler tool. The main changes include adding a new class for Azure ENS compliance, updating the main script to handle Azure ENS, and modifying the ENS compliance handling to be more generic.

Key changes:

Support for Azure ENS Compliance:

  • New Azure ENS Compliance Class:

    • Added AzureENS class to handle Azure ENS compliance output in prowler/lib/outputs/compliance/ens/ens_azure.py. This class includes methods for transforming findings into the Azure ENS compliance format and writing them to a file.
  • Azure ENS Model:

    • Introduced AzureENSModel class in prowler/lib/outputs/compliance/ens/models.py to generate findings' output in CSV ENS format for Azure.

Main Script Updates:

  • Import Azure ENS:

    • Imported the AzureENS class in the main script prowler/__main__.py.
  • Handle Azure ENS in Main Function:

    • Updated the prowler function in prowler/__main__.py to handle Azure ENS compliance output generation.

General ENS Compliance Handling:

  • Generic ENS Compliance Check:
    • Modified the get_ens_table function in prowler/lib/outputs/compliance/ens/ens.py to handle ENS compliance checks generically, without specifying the provider.

Checklist

License

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

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.

1 participant