Skip to content

Commit 862dd91

Browse files
committed
Improve handling of Auth0 login failures
1 parent 9aa1bfe commit 862dd91

File tree

1 file changed

+22
-2
lines changed

1 file changed

+22
-2
lines changed

api/src/auth0.ts

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,14 +86,34 @@ export const loginWithPasswordlessCode = withRetries('loginPWL', async (email: s
8686
email: email,
8787
code: code,
8888
scope: 'openid email offline_access app_metadata'
89-
})).data
89+
})).data,
90+
{
91+
shouldThrow: (e) => {
92+
console.log(`PWL login failed with status ${e.statusCode}: ${e.message}`);
93+
94+
// Don't retry request errors (e.g. auth failure) - return them directly
95+
if (e?.statusCode >= 400 && e?.statusCode < 500) {
96+
return new StatusError(401, "Login failed")
97+
} else return undefined;
98+
}
99+
}
90100
);
91101

92102
export const refreshToken = withRetries('refreshToken', async (refreshToken: string) =>
93103
(await authClient.oauth.refreshTokenGrant({
94104
grant_type: 'refresh_token',
95105
refresh_token: refreshToken
96-
})).data
106+
})).data,
107+
{
108+
shouldThrow: (e) => {
109+
console.log(`Refresh token failed with status ${e.statusCode}: ${e.message}`);
110+
111+
// Don't retry request errors (e.g. auth failure) - return them directly
112+
if (e?.statusCode >= 400 && e?.statusCode < 500) {
113+
return new StatusError(401, "Token refresh failed")
114+
} else return undefined;
115+
}
116+
}
97117
)
98118

99119
export type User = auth0.GetUsers200ResponseOneOfInner & {

0 commit comments

Comments
 (0)