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

Add Azure Cloud assessments for azure defender fetcher #66

Merged
merged 1 commit into from
Feb 9, 2023

Conversation

wenli200133
Copy link
Contributor

What

Add functionality of fetching assessments, sub-assessments and assessment metadata from Microsoft(Azure) Defender for Cloud.

Why

The assessments, sub-assessments and assessment metadata from Microsoft(Azure) Defender for Cloud can be used as evidence. For example the Azure user can use them to check healthy and unhealthy resources defined in Azure cloud and vulnerability issues in Azure Container Registry.

How

The fetchers in this PR get assessments, sub-assessments and assessment metadata list with Azure Cloud rest API and a valid Azure SPN.

Test

Provide a bulleted list of tests included in the pull request and/or tests
performed to validate the work included in the pull request.

Context

Provide a bulleted list of GitHub issues, or any other references (mailing list discussion, etc...) that reviewers can reference
for additional information regarding scope of the pull request.

@wenli200133 wenli200133 marked this pull request as ready for review February 7, 2023 13:08
@@ -13,4 +13,4 @@
# limitations under the License.
"""Arboretum - Checking your compliance & security posture, continuously."""

__version__ = "0.16.0"
__version__ = "0.16.1"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a new feature so I would bump it to 0.17.0

@@ -0,0 +1,4 @@
"""Common Azure constants."""

# AZURE management resource base url
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for this comment.

cletomartin
cletomartin previously approved these changes Feb 8, 2023
@cletomartin cletomartin merged commit 47472be into ComplianceAsCode:main Feb 9, 2023
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.

2 participants