From 2b35a4c8e2a9d2e734177b8b105330ce5c7d5195 Mon Sep 17 00:00:00 2001
From: Cyrus Goh <hello@cyrusgoh.com>
Date: Wed, 13 Nov 2024 10:50:46 -0800
Subject: [PATCH] rework sso, sso directly with email

---
 web-auth/src/components/Auth.svelte    | 28 +++++------
 web-auth/src/components/SSOForm.svelte | 70 --------------------------
 2 files changed, 12 insertions(+), 86 deletions(-)
 delete mode 100644 web-auth/src/components/SSOForm.svelte

diff --git a/web-auth/src/components/Auth.svelte b/web-auth/src/components/Auth.svelte
index dc4a3265d6b0..cef04da75082 100644
--- a/web-auth/src/components/Auth.svelte
+++ b/web-auth/src/components/Auth.svelte
@@ -9,7 +9,6 @@
   import EmailPasswordForm from "./EmailPasswordForm.svelte";
   import { getConnectionFromEmail } from "./utils";
   import OrSeparator from "./OrSeparator.svelte";
-  import SSOForm from "./SSOForm.svelte";
   import EmailSubmissionForm from "./EmailSubmissionForm.svelte";
   import Disclaimer from "./Disclaimer.svelte";
   import Spacer from "./Spacer.svelte";
@@ -75,28 +74,30 @@
     webAuth = new auth0.WebAuth(authOptions);
   }
 
+  function authorizeSSO(email: string, connectionName: string) {
+    webAuth.authorize({
+      connection: connectionName,
+      login_hint: email,
+      prompt: "login",
+    });
+  }
+
   function processEmailSubmission(event) {
     email = event.detail.email;
 
     const connectionName = getConnectionFromEmail(email, connectionMapObj);
 
     if (connectionName) {
-      step = AuthStep.SSO;
+      authorizeSSO(email, connectionName);
     } else {
       step = AuthStep.Login;
     }
   }
 
   function getHeadingText(step: AuthStep): string {
-    if (isLegacy) {
-      return "Log in";
-    }
-
     switch (step) {
       case AuthStep.Base:
         return "Log in or sign up";
-      case AuthStep.SSO:
-        return "Log in with SSO";
       case AuthStep.Login:
         return "Log in with email";
       case AuthStep.SignUp:
@@ -107,19 +108,15 @@
         return "";
     }
   }
-  $: headingText = getHeadingText(step);
 
   function getSubheadingText(step: AuthStep, email: string): string {
     switch (step) {
-      case AuthStep.SSO:
-        return `SAML SSO enabled workspace is associated with <span class="font-medium">${email}</span>`;
       case AuthStep.Login:
         return `Log in using <span class="font-medium">${email}</span>`;
       default:
         return "";
     }
   }
-  $: subheadingText = getSubheadingText(step, email);
 
   function backToBaseStep() {
     step = AuthStep.Base;
@@ -128,6 +125,9 @@
   onMount(() => {
     initConfig();
   });
+
+  $: headingText = getHeadingText(step);
+  $: subheadingText = getSubheadingText(step, email);
 </script>
 
 <AuthContainer>
@@ -169,10 +169,6 @@
       <EmailSubmissionForm on:submit={processEmailSubmission} />
     {/if}
 
-    {#if step === AuthStep.SSO}
-      <SSOForm {email} {connectionMapObj} {webAuth} on:back={backToBaseStep} />
-    {/if}
-
     {#if step === AuthStep.Login || step === AuthStep.SignUp}
       <EmailPasswordForm
         {step}
diff --git a/web-auth/src/components/SSOForm.svelte b/web-auth/src/components/SSOForm.svelte
deleted file mode 100644
index f355d25eb0d5..000000000000
--- a/web-auth/src/components/SSOForm.svelte
+++ /dev/null
@@ -1,70 +0,0 @@
-<script lang="ts">
-  import { createEventDispatcher } from "svelte";
-  import CtaButton from "@rilldata/web-common/components/calls-to-action/CTAButton.svelte";
-  import { getConnectionFromEmail } from "./utils";
-  import { ArrowLeftIcon } from "lucide-svelte";
-  import { WebAuth } from "auth0-js";
-
-  const dispatch = createEventDispatcher();
-
-  export let disabled = false;
-  export let email = "";
-  export let webAuth: WebAuth;
-  export let connectionMapObj: Record<string, string[]>;
-
-  let errorText = "";
-
-  function handleClick() {
-    void authorizeSSO(email.toLowerCase());
-  }
-
-  function displayError(err: any) {
-    errorText = err.message;
-  }
-
-  function authorizeSSO(email: string) {
-    disabled = true;
-    errorText = "";
-
-    const connectionName = getConnectionFromEmail(email, connectionMapObj);
-
-    if (!connectionName) {
-      displayError({
-        message: `IDP for the email ${email} not found. Please contact your administrator.`,
-      });
-      disabled = false;
-      return;
-    }
-
-    webAuth.authorize({
-      connection: connectionName,
-      login_hint: email,
-      prompt: "login",
-    });
-  }
-</script>
-
-<div class="flex flex-col gap-y-4">
-  <CtaButton {disabled} variant="primary" on:click={handleClick}>
-    <div class="flex justify-center font-medium">
-      <span>Continue with SAML SSO</span>
-    </div>
-  </CtaButton>
-  <CtaButton
-    {disabled}
-    variant="secondary"
-    gray
-    on:click={() => {
-      dispatch("back");
-    }}
-  >
-    <div class="flex justify-center items-center font-medium">
-      <ArrowLeftIcon class="mr-1" size={14} />
-      <span>Back</span>
-    </div>
-  </CtaButton>
-</div>
-
-{#if errorText}
-  <div class="mt-2 text-red-500 text-sm">{errorText}</div>
-{/if}