Skip to content

Default secure values for storage accounts#920

Merged
AlexanderSehr merged 27 commits intomainfrom
users/elanzelm/security
Jan 25, 2022
Merged

Default secure values for storage accounts#920
AlexanderSehr merged 27 commits intomainfrom
users/elanzelm/security

Conversation

@elanzel
Copy link
Copy Markdown
Contributor

@elanzel elanzel commented Jan 22, 2022

Change

  • added requireInfrastructureEncryption in deploy.bicep
  • Updated parameters descriptions for requireInfrastructureEncryption, privateEndpoints, networkAcls
  • set parameters for privateEndpoints, networkAcls, requireInfrastructureEncryption on parameters.json

Storage: StorageAccounts

close #770

Type of Change

  • New feature (non-breaking change which adds functionality)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

elenabatanero and others added 25 commits December 13, 2021 19:55
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 24, 2022

Unit Test Results

    1 files    1 suites   2m 32s ⏱️
  42 tests 42 ✔️     0 💤 0
375 runs  42 ✔️ 333 💤 0

Results for commit 0e8998f.

♻️ This comment has been updated with latest results.

@AlexanderSehr AlexanderSehr merged commit 52c2bed into main Jan 25, 2022
@AlexanderSehr AlexanderSehr deleted the users/elanzelm/security branch January 25, 2022 13:34
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.

Set default values to a secure value - Storage Account

5 participants