GithubHelp home page GithubHelp logo

Comments (4)

scheb avatar scheb commented on June 3, 2024

I don't see an easy way to do this. By design in Symfony security only one authentication provider is executed in a request.

What you'd normally do when you have an API with 2fa, after you've identified the user via username/password, your API returns a response telling the frontend that 2fa is required. You can do this by implementing your own success handler and setting it for the username/password authentication. Then let the frontend show the 2fa form. And then in a second request execute 2fa process.

from two-factor-bundle.

friedrichroell avatar friedrichroell commented on June 3, 2024

Thanks @scheb for the quick response!

I was thinking, as you are decorating the AuthenticationProviderInterface::authenticate() method, for a "single request" 2fa authentication you could skip IS_AUTHENTICATED_2FA_IN_PROGRESS and run the 2fa validation process immediately and then continue with normal authentication?

from two-factor-bundle.

scheb avatar scheb commented on June 3, 2024

That was something I was thinking as well, but it would only do a part of the job. You'd miss all the logic that is implemented in TwoFactorListener, which is mostly events and the trusted feature. And calling another listener within the same request is not something Symfony's Firewall wouldn't do:

https://github.com/symfony/symfony/blob/f64f59a9c0d92fdd65f9de3e44b612402b224aaf/src/Symfony/Component/Security/Http/Firewall.php#L110-L116

With the new "authenticator" system, that was introduced in Symfony 5.1, things look a bit different. But that system cannot be considered stable (yet) and your version is 4.4 anyways.

Besides that, there'd be a problem with the preparation of the two-factor authentication method. While Google Authenticator doesn't need any prepartation work to be done, other methods might need it (such as SMS or email). So there is no universal solution to this problem.

So many things that'd need to be considered and I cannot offer a solution to this - at least not right now.

from two-factor-bundle.

friedrichroell avatar friedrichroell commented on June 3, 2024

Ok, that makes sense. Thanks for clearing that up :)

I guess I might just move from json_login, create a custom guard firewall and implement a simple 2fa solution in there for now.

Cheers

from two-factor-bundle.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.