VIA-172 AJ/AS Changed the redirected page in middleware when session is not found

Author: ankur-jain-nhs

src/middleware.test.ts

@@ -3,6 +3,7 @@
  */
 
 import { auth } from "@project/auth";
+import { SESSION_LOGOUT_ROUTE } from "@src/app/session-logout/constants";
 import { middleware } from "@src/middleware";
 import { NextRequest } from "next/server";
 
@@ -26,7 +27,7 @@ describe("middleware", () => {
     jest.clearAllMocks();
   });
 
-  it("redirects users without active session to sso-failure page", async () => {
+  it("redirects users without active session to session-logout page", async () => {
     const testUrl = "https://testurl/abc";
     const mockRequest = getMockRequest(testUrl);
 
@@ -36,7 +37,7 @@ describe("middleware", () => {
 
     expect(result.status).toBe(307);
     expect(result.headers.get("Location")).toEqual(
-      `${mockRequest.nextUrl.origin}/sso-failure?error=No%20active%20session%20found`,
+      `${mockRequest.nextUrl.origin}${SESSION_LOGOUT_ROUTE}`,
     );
   });
 

src/middleware.ts

@@ -1,4 +1,5 @@
 import { auth } from "@project/auth";
+import { SESSION_LOGOUT_ROUTE } from "@src/app/session-logout/constants";
 import { Session } from "next-auth";
 import { NextRequest, NextResponse } from "next/server";
 import { logger } from "@src/utils/logger";
@@ -9,12 +10,10 @@ const log: Logger = logger.child({ name: "middleware" });
 export async function middleware(request: NextRequest) {
   log.info(`Inspecting ${request.nextUrl}`);
   const session: Session | null = await auth();
-  if (!session?.user?.birthdate) {
-    return NextResponse.redirect(
-      new URL(`/sso-failure?error=No active session found`, request.url),
-    );
-  }
   log.info(session, "Session Object");
+  if (!session?.user) {
+    return NextResponse.redirect(new URL(SESSION_LOGOUT_ROUTE, request.url));
+  }
   return NextResponse.next();
 }