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

[New control] GR7V1 and GR9V4 | Validation for storage account TLS version #275

Merged
merged 21 commits into from
Nov 25, 2024

Conversation

dutt0
Copy link
Contributor

@dutt0 dutt0 commented Nov 20, 2024

Overview/Summary

This new control determines whether every storage account is using a minimum of TLS 1.2. If all storage accounts have TLS 1.2+ than pass. If one of more storage accounts have TLS 1.1 or below the check will be non-compliant and return the list of storage accounts that do not meet this requirement.

This PR fixes/adds/changes/removes

This pull request creates two new control for GR7V1 and GR9V4. It checks all storage accounts for TLS version and determines the compliance status as per the requirement of this validation

Breaking Changes

N/A

Testing Evidence

Positive testing:
image

Negative testing:
image

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.
  • Ensure PowerShell module versions have been updated (manually or with the ./tools/Update-ModuleVersions.ps1 script)

@dutt0 dutt0 changed the title GR7V1 control for storage acc TLS version [New control] GR7V1 and GR9V4 | Validation for storage account TLS version Nov 20, 2024
@dutt0 dutt0 marked this pull request as ready for review November 22, 2024 20:30
@dutt0 dutt0 merged commit a96f2ab into main Nov 25, 2024
4 checks passed
@dutt0 dutt0 deleted the idutta/GR7_v1_TLS_storage_acccount branch November 25, 2024 18:57
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.

GR9 | Validation 4 | Storage Accounts TLS 1.2 (M) GR7 | Validation 1 | Storage Accounts TLS 1.2 (M)
3 participants