Add default logout URI if non defined in config (#1310)

vmware-tanzu · Nov 22, 2019 · dc8b01a · dc8b01a
1 parent cc9c2d6
commit dc8b01a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/dashboard/src/shared/Auth.test.ts b/dashboard/src/shared/Auth.test.ts
@@ -170,5 +170,11 @@ describe("Auth", () => {
       expect(mockedAssign).toBeCalledWith(logoutURI);
       expect(mockedLocalStorageRemove).toBeCalled();
     });
+
+    it("defaults to the oauth2-proxy logout URI", () => {
+      Auth.unsetAuthCookie({ logoutURI: "", namespace: "ns", appVersion: "2" });
+
+      expect(mockedAssign).toBeCalledWith("/oauth2/sign_out");
+    });
   });
 });
diff --git a/dashboard/src/shared/Auth.ts b/dashboard/src/shared/Auth.ts
@@ -28,7 +28,7 @@ export class Auth {
     // JS, so force browser to load the sign-out URI (which expires the
     // session cookie).
     localStorage.removeItem(AuthTokenOIDCKey);
-    document.location.assign(config.logoutURI);
+    document.location.assign(config.logoutURI || "/oauth2/sign_out");
   }
 
   public static usingOIDCToken() {