Skip to content

Commit 545285c

Browse files
committed
Merge branch 'develop'
2 parents 8baaf45 + c65db75 commit 545285c

File tree

1 file changed

+17
-4
lines changed

1 file changed

+17
-4
lines changed

‎cicd/bicep/app.bicep‎

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -310,14 +310,27 @@ resource containerApp 'Microsoft.App/containerApps@2024-02-02-preview' = {
310310
resource storageAccount 'Microsoft.Storage/storageAccounts@2023-05-01' = {
311311
name: replace(toLower(prefix), '-', '')
312312
location: location
313-
tags: tags
313+
tags: shallowMerge([
314+
tags
315+
// Ignoring "Secure PaaS Resources" (see: https://eng.ms/docs/cloud-ai-platform/azure-core/azure-networking/sdn-dbansal/azure-virtual-network-manager/nsp-dataplane-library/articles/ns22tsg/ns22tsgoverview)
316+
{
317+
SecurityControl: 'Ignore'
318+
}
319+
])
314320
sku: {
315321
name: 'Standard_ZRS'
316322
}
317323
kind: 'StorageV2'
318324
properties: {
319-
allowSharedKeyAccess: false
325+
// Force using Entra ID authentication
326+
defaultToOAuthAuthentication: true // Entra ID authorization in the Azure portal
327+
isLocalUserEnabled: false // Disable access keys
328+
// Secure transfers
329+
minimumTlsVersion: 'TLS1_2'
320330
supportsHttpsTrafficOnly: true
331+
// Internet access is required to publish the front-end binaries to the container
332+
// Note: Remediation rule "Secure PaaS Resources" requires to be disabled
333+
publicNetworkAccess: 'Enabled'
321334
}
322335
}
323336

@@ -811,8 +824,8 @@ resource cosmos 'Microsoft.DocumentDB/databaseAccounts@2024-05-15' = {
811824
consistencyPolicy: {
812825
defaultConsistencyLevel: 'Strong'
813826
}
814-
databaseAccountOfferType: 'Standard'
815-
disableLocalAuth: true
827+
databaseAccountOfferType: 'Standard' // Required for serverless
828+
disableLocalAuth: true // Disable access keys
816829
locations: [
817830
{
818831
locationName: location

0 commit comments

Comments
 (0)