laravel / jetstream

Tailwind scaffolding for the Laravel framework.
https://jetstream.laravel.com
MIT License
3.96k stars 810 forks source link

Support confirmPassword false for 2FA #1464

Closed Suven closed 6 months ago

Suven commented 6 months ago

Fortify, as well as the Jetstreams 2FA component class has code that supports skipping the password confirmation, if set in the according configuration via a feature-flag.

The current implementation of the livewire view however, does not respect that config.

This admittedly naive approach fixes this.

The feature is especially useful for applications that don't make use of passwords (for example because they are using business SSO but still want to enforce 2FA)

This adresses issue #1463

taylorotwell commented 6 months ago

I don't think we ever really meant this to be disabled. It was only added for backwards compatibility at the time. I suggest just removing stuff from your stubs if you want to disable it.