test: playwright - OpenID auth flow test

packages/nc-gui/middleware/auth.global.ts

@@ -51,7 +51,7 @@ export default defineNuxtRouteMiddleware(async (to, from) => {
     await tryGoogleAuth(api, state.signIn)
   }
 
-  await tryShortTokenAuth(api, state.signIn)
+  if (!state.signedIn.value) await tryShortTokenAuth(api, state.signIn)
 
   /** if public allow all visitors */
   if (to.meta.public) return
@@ -166,9 +166,7 @@ async function tryShortTokenAuth(api: Api<any>, signIn: Actions['signIn']) {
     let extraProps: any = {}
     try {
       // `extra` prop is used in our cloud implementation, so we are keeping it
-      const {
-        data,
-      } = await api.instance.post(
+      const { data } = await api.instance.post(
         `/auth/long-lived-token`,
         {},
         {
@@ -178,11 +176,8 @@ async function tryShortTokenAuth(api: Api<any>, signIn: Actions['signIn']) {
         },
       )
 
-      console.log(data)
-      debugger
       const { token, extra } = data
 
-
       // if extra prop is null/undefined set it as an empty object as fallback
       extraProps = extra || {}
 

packages/nocodb/src/strategies/jwt.strategy.ts

@@ -14,7 +14,9 @@ export class JwtStrategy extends PassportStrategy(Strategy) {
   }
 
   async validate(req, jwtPayload) {
-    if (!jwtPayload?.email) return jwtPayload;
+    if (!jwtPayload?.email) {
+      return jwtPayload
+    }
 
     const user = await User.getByEmail(jwtPayload?.email);
 

tests/playwright/pages/Account/Authentication.ts

@@ -92,25 +92,36 @@ export class AccountAuthenticationPage extends BasePage {
     });
   }
 
-  async createOIDCProvider(p: {
-    title: string;
-    clientId: string;
-    clientSecret: string;
-    authUrl: string;
-    userInfoUrl: string;
-    tokenUrl: string;
-    jwkUrl: string;
-    scopes: Array<string>;
-    userAttributes: string;
-  }) {
+  async createOIDCProvider(
+    p: {
+      issuer: string;
+      title: string;
+      clientId: string;
+      clientSecret: string;
+      authUrl: string;
+      userInfoUrl: string;
+      tokenUrl: string;
+      jwkUrl: string;
+      scopes: Array<string>;
+      userAttributes: string;
+    },
+    setupRedirectUrlCbk?: ({ redirectUrl: string }) => Promise<void>
+  ) {
     const newOIDCBtn = this.get().locator('[data-test-id="nc-new-oidc-provider"]');
 
     await newOIDCBtn.click();
 
     const oidcModal = this.accountPage.rootPage.locator('.nc-oidc-modal');
 
+    if (setupRedirectUrlCbk) {
+      const redirectUrl = (await oidcModal.locator('[data-test-id="nc-openid-redirect-url"]').textContent()).trim();
+      await setupRedirectUrlCbk({ redirectUrl });
+    }
+
     await oidcModal.locator('[data-test-id="nc-oidc-title"]').fill(p.title);
 
+    await oidcModal.locator('[data-test-id="nc-oidc-issuer"]').fill(p.issuer);
+
     await oidcModal.locator('[data-test-id="nc-oidc-client-id"]').fill(p.clientId);
 
     await oidcModal.locator('[data-test-id="nc-oidc-client-secret"]').fill(p.clientSecret);