fix: oauth should be independent from allowPasswordAuth [#349]

ytsaurus · Feb 14, 2024 · 17f3cdf · 17f3cdf
1 parent f204319
commit 17f3cdf
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/packages/ui/src/server/index.ts b/packages/ui/src/server/index.ts
@@ -21,14 +21,24 @@ nodekit.ctx.log('AppConfig details', {
     appDevMode,
 });
 
-const {allowPasswordAuth} = nodekit.config;
+const {allowPasswordAuth, ytOAuthSettings} = nodekit.config;
+
+const authMiddlewares = [];
+
+if (ytOAuthSettings) {
+    authMiddlewares.push(authorizationResolver(createOAuthAuthorizationResolver()));
+}
 
 if (allowPasswordAuth) {
+    authMiddlewares.push(authorizationResolver(createYTAuthorizationResolver()));
+}
+
+if (authMiddlewares.length) {
     nodekit.config.appBeforeAuthMiddleware = [
         ...(nodekit.config.appBeforeAuthMiddleware || []),
-        authorizationResolver(createOAuthAuthorizationResolver()),
-        authorizationResolver(createYTAuthorizationResolver()),
+        ...authMiddlewares,
     ];
+
     nodekit.config.appAuthHandler = createAuthMiddleware();
 }