Merge pull request microsoft#333 from derkoe/feat/show-only-enabled-auth

Show only enabled authentication mechanisms
drivendataorg · Sep 11, 2024 · f98e27d · f98e27d
2 parents 47f4aa9 + e5d279e
commit f98e27d
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/src/app/page.tsx b/src/app/page.tsx
@@ -5,7 +5,11 @@ export default async function Home() {
   await redirectIfAuthenticated();
   return (
     <main className="container max-w-lg flex items-center">
-      <LogIn isDevMode={process.env.NODE_ENV === "development"} />
+      <LogIn
+        isDevMode={process.env.NODE_ENV === "development"}
+        githubEnabled={!!process.env.AUTH_GITHUB_ID}
+        entraIdEnabled={!!process.env.AZURE_AD_CLIENT_ID}
+      />
     </main>
   );
 }
diff --git a/src/features/auth-page/login.tsx b/src/features/auth-page/login.tsx
@@ -14,6 +14,8 @@ import {
 
 interface LoginProps {
   isDevMode: boolean;
+  githubEnabled: boolean;
+  entraIdEnabled: boolean;
 }
 
 export const LogIn: FC<LoginProps> = (props) => {
@@ -31,13 +33,17 @@ export const LogIn: FC<LoginProps> = (props) => {
         </CardDescription>
       </CardHeader>
       <CardContent className="grid gap-4">
-        <Button onClick={() => signIn("github")}>GitHub</Button>
-        <Button onClick={() => signIn("azure-ad")}> Microsoft 365</Button>
-        {props.isDevMode ? (
+        {props.githubEnabled && (
+          <Button onClick={() => signIn("github")}>GitHub</Button>
+        )}
+        {props.entraIdEnabled && (
+          <Button onClick={() => signIn("azure-ad")}>Microsoft 365</Button>
+        )}
+        {props.isDevMode && (
           <Button onClick={() => signIn("localdev")}>
             Basic Auth (DEV ONLY)
           </Button>
-        ) : null}
+        )}
       </CardContent>
     </Card>
   );