Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Confirming 2FA requires additional changues to an external config file #1025

Closed
flick36 opened this issue Mar 30, 2022 · 3 comments
Closed

Comments

@flick36
Copy link

flick36 commented Mar 30, 2022

  • Jetstream Version: 2.7.0
  • Jetstream Stack: Inertia / Livewire
  • Uses Teams: yes / no
  • Laravel Version: 9.6.0
  • PHP Version: 8.1.4
  • Database Driver & Version: Postgre 14.2

Description:

Well me again, I'm sorry. Confirming 2FA #992 it's a nice and really appreciated edition problem is, if we do a simple composer update well it doesn't kind of work because we have to add 'confirm' => true to the fortify config file. it's not a big of a problem (once you found out why the newly addition doesn't work. I'm ashame to admint it took longer than i wished to figure it out) to add a simple line it's not a big deal, the thing is when you do a composer update i believe you are expected to have things broke on the updates of the package you require, but we don't explicitly require 'laravel/fortify' it's required within the package and the new 'laravel/fortify' version adds that line laravel/fortify@0fa359a so, i stil thing it should bea breaking changue update but it's here now, and for the people that don't know why this newly addition isn't working, maybe try and see if you have that line.

Steps To Reproduce:

Install previous jetstream version (2.6.8) do a composer update, and 2FA isn't really working

@driesvints
Copy link
Member

Yeah, you definitely have to do this manually if you want to enable 2FA confirmation. Normally nothing should break though. I can't immediately see this from your explanation but do things break when you simply run composer update? And if so: what error do you get?

@ghost
Copy link

ghost commented Mar 31, 2022

Thoughts on the having to edit the fortify config file or any file that needs manual edits maybe needs to be documented in the UPGRADING.md file as you should not really need to dig through the source code to upgrade between 2.x (MINOR) versions? (I miss the recipes:update from Symfony)

I just upgraded an old Jetstream 2.6 project and I'm unable to reproduce any errors or unexpected behaviour without the 'confirm' => true for 2FA using the Google Authenticator App.

@driesvints
Copy link
Member

Right. I don't think anything will break if you run a composer update.

Listing all upgrade steps between minor versions is an impossible task as there's way too much files involved. It basically boils down to copying the changes from the stubs. Remember that Jetstream is a scaffold and there's no easy way to upgrade to newer functionality in stubs. People might have edited their stubs already so it's much more complicated than just copy/paste.

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

No branches or pull requests

2 participants