Email verification #635
Merged
Email verification #635
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Implement Laravel's MustVerifyEmail, verification routes, and protected middleware to ensure users verify their email before accessing protected routes. Improves security and prevents fake sign-ups. Implement email.verified middleware (EnsureEmailIsVerified Middleware) Updated the test to factor in email verification
syncing local branch with remote upstream repo
bhimbho
requested changes
Feb 28, 2025
- Catch InvalidSignatureException - to check for Invalid/Expired Verification Link - Ensure expired or invalid verification links return a proper JSON response
bhimbho
approved these changes
Mar 1, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR ensures that users must have a verified email before performing certain actions by enforcing email verification checks in middleware. It also updates test cases to properly handle email verification when creating users.
Related Issue (Link to Github issue)
Feat #620
Motivation and Context
Additional Security and Authentication Layer -> This PR addresses that by ensuring users have the
email_verified_atfield set. Additionally, it reinforces middleware checks for verified users.
How Has This Been Tested?
email_verified_atset.-Modified previous test to factor in the new Email Verification system. This test include:
Types of changes
Checklist: