Keep original error in AuthorizationError#cause (#204)

src/error.ts

@@ -1 +1,5 @@
-export class AuthorizationError extends Error {}
+export class AuthorizationError extends Error {
+  constructor(message?: string, public cause?: Error) {
+    super(message);
+  }
+}

src/strategy.ts

@@ -117,11 +117,12 @@ export abstract class Strategy<User, VerifyOptions> {
     message: string,
     request: Request,
     sessionStorage: SessionStorage,
-    options: AuthenticateOptions
+    options: AuthenticateOptions,
+    cause?: Error
   ): Promise<never> {
     // if a failureRedirect is not set, we throw a 401 Response or an error
     if (!options.failureRedirect) {
-      if (options.throwOnError) throw new AuthorizationError(message);
+      if (options.throwOnError) throw new AuthorizationError(message, cause);
       throw json<{ message: string }>({ message }, 401);
     }
 

test/authenticator.test.ts

@@ -3,7 +3,12 @@ import {
   redirect,
   SessionStorage,
 } from "@remix-run/server-runtime";
-import { AuthenticateOptions, Authenticator, Strategy } from "../src";
+import {
+  AuthenticateOptions,
+  Authenticator,
+  AuthorizationError,
+  Strategy,
+} from "../src";
 
 class MockStrategy<User> extends Strategy<User, Record<string, never>> {
   name = "mock";
@@ -19,7 +24,8 @@ class MockStrategy<User> extends Strategy<User, Record<string, never>> {
       "Invalid credentials",
       request,
       sessionStorage,
-      options
+      options,
+      new Error("Invalid credentials")
     );
   }
 }
@@ -90,6 +96,7 @@ describe(Authenticator, () => {
       expect(strategy).toBe("mock");
     }
   });
+
   test("should store the provided strategy name in the session", async () => {
     let user = { id: "123" };
     let session = await sessionStorage.getSession();
@@ -190,4 +197,25 @@ describe(Authenticator, () => {
       ).rejects.toEqual(response);
     });
   });
+
+  describe("authenticate", () => {
+    test("should throw an error if throwOnError is enabled", async () => {
+      let request = new Request("/");
+      let authenticator = new Authenticator(sessionStorage);
+
+      authenticator.use(new MockStrategy(async () => null));
+
+      let error = await authenticator
+        .authenticate("mock", request, {
+          throwOnError: true,
+        })
+        .catch((error) => error);
+
+      expect(error).toEqual(new AuthorizationError("Invalid credentials"));
+
+      expect((error as AuthorizationError).cause).toEqual(
+        new TypeError("Invalid credentials")
+      );
+    });
+  });
 });