fix: use reject instead of throw (#1134)

Co-authored-by: pilcrowOnPaper <[email protected]>
lucia-auth · Sep 18, 2023 · 8507a53 · 8507a53
1 parent 13a333a
commit 8507a53
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/.auri/$8145tnre.md b/.auri/$8145tnre.md
@@ -0,0 +1,6 @@
+---
+package: "lucia"
+type: "patch"
+---
+
+Fix unhandled rejection error when using `AuthRequest.validate()` and `AuthRequest.validateBearerToken()`
diff --git a/packages/lucia/src/auth/request.ts b/packages/lucia/src/auth/request.ts
@@ -124,7 +124,7 @@ export class AuthRequest<_Auth extends Auth = any> {
 			debug.request.info("Using cached result for session validation");
 			return this.validatePromise;
 		}
-		this.validatePromise = new Promise(async (resolve) => {
+		this.validatePromise = new Promise(async (resolve, reject) => {
 			if (!this.storedSessionId) return resolve(null);
 			try {
 				const session = await this.auth.validateSession(this.storedSessionId);
@@ -140,7 +140,7 @@ export class AuthRequest<_Auth extends Auth = any> {
 					this.maybeSetSession(null);
 					return resolve(null);
 				}
-				throw e;
+				return reject(e);
 			}
 		});
 
@@ -152,7 +152,7 @@ export class AuthRequest<_Auth extends Auth = any> {
 			debug.request.info("Using cached result for bearer token validation");
 			return this.validatePromise;
 		}
-		this.validatePromise = new Promise(async (resolve) => {
+		this.validatePromise = new Promise(async (resolve, reject) => {
 			if (!this.bearerToken) return resolve(null);
 			try {
 				const session = await this.auth.validateSession(this.bearerToken);
@@ -161,7 +161,7 @@ export class AuthRequest<_Auth extends Auth = any> {
 				if (e instanceof LuciaError) {
 					return resolve(null);
 				}
-				throw e;
+				return reject(e);
 			}
 		});