php - 只是检查 : abstain vote from Symfony AbstractVoter isGranted

Question

我正在开发一个 Symfony2 应用程序，我们想在其中引入 Security Voters . DX 计划(jay!)给我们带来了the simpler version这种机制，所以我想使用它。关于 \Symfony\Component\Security\Core\Authorization\Voter\AbstractVoter 的使用，唯一让我有点困扰的是，当我只实现抽象 isGranted 方法。我想为我们的应用程序更改它，但非常感谢您从安全角度对此事提出的意见。

明确地说，在我们的应用程序中，我们将使用unanimous 访问决策管理器策略 - 简而言之:您至少需要一个 ACCESS_GRANTED 而不需要 ACCESS_DENIED - 因为我们稍后可能想介绍禁止 IP 地址或其他恶作剧的选民。

如下面的相关代码所示，只要 AbstractVoter 的实现支持某个属性，投票就会默认为 Denied。

public function vote(TokenInterface $token, $object, array $attributes)
{
    if (!$object || !$this->supportsClass(get_class($object))) {
        return self::ACCESS_ABSTAIN;
    }

    // abstain vote by default in case none of the attributes are supported
    $vote = self::ACCESS_ABSTAIN;

    foreach ($attributes as $attribute) {
        if (!$this->supportsAttribute($attribute)) {
            continue;
        }

        // as soon as at least one attribute is supported, default is to deny access
        $vote = self::ACCESS_DENIED;

        if ($this->isGranted($attribute, $object, $token->getUser())) {
            // grant access as soon as at least one voter returns a positive response
            return self::ACCESS_GRANTED;
        }
    }

    return $vote;
}

我想做的是用下面的代码覆盖它

public function vote(TokenInterface $token, $object, array $attributes)
{
    if (!$object || !$this->supportsClass(get_class($object))) {
        return self::ACCESS_ABSTAIN;
    }

    $vote = self::ACCESS_ABSTAIN;

    foreach ($attributes as $attribute) {
        if (!$this->supportsAttribute($attribute)) {
            continue;
        }

        // This is where we differ from SymfonyAbstractVoter. Only if there is an outspoken yes or no, a vote is cast
        // When returning null, it will still abstain
        $grant = $this->isGranted($attribute, $object, $token->getUser());
        if ($grant === true) {
            return self::ACCESS_GRANTED;
        }
        if($grant === false) {
            return self::ACCESS_DENIED;
        }

    }

    return $vote;
}

你怎么看？当从 isGranted 返回 null 时返回 ACCESS_ABSTAIN 是否安全，并且如果投票者只施放 ACCESS_GRANTED 或 ACCESS_DENIED返回真还是假？

---

我为什么要这样做？仍然使用 AbstractVoter(因为，是的，我是一个懒惰的开发人员)但要分开我的关注点。

假设我为一个帖子设置了一个投票者，PostVoter，这将允许我编辑我自己的帖子。然后它将转换 ACCESS_GRANTED。但只要我不拥有该帖子，这个投票者就真的不在乎:它应该只投一个 ACCESS_ABSTAIN。稍后它可能会遇到 AdminVoter，它仍然会转换 ACCESS_GRANTED。但是，当使用 unanimous 访问决策管理器策略时，如果此 PostVoter 已经投了一个 ACCESS_DENIED，它将永远不会到达任何其他选民。

我现在无法在 AbstractVoter 中转换 ACCESS_ABSTAIN 而不重写 vote 方法以及我脑海中简单选民的全部观点好吧，是为了让它更简单 :)

Answer 1

AbstractVoter 类是大多数用例的简单 Bootstrap 实现，它模拟了 internal 的实现方式。 voters behave ，所以一旦您觉得您的解决方案与基本情况不同，您应该可以随意使用您自己的方法进行重写。