Merge pull request #24 from Azure-Samples/force_app_diag

Force App Service Diagnostic Logging
Azure-Samples · May 6, 2024 · eeea67b · eeea67b
2 parents 9a0284a + 27fed13
commit eeea67b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/.cruft.json b/.cruft.json
@@ -1,6 +1,6 @@
 {
   "template": "https://github.com/Azure-Samples/Azure-Python-Standardization-Template-Generator",
-  "commit": "071985944385a0faa5d5510469b94eac92339d1f",
+  "commit": "a8bbfb5f41e1b73129167cd91c50671ad604e206",
   "checkout": null,
   "context": {
     "cookiecutter": {

diff --git a/infra/core/host/appservice.bicep b/infra/core/host/appservice.bicep
@@ -17,8 +17,6 @@ param runtimeName string
 param runtimeNameAndVersion string = '${runtimeName}|${runtimeVersion}'
 param runtimeVersion string
 
-param enableDiagnosticLogging bool = true
-
 // Microsoft.Web/sites Properties
 param kind string = 'app,linux'
 
@@ -61,9 +59,9 @@ resource appService 'Microsoft.Web/sites@2022-03-01' = {
         allowedOrigins: union([ 'https://portal.azure.com', 'https://ms.portal.azure.com' ], allowedOrigins)
       }
       // Diagnostic logging
-      detailedErrorLoggingEnabled: enableDiagnosticLogging
-      httpLoggingEnabled: enableDiagnosticLogging
-      requestTracingEnabled: enableDiagnosticLogging
+      detailedErrorLoggingEnabled: true
+      httpLoggingEnabled: true
+      requestTracingEnabled: true
     }
     clientAffinityEnabled: clientAffinityEnabled
     httpsOnly: true