From 4517241ead3e47aaf60388f53dda6f8ec42e4f2f Mon Sep 17 00:00:00 2001
From: Darren Clarke <darren@redaranj.com>
Date: Wed, 22 Feb 2023 13:05:52 +0000
Subject: [PATCH] Middleware and login updates

---
 .vscode/swissknifeDecorators.json         |  1 +
 Makefile                                  |  4 +-
 apps/link/.dockerignore                   |  2 +
 apps/link/components/Sidebar.tsx          | 14 +++---
 apps/link/components/ZammadWrapper.tsx    | 14 +++---
 apps/link/middleware.ts                   | 32 +++++++++----
 apps/link/next.config.js                  | 25 +++++-----
 apps/link/pages/admin/zammad.tsx          |  2 +-
 apps/link/pages/api/auth/[...nextauth].ts |  4 +-
 apps/link/pages/index.tsx                 |  2 +-
 apps/link/pages/link.tsx                  |  2 +-
 apps/link/pages/login.tsx                 |  6 ++-
 apps/link/pages/profile.tsx               |  2 +-
 apps/link/pages/tickets/[...id].tsx       | 54 ---------------------
 apps/link/pages/tickets/[id].tsx          | 58 +++++++++++++++++++++++
 apps/link/pages/tickets/assigned.tsx      |  2 +-
 apps/link/pages/tickets/pending.tsx       |  2 +-
 apps/link/pages/tickets/unassigned.tsx    |  2 +-
 apps/link/pages/tickets/urgent.tsx        |  2 +-
 apps/zammad-proxy/.dockerignore           |  2 +
 docker-compose.yml                        | 20 ++++----
 docker/elasticsearch/Dockerfile           |  2 +-
 docker/redis/Dockerfile                   |  2 +-
 23 files changed, 144 insertions(+), 112 deletions(-)
 create mode 100644 .vscode/swissknifeDecorators.json
 create mode 100644 apps/link/.dockerignore
 delete mode 100644 apps/link/pages/tickets/[...id].tsx
 create mode 100644 apps/link/pages/tickets/[id].tsx
 create mode 100644 apps/zammad-proxy/.dockerignore

diff --git a/.vscode/swissknifeDecorators.json b/.vscode/swissknifeDecorators.json
new file mode 100644
index 0000000..0637a08
--- /dev/null
+++ b/.vscode/swissknifeDecorators.json
@@ -0,0 +1 @@
+[]
\ No newline at end of file
diff --git a/Makefile b/Makefile
index 56eafbd..32e9ac6 100644
--- a/Makefile
+++ b/Makefile
@@ -62,8 +62,8 @@ force-run-migrations:
 	docker-compose -p link-docker-compose exec zammad-railsserver bundle exec rails r 'require "/opt/zammad/db/addon/cdr_signal/20210525091356_cdr_signal_channel.rb";require "/opt/zammad/db/addon/cdr_voice/20210525091357_cdr_voice_channel.rb";require "/opt/zammad/db/addon/cdr_whatsapp/20210525091358_cdr_whatsapp_channel.rb"; require "/opt/zammad/db/addon/pgpsupport/20220403000001_pgpsupport.rb";CdrSignalChannel.new.up;CdrVoiceChannel.new.up;CdrWhatsappChannel.new.up;PGPSupport.new.up;'
 
 sso:
-	docker-compose -p link-shell exec zammad-nginx sed -i '/proxy_set_header X-Forwarded-User "";/d' /opt/zammad/contrib/nginx/zammad.conf;
-	docker-compose -p link-shell exec zammad-nginx service nginx restart;
+	docker-compose exec zammad-nginx sed -i '/proxy_set_header X-Forwarded-User "";/d' /opt/zammad/contrib/nginx/zammad.conf;
+	docker-compose exec zammad-nginx service nginx restart;
 
 start:
 	CURRENT_UID=$(CURRENT_UID) docker-compose up -d
diff --git a/apps/link/.dockerignore b/apps/link/.dockerignore
new file mode 100644
index 0000000..b90a368
--- /dev/null
+++ b/apps/link/.dockerignore
@@ -0,0 +1,2 @@
+node_modules
+.next
diff --git a/apps/link/components/Sidebar.tsx b/apps/link/components/Sidebar.tsx
index dd4c216..e155285 100644
--- a/apps/link/components/Sidebar.tsx
+++ b/apps/link/components/Sidebar.tsx
@@ -24,6 +24,7 @@ import { useRouter } from "next/router";
 import Link from "next/link";
 import Image from "next/image";
 import LinkLogo from "public/link-logo-small.png";
+import { useSession } from "next-auth/react";
 
 const openWidth = 270;
 const closedWidth = 100;
@@ -152,7 +153,8 @@ interface SidebarProps {
 
 export const Sidebar: FC<SidebarProps> = ({ open, setOpen }) => {
   const { pathname } = useRouter();
-  const [username, setUsername] = useState("Nicholas Smith");
+  const { data: session } = useSession()
+  const username = session?.user?.name || "User"
 
   return (
     <Drawer
@@ -273,9 +275,9 @@ export const Sidebar: FC<SidebarProps> = ({ open, setOpen }) => {
             {open
               ? username
               : username
-                  .split(" ")
-                  .map((name) => name.substring(0, 1))
-                  .join("")}
+                .split(" ")
+                .map((name) => name.substring(0, 1))
+                .join("")}
           </Typography>
         </Grid>
         <Grid item>
@@ -384,10 +386,10 @@ export const Sidebar: FC<SidebarProps> = ({ open, setOpen }) => {
                 />
                 <MenuItem
                   name="New Ticket UI"
-                  href="/tickets/181"
+                  href="/tickets/2"
                   Icon={SettingsIcon}
                   iconSize={0}
-                  selected={pathname.endsWith("/tickets/181")}
+                  selected={pathname.endsWith("/tickets/2")}
                   open={open}
                 />
               </List>
diff --git a/apps/link/components/ZammadWrapper.tsx b/apps/link/components/ZammadWrapper.tsx
index f9d597e..48924a1 100644
--- a/apps/link/components/ZammadWrapper.tsx
+++ b/apps/link/components/ZammadWrapper.tsx
@@ -2,24 +2,26 @@ import { FC, useState } from "react";
 import Iframe from "react-iframe";
 
 type ZammadWrapperProps = {
-  url: string;
+  path: string;
   hideSidebar?: boolean;
 };
 
 export const ZammadWrapper: FC<ZammadWrapperProps> = ({
-  url,
+  path,
   hideSidebar = true,
 }) => {
-  const [display, setDisplay] = useState("inherit");
-
+  const [display, setDisplay] = useState("hidden");
+  const url = `https://redaranj-bookish-tribble-56jwjx5wh4j4w-8003.preview.app.github.dev/zammad${path}`;
+  console.log({ base: process.env.LINK_URL, path, url })
   return (
-    <Iframe
+    < Iframe
       id="link"
       url={url}
       width="100%"
       height="100%"
       frameBorder={0}
-      styles={{ display }}
+      styles={{ display }
+      }
       onLoad={() => {
         const linkElement = document.querySelector("iframe");
         if (
diff --git a/apps/link/middleware.ts b/apps/link/middleware.ts
index 758fafa..c3d6af6 100644
--- a/apps/link/middleware.ts
+++ b/apps/link/middleware.ts
@@ -1,19 +1,31 @@
 // middleware.ts
 import { NextResponse } from 'next/server'
 import type { NextRequest } from 'next/server'
+import { withAuth } from "next-auth/middleware"
 
-// This function can be marked `async` if using `await` inside
-export function middleware(request: NextRequest) {
+export default withAuth((request: NextRequest) => {
+  if (request.nextUrl.pathname.startsWith('/login')) {
+    return NextResponse.next()
+  }
+
+  // @ts-expect-error
+  if (!request.nextauth.token) {
+    return NextResponse.redirect("/login")
+  }
+  // @ts-expect-error
+  console.log({ token: request.nextauth.token })
   console.log("INTO middleware")
   const path = request.nextUrl.pathname
   console.log({ path })
-  if (path.startsWith('/zammad')) {
-    console.log("INTO middleware 2")
+  const zammadPaths = ['/zammad', '/assets', '/api/v1', '/auth/sso'];
+  if (zammadPaths.some((p) => path.startsWith(p))) {
+    console.log("MATCHED ZAMMAD PATH")
     const finalURL = new URL(path.replace("/zammad", ""), process.env.ZAMMAD_URL)
     console.log(finalURL.toString())
 
-    const requestHeaders = new Headers()
-    requestHeaders.set('X-Forwarded-User', 'darren@redaranj.com')
+    const requestHeaders = new Headers(request.headers)
+    // @ts-expect-error
+    requestHeaders.set('X-Forwarded-User', request.nextauth.token?.email)
     requestHeaders.set('Host', 'zammad.example.com')
 
     console.log(requestHeaders)
@@ -23,7 +35,11 @@ export function middleware(request: NextRequest) {
       }
     })
   } else {
-    console.log("INTO middleware 3")
+    console.log("DID NOT MATCH ZAMMAD PATH")
     return NextResponse.next()
   }
-}
+}, {
+  pages: {
+    signIn: '/login',
+  }
+})
diff --git a/apps/link/next.config.js b/apps/link/next.config.js
index ad9635c..8cb49aa 100644
--- a/apps/link/next.config.js
+++ b/apps/link/next.config.js
@@ -3,6 +3,7 @@ module.exports = {
     locales: ["en", "fr"],
     defaultLocale: "en",
   },
+  /*
   async redirects() {
     return [
       {
@@ -13,15 +14,17 @@ module.exports = {
     ];
   },
   async rewrites() {
-    return [
-      {
-        source: "/zammad",
-        destination: `http://link-shell-zammad-proxy-1:3000`,
-      },
-      {
-        source: "/zammad/:path*",
-        destination: `http://link-shell-zammad-proxy-1:3000/:path*`,
-      },
-    ];
-  },
+    return {
+      beforeFiles: [
+        {
+          source: "/assets/:path*",
+          destination: `/zammad/assets/:path*`,
+        },
+        {
+          source: "/api/v1/:path*",
+          destination: `/zammad/api/v1/:path*`,
+        },
+      ],
+    };
+  }, */
 };
diff --git a/apps/link/pages/admin/zammad.tsx b/apps/link/pages/admin/zammad.tsx
index cc91099..d7dc070 100644
--- a/apps/link/pages/admin/zammad.tsx
+++ b/apps/link/pages/admin/zammad.tsx
@@ -22,7 +22,7 @@ const Link = () => (
         }}
       >
         <ZammadWrapper
-          url="http://localhost:3000/zammad/#manage"
+          path="/#manage"
           hideSidebar={false}
         />
       </Grid>
diff --git a/apps/link/pages/api/auth/[...nextauth].ts b/apps/link/pages/api/auth/[...nextauth].ts
index 98a3d15..939e041 100644
--- a/apps/link/pages/api/auth/[...nextauth].ts
+++ b/apps/link/pages/api/auth/[...nextauth].ts
@@ -1,6 +1,6 @@
 import NextAuth from "next-auth";
 import Google from "next-auth/providers/google";
-import Apple from "next-auth/providers/apple";
+// import Apple from "next-auth/providers/apple";
 
 export default NextAuth({
   providers: [
@@ -8,10 +8,12 @@ export default NextAuth({
       clientId: process.env.GOOGLE_CLIENT_ID,
       clientSecret: process.env.GOOGLE_CLIENT_SECRET,
     }),
+    /*
     Apple({
       clientId: process.env.APPLE_CLIENT_ID,
       clientSecret: process.env.APPLE_CLIENT_SECRET
     }),
+    */
   ],
   secret: process.env.NEXTAUTH_SECRET,
 });
diff --git a/apps/link/pages/index.tsx b/apps/link/pages/index.tsx
index 9d95278..633c23e 100644
--- a/apps/link/pages/index.tsx
+++ b/apps/link/pages/index.tsx
@@ -21,7 +21,7 @@ const Home = () => (
           width: "100%",
         }}
       >
-        <ZammadWrapper url="https://8003-digiresilienc-linkshell-c2tqwgcccbs.ws-eu86.gitpod.io/zammad/#dashboard" />
+        <ZammadWrapper path="/#dashboard" />
       </Grid>
     </Grid>
   </Layout>
diff --git a/apps/link/pages/link.tsx b/apps/link/pages/link.tsx
index 96dd8d5..c7dcfb3 100644
--- a/apps/link/pages/link.tsx
+++ b/apps/link/pages/link.tsx
@@ -21,7 +21,7 @@ const Link = () => (
           width: "100%",
         }}
       >
-        <ZammadWrapper url="http://localhost:3000/zammad/#ticket/zoom/518/1490" />
+        <ZammadWrapper path="/zammad/#ticket/zoom/518/1490" />
       </Grid>
     </Grid>
   </Layout>
diff --git a/apps/link/pages/login.tsx b/apps/link/pages/login.tsx
index ca05eaf..e4d29eb 100644
--- a/apps/link/pages/login.tsx
+++ b/apps/link/pages/login.tsx
@@ -40,13 +40,15 @@ const Login = ({ session }) => {
                     sx={buttonStyles}
                     onClick={() =>
                       signIn("google", {
-                        callbackUrl: `${window.location.origin}/setup`,
+                        callbackUrl: `https://redaranj-bookish-tribble-56jwjx5wh4j4w-8003.preview.app.github.dev/auth/sso`,
                       })
                     }
                   >
                     <GoogleIcon sx={{ mr: 1 }} />
+                    Google
                   </IconButton>
                 </Grid>
+                {/*
                 <Grid item sx={{ width: "100%" }}>
                   <IconButton
                     sx={buttonStyles}
@@ -58,7 +60,7 @@ const Login = ({ session }) => {
                   >
                     <AppleIcon sx={{ mr: 1 }} />
                   </IconButton>
-                </Grid>
+                </Grid>*/}
                 <Grid item sx={{ mt: 2 }} />
               </Grid>
             ) : null}
diff --git a/apps/link/pages/profile.tsx b/apps/link/pages/profile.tsx
index 3a647ec..342a3f6 100644
--- a/apps/link/pages/profile.tsx
+++ b/apps/link/pages/profile.tsx
@@ -22,7 +22,7 @@ const Profile = () => (
         }}
       >
         <ZammadWrapper
-          url="http://localhost:3000/zammad/#profile"
+          path="/#profile"
           hideSidebar={false}
         />
       </Grid>
diff --git a/apps/link/pages/tickets/[...id].tsx b/apps/link/pages/tickets/[...id].tsx
deleted file mode 100644
index a1c1e09..0000000
--- a/apps/link/pages/tickets/[...id].tsx
+++ /dev/null
@@ -1,54 +0,0 @@
-import Head from "next/head";
-import { NextPage, GetServerSideProps, GetServerSidePropsContext } from "next";
-import { Grid, Box } from "@mui/material";
-import { Layout } from "components/Layout";
-import { TicketDetail } from "components/TicketDetail";
-import { TicketEdit } from "components/TicketEdit";
-
-const Link: NextPage = ({ ticket, articles }: any) => (
-  <Layout>
-    <Head>
-      <title>Link Shell</title>
-    </Head>
-    <Grid container spacing={0} sx={{ height: "100vh" }} direction="row">
-      <Grid item sx={{ height: "100vh" }} xs={10}>
-        <TicketDetail ticket={ticket} articles={articles} />
-      </Grid>
-      <Grid item xs={2} sx={{ height: "100vh" }}>
-        <TicketEdit ticket={ticket} />
-      </Grid>
-    </Grid>
-  </Layout>
-);
-
-export const getServerSideProps: GetServerSideProps = async (
-  context: GetServerSidePropsContext
-) => {
-  const {
-    params: { id },
-  } = context;
-  const baseURL = `${process.env.ZAMMAD_URL}/api/v1`;
-  const token = process.env.ZAMMAD_API_TOKEN;
-  const headers = { Authorization: `Token ${token}` };
-  const rawTicket = await fetch(`${baseURL}/tickets/${id}`, {
-    headers,
-  });
-  const ticket = await rawTicket.json();
-  const rawArticles = await fetch(
-    `${baseURL}/ticket_articles/by_ticket/${id}`,
-    {
-      headers,
-    }
-  );
-  const articles = await rawArticles.json();
-
-  console.log({ ticket, articles });
-
-  return {
-    props: {
-      ticket,
-      articles,
-    },
-  };
-};
-export default Link;
diff --git a/apps/link/pages/tickets/[id].tsx b/apps/link/pages/tickets/[id].tsx
new file mode 100644
index 0000000..bed83f4
--- /dev/null
+++ b/apps/link/pages/tickets/[id].tsx
@@ -0,0 +1,58 @@
+import { GetServerSideProps, GetServerSidePropsContext } from "next";
+import Head from "next/head";
+import useSWR from "swr";
+import { NextPage } from "next";
+import { Grid, } from "@mui/material";
+import { Layout } from "components/Layout";
+import { TicketDetail } from "components/TicketDetail";
+import { TicketEdit } from "components/TicketEdit";
+
+type TicketProps = {
+  id: string;
+};
+
+const Ticket: NextPage<TicketProps> = ({ id }) => {
+  const fetcher = async (url: string) => {
+    const res = await fetch(url);
+    return res.json();
+  }
+
+  const { data: ticketData, error: ticketError } = useSWR(
+    `https://redaranj-bookish-tribble-56jwjx5wh4j4w-8003.preview.app.github.dev/api/v1/tickets/${id}`,
+    fetcher
+  );
+
+  const { data: articlesData, error: articlesError } = useSWR(
+    `https://redaranj-bookish-tribble-56jwjx5wh4j4w-8003.preview.app.github.dev/api/v1/ticket_articles/by_ticket/${id}`,
+    fetcher
+  );
+
+  const shouldRender = !ticketError && !articlesError && ticketData && articlesData && !ticketData.error && !articlesData.error;
+
+  return (
+    <Layout>
+      <Head>
+        <title>Link Shell</title>
+      </Head>
+      {shouldRender && (
+        <Grid container spacing={0} sx={{ height: "100vh" }} direction="row">
+          <Grid item sx={{ height: "100vh" }} xs={10}>
+
+            <TicketDetail ticket={ticketData} articles={articlesData} />
+          </Grid>
+          <Grid item xs={2} sx={{ height: "100vh" }}>
+            <TicketEdit ticket={ticketData} />
+          </Grid>
+        </Grid>)}
+      {ticketError && <div>{ticketError.toString()}</div>}
+    </Layout>
+  );
+}
+
+export const getServerSideProps: GetServerSideProps = async (
+  context: GetServerSidePropsContext) => {
+  const { id } = context.query;
+  return { props: { id } };
+}
+
+export default Ticket;
diff --git a/apps/link/pages/tickets/assigned.tsx b/apps/link/pages/tickets/assigned.tsx
index 0b9a554..d31bea7 100644
--- a/apps/link/pages/tickets/assigned.tsx
+++ b/apps/link/pages/tickets/assigned.tsx
@@ -21,7 +21,7 @@ const Link = () => (
           width: "100%",
         }}
       >
-        <ZammadWrapper url="http://localhost:8003/zammad/#ticket/view/my_tickets" />
+        <ZammadWrapper path="/#ticket/view/my_tickets" />
       </Grid>
     </Grid>
   </Layout>
diff --git a/apps/link/pages/tickets/pending.tsx b/apps/link/pages/tickets/pending.tsx
index bf6d563..d543e6e 100644
--- a/apps/link/pages/tickets/pending.tsx
+++ b/apps/link/pages/tickets/pending.tsx
@@ -21,7 +21,7 @@ const Link = () => (
           width: "100%",
         }}
       >
-        <ZammadWrapper url="http://localhost:3000/zammad/#ticket/view/my_pending_reached" />
+        <ZammadWrapper path="/#ticket/view/my_pending_reached" />
       </Grid>
     </Grid>
   </Layout>
diff --git a/apps/link/pages/tickets/unassigned.tsx b/apps/link/pages/tickets/unassigned.tsx
index bf9d85f..92e1fdb 100644
--- a/apps/link/pages/tickets/unassigned.tsx
+++ b/apps/link/pages/tickets/unassigned.tsx
@@ -21,7 +21,7 @@ const Link = () => (
           width: "100%",
         }}
       >
-        <ZammadWrapper url="http://localhost:3000/zammad/#ticket/view/all_unassigned" />
+        <ZammadWrapper path="/#ticket/view/all_unassigned" />
       </Grid>
     </Grid>
   </Layout>
diff --git a/apps/link/pages/tickets/urgent.tsx b/apps/link/pages/tickets/urgent.tsx
index 44eddad..6d9c3e7 100644
--- a/apps/link/pages/tickets/urgent.tsx
+++ b/apps/link/pages/tickets/urgent.tsx
@@ -21,7 +21,7 @@ const Link = () => (
           width: "100%",
         }}
       >
-        <ZammadWrapper url="http://localhost:3000/zammad/#ticket/view/all_escalated" />
+        <ZammadWrapper path="/#ticket/view/all_escalated" />
       </Grid>
     </Grid>
   </Layout>
diff --git a/apps/zammad-proxy/.dockerignore b/apps/zammad-proxy/.dockerignore
new file mode 100644
index 0000000..b90a368
--- /dev/null
+++ b/apps/zammad-proxy/.dockerignore
@@ -0,0 +1,2 @@
+node_modules
+.next
diff --git a/docker-compose.yml b/docker-compose.yml
index 31ae1d6..5c2e43f 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -218,18 +218,14 @@ services:
       ZAMMAD_URL: ${ZAMMAD_URL}
       ZAMMAD_API_TOKEN: ${ZAMMAD_API_TOKEN}
       ZAMMAD_VIRUAL_HOST: ${ZAMMAD_VIRTUAL_HOST}
-
-  zammad-proxy:
-    container_name: zammad-proxy
-    build: ./apps/zammad-proxy
-    expose:
-      - "3000"
-    ports:
-      - "8004:3000"
-    environment:
-      ZAMMAD_URL: ${ZAMMAD_URL}
-      ZAMMAD_API_TOKEN: ${ZAMMAD_API_TOKEN}
-      ZAMMAD_VIRUAL_HOST: ${ZAMMAD_VIRTUAL_HOST}
+      LINK_URL: ${LINK_URL}
+      NEXTAUTH_URL: ${LINK_URL}
+      NEXTAUTH_SECRET: ${NEXTAUTH_SECRET}
+      NEXTAUTH_AUDIENCE: ${NEXTAUTH_AUDIENCE}
+      NEXTAUTH_SIGNING_KEY_B64: ${NEXTAUTH_SIGNING_KEY_B64}
+      NEXTAUTH_ENCRYPTION_KEY_B64: ${NEXTAUTH_ENCRYPTION_KEY_B64}
+      GOOGLE_CLIENT_ID: ${GOOGLE_CLIENT_ID}
+      GOOGLE_CLIENT_SECRET: ${GOOGLE_CLIENT_SECRET}
 
 volumes:
   elasticsearch-data:
diff --git a/docker/elasticsearch/Dockerfile b/docker/elasticsearch/Dockerfile
index b0ad280..cb92a42 100644
--- a/docker/elasticsearch/Dockerfile
+++ b/docker/elasticsearch/Dockerfile
@@ -1 +1 @@
-FROM zammad/zammad-docker-compose:zammad-elasticsearch-5.2.3-32
+FROM zammad/zammad-docker-compose:zammad-elasticsearch-5.2.3-31
diff --git a/docker/redis/Dockerfile b/docker/redis/Dockerfile
index 4e42c3d..c68bf93 100644
--- a/docker/redis/Dockerfile
+++ b/docker/redis/Dockerfile
@@ -1 +1 @@
-FROM redis:6.2.5-alpine
+FROM redis:7.0.5-alpine