From aa764988a0c96b133adf7e8960fe10d4a92fba4f Mon Sep 17 00:00:00 2001
From: Vibhuti Sharma <vibsharm@microsoft.com>
Date: Wed, 25 Aug 2021 16:01:26 +0530
Subject: [PATCH] addressing MR comment

---
 devops/azure/chart/templates/deployment.yaml           |  2 ++
 .../src/main/resources/application.properties          | 10 ++++++----
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/devops/azure/chart/templates/deployment.yaml b/devops/azure/chart/templates/deployment.yaml
index 9cad45fe5..db75c9bb5 100644
--- a/devops/azure/chart/templates/deployment.yaml
+++ b/devops/azure/chart/templates/deployment.yaml
@@ -90,3 +90,5 @@ spec:
           value: "api://$(aad_client_id)"
         - name: azure_activedirectory_session_stateless
           value: "true"
+        - name: azure_istioauth_enabled
+          value: "true"
\ No newline at end of file
diff --git a/provider/partition-azure/src/main/resources/application.properties b/provider/partition-azure/src/main/resources/application.properties
index a047bd97f..a259ca4b7 100644
--- a/provider/partition-azure/src/main/resources/application.properties
+++ b/provider/partition-azure/src/main/resources/application.properties
@@ -11,10 +11,12 @@ logging.slf4jlogger.enabled=true
 logging.mdccontext.enabled=true
 
 # AAD properties
-azure.activedirectory.app-resource-id=${aad_client_id}
-azure.activedirectory.client-id=${aad_client_id}
-azure.activedirectory.AppIdUri=api://${azure.activedirectory.client-id}
-azure.activedirectory.session-stateless=true
+# Azure AD configuration, commented below settings to disable AAD AuthN,
+# Uncomment it in the Istio AUTHN disabled Scenario
+#azure.activedirectory.app-resource-id=${aad_client_id}
+#azure.activedirectory.client-id=${aad_client_id}
+#azure.activedirectory.AppIdUri=api://${azure.activedirectory.client-id}
+#azure.activedirectory.session-stateless=true
 
 # Istio
 azure.istio.auth.enabled=${azure_istioauth_enabled}
-- 
GitLab