Merge pull request microsoft#1075 from AbdulMujeeb-2000/dev

fix: Resolve intermittent app settings environment value skipping

Author: Roopan-Microsoft

infra/avm/modules/compute/app-service.bicep

@@ -112,6 +112,7 @@ module appService 'br/public:avm/res/web/site:0.23.1' = {
       healthCheckPath: !empty(healthCheckPath) ? healthCheckPath : null
       webSocketsEnabled: webSocketsEnabled
       appCommandLine: appCommandLine
+      vnetRouteAllEnabled: vnetRouteAllEnabled
       acrUseManagedIdentityCreds: !empty(acrUserManagedIdentityClientId)
       acrUserManagedIdentityID: !empty(acrUserManagedIdentityClientId) ? acrUserManagedIdentityClientId : null
     }
@@ -122,21 +123,6 @@ module appService 'br/public:avm/res/web/site:0.23.1' = {
         properties: appSettings
         applicationInsightResourceId: !empty(applicationInsightResourceId) ? applicationInsightResourceId : null
       }
-      {
-        name: 'logs'
-        properties: {
-          applicationLogs: { fileSystem: { level: 'Verbose' } }
-          detailedErrorMessages: { enabled: true }
-          failedRequestsTracing: { enabled: true }
-          httpLogs: { fileSystem: { enabled: true, retentionInDays: 1, retentionInMb: 35 } }
-        }
-      }
-      {
-        name:'web'
-        properties: {
-          vnetRouteAllEnabled: vnetRouteAllEnabled
-          }
-      }
     ]
     outboundVnetRouting: {
       contentShareTraffic: contentShareTraffic
@@ -159,6 +145,28 @@ module appService 'br/public:avm/res/web/site:0.23.1' = {
   }
 }
 
+// ============================================================================
+// Logs Configuration (deployed serially after appsettings)
+// Deploy logs after app settings to prevent concurrent config writes.
+// ============================================================================
+resource site 'Microsoft.Web/sites@2025-03-01' existing = {
+  name: name
+}
+
+resource logsConfig 'Microsoft.Web/sites/config@2025-03-01' = {
+  parent: site
+  name: 'logs'
+  properties: {
+    applicationLogs: { fileSystem: { level: 'Verbose' } }
+    detailedErrorMessages: { enabled: true }
+    failedRequestsTracing: { enabled: true }
+    httpLogs: { fileSystem: { enabled: true, retentionInDays: 1, retentionInMb: 35 } }
+  }
+  dependsOn: [
+    appService
+  ]
+}
+
 // ============================================================================
 // Outputs
 // ============================================================================
@@ -176,3 +184,4 @@ output appUrl string = 'https://${appService.outputs.defaultHostname}'
 
 @description('System-assigned identity principal ID.')
 output identityPrincipalId string = appService.outputs.?systemAssignedMIPrincipalId ?? ''
+