StorageAccount: Cleanup + add publicNetworkAccess parameter

[Marius Storhaug (MariusStorhaug)]

Change

Clean-up Storage Account properties/variables and add some missing properties

Problem statement:
The requiredInfrastructureEncryption looked good, but didn't work. We loose tool support when authoring (intellisense and linter) by having nested vars. It didn't show a warning when in a var, but bringing it in to the resource declaration and under property, it showed the issue.

Adding:

• publicNetworkAccess
• move enableInfrastructureEncryption (was placed wrong and was not correctly configuring the setting before)
• ...

Type of Change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update (Wiki)

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

[github-actions]

Unit Test Results

    1 files    1 suites   2m 28s ⏱️
  42 tests 42 ✔️     0 💤 0 ❌
386 runs  42 ✔️ 344 💤 0 ❌

Results for commit b054921.

♻️ This comment has been updated with latest results.