Symfony中SchebTwoFactorBundle自定义2FA触发条件方法不兼容问题

错误原因 #

你编写的shouldPerformTwoFactorAuthentication方法参数与TwoFactorConditionInterface接口定义的方法签名不匹配。该接口要求方法仅接受AuthenticationContextInterface $context这一个参数，而你额外添加了ManagerRegistry和UserInterface参数，导致类型兼容错误。

修正方案 #

1. 对齐方法签名：严格按照接口定义，方法仅保留AuthenticationContextInterface $context参数
2. 通过构造函数注入依赖：如果需要使用Doctrine，将ManagerRegistry注入到类的构造函数中，而非方法参数
3. 直接从Context获取用户：AuthenticationContextInterface已经包含认证后的用户对象，无需再次查询数据库

修改后的代码示例 #

<?php

namespace App\Security\TwoFactor;

use Scheb\TwoFactorBundle\Security\TwoFactor\Condition\TwoFactorConditionInterface;
use Scheb\TwoFactorBundle\Security\Authentication\AuthenticationContextInterface;
use Doctrine\Persistence\ManagerRegistry;
use App\Entity\User;

class TwoFactorCondition implements TwoFactorConditionInterface
{
    private $doctrine;

    // 构造函数注入ManagerRegistry
    public function __construct(ManagerRegistry $doctrine)
    {
        $this->doctrine = $doctrine;
    }

    public function shouldPerformTwoFactorAuthentication(AuthenticationContextInterface $context): bool
    {
        // 从Context获取当前认证用户
        $user = $context->getUser();
        
        // 确保用户是自定义的User实体类型
        if (!$user instanceof User) {
            return false;
        }

        // 根据用户的2FA启用状态决定是否触发认证
        // 这里根据实际需求调整返回逻辑，示例为：当Google 2FA启用时触发2FA
        return $user->isIsGoogleEnabled();
    }
}

关键说明 #

• 接口的方法签名是强制要求，必须严格遵守，否则会触发兼容性错误
• $context->getUser()返回的是已经通过身份验证的用户实例，直接调用其方法即可，无需重复查询数据库
• 类型判断$user instanceof User可以避免非预期的类型错误，提升代码健壮性
• 你原代码中无论条件如何都返回false，意味着永远不会触发2FA，需根据业务需求调整返回值

内容的提问来源于stack exchange，提问作者ProgTeam10