scheb / 2fa

Two-factor authentication for Symfony applications 🔐
MIT License
493 stars 72 forks source link

Allow CSRF token to be retrieved via the X-XSRF-TOKEN header #242

Open SherinBloemendaal opened 1 month ago

SherinBloemendaal commented 1 month ago

Description

I am integrating a 2FA flow within a Single Page Application (SPA) and have implemented an endpoint (GET /set_csrf_cookie) similar to Laravel Sanctum. This endpoint sets a CSRF-TOKEN cookie (httpOnly: false + SameSite: LAX) for the current session. Consequently, libraries like Axios and other JS XHR tools automatically include the CSRF token in the X-XSRF-TOKEN header of every request.

While it might initially seem unsafe to store a CSRF token in a cookie, this approach is secure due to the SameSite policy combined with CORS. Since a custom header is used and the content type is application/json, the request is not classified as "simple," mitigating potential risks. Although it is also possible to send the CSRF token within the JSON payload, I think we all prefer a single source of truth, rather than varying the method based on endpoints.

It would be very nice if some extension point or configuration option was being added to specify that the CSRF token should be extracted from the request header. Adding such an extension point or configuration option would significantly simplify this process. If this feature request is approved I can create a PR :)

scheb commented 3 weeks ago

That's outside of the bundle's influence. It is using Symfony's CSRF token manager, specifically the security.csrf.token_manager service. If you want to change CSRF token behaviour in your application, you have to configure it on the framework-level. Please have a look at the official Symfony documentation on CSRF token.

Alternatively, you could inject your own implementation of Symfony\Component\Security\Csrf\CsrfTokenManagerInterface as scheb_two_factor.csrf_token_manager.

SherinBloemendaal commented 3 weeks ago

Thanks for the reply :) I've realised my initial explanation wasn't clear enough. Let me clarify: In TwoFactorFirewallConfig.php::114, the CSRF token is extracted from the request payload, either from JSON or FormData. However, I'm unable to find a way to decorate or modify this behavior without fully cloning TwoFactorAuthenticator.php. Is there an alternative approach to achieve this without duplicating the entire authenticator class?

scheb commented 2 weeks ago

Okay, I see what the problem is. To be honest, I totally forgot this was part of the implementation.

Stupid question, how did you solve this for the initial login? Symfony's login mechanism should have the exact same challenge and I couldn't find anything how to make it read the csrf token from a header.

SherinBloemendaal commented 2 weeks ago

For the initial login, I resolved the issue by creating a custom authenticator that extracts the token from the header and passes it to a CsrfTokenBadge.

I wanted to implement the same approach for the TwoFactorAuthenticator. However, doing so requires removing the entire two_factor configuration from the firewall, which in turn disables prepare_on_login, prepare_on_access_denied and other features.

An alternative solution could be to force the JSON content-type instead, which would eliminate the need for CSRF protection if I am not mistaken, similar to how Symfony's JsonLoginAuthenticator operates.