From 95f731fd59ad905d67d4e6b6ddcf907d424666f8 Mon Sep 17 00:00:00 2001 From: Piotr Karpala Date: Thu, 4 Jan 2024 10:39:43 -0500 Subject: [PATCH 1/3] Update README.md - fix E2E link --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 65ba3e6..32b562e 100644 --- a/README.md +++ b/README.md @@ -1,4 +1,4 @@ -[![Build, test, push to multiple registries and e2e test](https://github.com/Azure/EasyAuthForK8s/actions/workflows/ci.yml/badge.svg)](https://github.com/Azure/EasyAuthForK8s/actions/workflows/ci.yml) +[![Build, test, push to multiple registries and e2e test](https://github.com/Azure/EasyAuthForK8s/actions/workflows/E2E.yml/badge.svg)](https://github.com/Azure/EasyAuthForK8s/actions/workflows/E2E.yml) # EasyAuth for Kubernetes From 977ea72921a64ca37330b60fc2ab94ce942bc933 Mon Sep 17 00:00:00 2001 From: Piotr Karpala Date: Thu, 4 Jan 2024 10:54:21 -0500 Subject: [PATCH 2/3] Update deploy-to-existing-cluster.md fixes to scripts --- docs/deploy-to-existing-cluster.md | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/docs/deploy-to-existing-cluster.md b/docs/deploy-to-existing-cluster.md index 24af372..9c7631d 100644 --- a/docs/deploy-to-existing-cluster.md +++ b/docs/deploy-to-existing-cluster.md @@ -18,6 +18,10 @@ This project has been validated to work on the following: Review these very carefully and modify. + # Set your AKS cluster name and resource group + CLUSTER_NAME=easy-auth-proxy-aks + CLUSTER_RG=easy-auth-proxy-rg + # Set your backend service variables SERVICE= SERVICE_PORT= @@ -97,10 +101,10 @@ EOF cat manifest.json # Create the Azure AD SP for our application and save the Client ID to a variable -CLIENT_ID=$(az ad app create --display-name $AD_APP_NAME --homepage $HOMEPAGE --reply-urls $REPLY_URLS --required-resource-accesses @manifest.json -o json | jq -r '.appId') +CLIENT_ID=$(az ad app create --display-name $AD_APP_NAME --web-home-page-url $HOMEPAGE --web-redirect-uris $REPLY_URLS --required-resource-accesses @manifest.json -o json | jq -r '.appId') echo $CLIENT_ID -OBJECT_ID=$(az ad app show --id $CLIENT_ID -o json | jq '.objectId' -r) +OBJECT_ID=$(az ad app show --id $CLIENT_ID -o json | jq '.id' -r) echo $OBJECT_ID az ad app update --id $OBJECT_ID --set oauth2Permissions[0].isEnabled=false @@ -236,4 +240,4 @@ NOTE: Wait 5 minutes for the easy auth proxy and Azure AD to get up and running. Open your homepage in the browser. You now have Easy Auth applied to your application! -To apply more advanced scenarios to your existing cluster (RBAC, Graph Query, Anonymous, etc), please see sample-ingress.yaml in root > sample > templates > sample-ingress.yaml for examples \ No newline at end of file +To apply more advanced scenarios to your existing cluster (RBAC, Graph Query, Anonymous, etc), please see sample-ingress.yaml in root > sample > templates > sample-ingress.yaml for examples From 9c522af5ff16cf2754fa741afb1e4338ce31d4ff Mon Sep 17 00:00:00 2001 From: Piotr Karpala Date: Fri, 9 Feb 2024 11:42:48 -0500 Subject: [PATCH 3/3] fix: fixing typo in the chart - missing - --- charts/easyauth-proxy/templates/statefulset.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/charts/easyauth-proxy/templates/statefulset.yaml b/charts/easyauth-proxy/templates/statefulset.yaml index b8114b0..252d8b0 100644 --- a/charts/easyauth-proxy/templates/statefulset.yaml +++ b/charts/easyauth-proxy/templates/statefulset.yaml @@ -63,7 +63,7 @@ spec: value: "{{ .Values.easyAuthForK8s.allowBearerToken }}" - name: EasyAuthForK8s__DefaultRedirectAfterSignin value: "{{ .Values.easyAuthForK8s.defaultRedirectAfterSignin }}" - name: EasyAuthForK8s__CompressCookieClaims + - name: EasyAuthForK8s__CompressCookieClaims value: "{{ .Values.easyAuthForK8s.compressCookieClaims }}" - name: EasyAuthForK8s__ResponseHeaderPrefix value: "{{ .Values.easyAuthForK8s.responseHeaderPrefix }}"