Skip to content

fix: email re-verification, OAuth collision, token blacklist, RolesGuard tests - #1223

Open
temiport25 wants to merge 1 commit into
CodeGirlsInc:mainfrom
temiport25:feat/temiport25-auth-security-improvements
Open

fix: email re-verification, OAuth collision, token blacklist, RolesGuard tests#1223
temiport25 wants to merge 1 commit into
CodeGirlsInc:mainfrom
temiport25:feat/temiport25-auth-security-improvements

Conversation

@temiport25

Copy link
Copy Markdown

Closes #1133
Closes #1132
Closes #1131
Closes #1130

…ard tests

- Reset isVerified on email change and send verification email
- Handle OAuth email collision with password-based accounts
- Add access token blacklist on logout via AuthService
- Check token blacklist in JwtStrategy.validate
- Add POST /auth/logout endpoint
- Create roles.guard.spec.ts with multi-role coverage
- Create auth.service.spec.ts with OAuth and logout tests
- Add plaintext fallback to email templates
@drips-wave

drips-wave Bot commented Aug 25, 2026

Copy link
Copy Markdown

@temiport25 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@vercel

vercel Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

@temiport25 is attempting to deploy a commit to the Mftee's projects Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant