fix pageMiddleware receiving a commonContext, not the user context (#…

…8125)
keystonejs · Nov 27, 2022 · 1bdfe37 · 1bdfe37 · vercel · Nov 27, 2022
1 parent f911ce5
commit 1bdfe37
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/core/src/lib/server/createAdminUIMiddleware.ts b/packages/core/src/lib/server/createAdminUIMiddleware.ts
@@ -31,7 +31,7 @@ function defaultIsAccessAllowed({ session, sessionStrategy }: KeystoneContext) {
 
 export function createAdminUIMiddlewareWithNextApp(
   config: KeystoneConfig,
-  context: KeystoneContext,
+  commonContext: KeystoneContext,
   nextApp: NextApp
 ) {
   const handle = nextApp.getRequestHandler();
@@ -55,8 +55,8 @@ export function createAdminUIMiddlewareWithNextApp(
         return;
       }
 
-      const userContext = await context.withRequest(req, res);
-      const isValidSession = await isAccessAllowed(userContext); // TODO: rename "isValidSession" to "wasAccessAllowed"?
+      const context = await commonContext.withRequest(req, res);
+      const isValidSession = await isAccessAllowed(context); // TODO: rename "isValidSession" to "wasAccessAllowed"?
       const shouldRedirect = await pageMiddleware?.({
         context,
         isValidSession,