app-admin-ui: Support auth strategies that have loginPath property, c…

…loses keystonejs#2658
zenflow · Apr 5, 2020 · f06410a · f06410a
1 parent 10aa5ee
commit f06410a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/.changeset/empty-emus-lay.md b/.changeset/empty-emus-lay.md
@@ -0,0 +1,5 @@
+---
+'@keystonejs/app-admin-ui': minor
+---
+
+Support auth strategies that have `loginPath` property
diff --git a/packages/app-admin-ui/index.js b/packages/app-admin-ui/index.js
@@ -26,8 +26,8 @@ class AdminUIApp {
       throw new Error("Admin path cannot be the root path. Try; '/admin'");
     }
 
-    if (authStrategy && authStrategy.authType !== 'password') {
-      throw new Error('Keystone 5 Admin currently only supports the `PasswordAuthStrategy`');
+    if (authStrategy && authStrategy.authType !== 'password' && !authStrategy.loginPath) {
+      throw new Error('Keystone 5 Admin does not support the given auth strategy');
     }
 
     this.adminPath = adminPath;
@@ -139,6 +139,10 @@ class AdminUIApp {
       if (req.user) {
         return res.redirect(req.query.redirect || '/');
       }
+      if (this.authStrategy && this.authStrategy.loginPath) {
+        const query = req.url.split('?')[1] || '';
+        return res.redirect(`${this.authStrategy.loginPath}?${query}`);
+      }
       next();
     });