-
-
Notifications
You must be signed in to change notification settings - Fork 203
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Bug Description
In internal/graphql/verify_email.go and internal/graphql/verify_otp.go, there is no check for user.RevokedTimestamp. The Login method correctly checks user.RevokedTimestamp != nil and returns an error, but these verification flows skip this check entirely.
Impact
HIGH — A user whose access was revoked can still verify email/OTP and obtain valid authentication tokens, bypassing the revocation.
Fix
Add if user.RevokedTimestamp != nil { return error } check after fetching user in both files.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working