Skip to content

Commit e1a4b15

Browse files
less verbose comment
1 parent f054e81 commit e1a4b15

1 file changed

Lines changed: 1 addition & 8 deletions

File tree

packages/web/src/auth.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -318,14 +318,7 @@ const nextAuthResult = NextAuth(async () => ({
318318
return false;
319319
}
320320

321-
// Reject any sign-in that arrives without an email. `email` is a required
322-
// column, so a null would otherwise fail the `createUser` insert at the
323-
// database; historically these rows also crashed the members list and other
324-
// surfaces that assume an email is present. Returning false surfaces the auth
325-
// error page instead. In practice only OAuth/OIDC profiles can lack an email
326-
// (credentials and email providers always carry one), but the check is left
327-
// unconditional so any future provider or edge case is covered too. `user` is
328-
// always defined in the signIn callback, so it needs no guard.
321+
// Reject any sign-in that arrives without an email.
329322
// @see 20260616000000_make_user_email_required/migration.sql
330323
if (!user.email) {
331324
return false;

0 commit comments

Comments
 (0)