Skip to content

Conversation

@pushpak1300
Copy link
Member

@pushpak1300 pushpak1300 commented Oct 1, 2025

When we first built the Starter Kits, all authentication logic lived directly within the kit itself. Recently, we integrated Laravel Fortify to handle Two-Factor Authentication (2FA). Fortify is a mature, configurable, and battle tested authentication backend that publishes its own actions making it easier to override and extend.

Since Fortify is already part of the stack for 2FA, standardising the rest of our authentication flow (login, registration, password reset, etc.) on Fortify is the natural next step.

About This Merge Request (Part 1: Login & Logout)

This MR focuses on replacing the Login and Logout logic with Fortify.
The existing views are preserved as they are tightly coupled with Fortify’s flow.

To make reviews easier, the overall feature is divided into four separate MRs.

Related Merge Requests

Browser Test

@pushpak1300 pushpak1300 changed the title Refactor authentication by replacing it with fortify Refactor Login and Logout by replacing it with fortify Oct 1, 2025
@pushpak1300 pushpak1300 force-pushed the replace_login_with_fortify branch from a2cc3c0 to e8582a8 Compare October 11, 2025 07:54
@pushpak1300 pushpak1300 changed the title Refactor Login and Logout by replacing it with fortify Replace Login and Logout by replacing it with fortify Oct 11, 2025
@pushpak1300 pushpak1300 marked this pull request as ready for review October 12, 2025 12:24
@pushpak1300 pushpak1300 force-pushed the replace_login_with_fortify branch from 7aa0ce0 to 27d83d5 Compare October 13, 2025 11:33
@pushpak1300
Copy link
Member Author

pushpak1300 commented Oct 13, 2025

A browser test is failing because the rate limiting behaviour which has slightly changed with fortify. PR for a fix is here https://github.com/laravel-labs/starter-kit-browser-tests/pull/6/files
Screenshot 2025-10-13 at 5 12 50 PM

Copy link
Contributor

@joetannenbaum joetannenbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@joetannenbaum joetannenbaum merged commit 854c5d6 into main Oct 13, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants