From 6f15c52d0e6034d0c337939dad585225b8cecaec Mon Sep 17 00:00:00 2001
From: Matt Leon <ml@mattleon.com>
Date: Thu, 4 Mar 2021 06:59:15 -0500
Subject: [PATCH] fix(522): Serialize and deserialze sessions with email and
 userType

---
 src/server/auth/index.ts | 19 ++++++++++++++++---
 src/server/index.ts      |  2 +-
 2 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/src/server/auth/index.ts b/src/server/auth/index.ts
index 63c0037c2..887bb6657 100644
--- a/src/server/auth/index.ts
+++ b/src/server/auth/index.ts
@@ -1,5 +1,8 @@
 import { Express } from 'express';
 import passport from 'passport';
+import { User, UserType } from '../generated/graphql';
+import { Models } from '../models';
+import { fetchUser } from '../resolvers/helpers';
 
 export interface StrategyNames {
 	displayName: string;
@@ -7,9 +10,19 @@ export interface StrategyNames {
 	scopes: string[];
 }
 
-export const registerAuthRoutes = (app: Express, strategies: StrategyNames[]): void => {
-	passport.serializeUser((user, done) => void done(null, user));
-	passport.deserializeUser((user, done) => void done(null, user));
+export const registerAuthRoutes = (
+	app: Express,
+	strategies: StrategyNames[],
+	models: Models
+): void => {
+	passport.serializeUser(({ email, userType }: User, done) => {
+		void done(null, { email, userType });
+	});
+	passport.deserializeUser(
+		async ({ email, userType }: { email: string; userType: UserType }, done) => {
+			done(null, await fetchUser({ email, userType }, models));
+		}
+	);
 
 	strategies.forEach(({ name: authName, scopes }) => {
 		app.get(`/api/auth/${authName}`, passport.authenticate(authName, { scope: scopes }));
diff --git a/src/server/index.ts b/src/server/index.ts
index 6e8b7cb49..ca18a0b14 100644
--- a/src/server/index.ts
+++ b/src/server/index.ts
@@ -92,7 +92,7 @@ export const schema = makeExecutableSchema({
 		// console.error(config); // sanity check for auth plugin
 	});
 
-	registerAuthRoutes(app, oAuthStrategies);
+	registerAuthRoutes(app, oAuthStrategies, models);
 
 	app.use((req, res, next) =>
 		passport.authenticate(