Voter non reconnu

Bonjour,

J'aurais besoin d'un œil extérieur.

Dans l'idée de mettre en place des sécurités d'accès sur certains objets, j'ai donc créé un voter. L'idée de base est de pouvoir choisir les rôles qui auront accès à l'objet lors de la création ou de l'édition de celui-ci.

Le hic, c'est que c'est comme si mon voter n'existait pas, en cela que je ne passe apparemment pas ni dans la méthode supports(…), ni dans voteOnAttribute(…), alors que le service est bien instancié.

Est-ce que quelqu'un pourrait me dire qu'est-ce qui cloche ?

Merci d'avance 

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34

<?php
namespace My\WonderfulBundle\Security;

use Symfony\Component\Security\Core\Authorization\Voter\Voter;
use Symfony\Component\Security\Core\Authorization\AccessDecisionManagerInterface;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use My\WonderfulBundle\Entity\Access;

class AccessVoter extends Voter
{
    private $decisionManager;

    public function __construct(AccessDecisionManagerInterface $decisionManager)
    {
        $this->decisionManager = $decisionManager;
    }

    protected function voteOnAttribute($attribute, $subject, TokenInterface $token)
    {
        if ($this->decisionManager->decide($token, $subject->getRoles())) {
            return true;
        }
    }

    protected function supports($attribute, $subject)
    {
        // only vote on Access objects inside this voter
        if (!$subject instanceof Access) {
            return false;
        }

        return true;
    }
}

Tu peux nous donner la configuration, histoire qu'on vérifie ensemble ?

Elle doit ressembler à ça:

1
2
3
4
5
6
7

services:
    app.access_voter:
        class: My\WonderfullBundle\Security\AccessVoter
        arguments: ['@security.access.decision_manager']
        public: false
        tags:
            - { name: security.voter }

Tu as bien mis arguments ?

J'ai bien mis arguments avec le même service, je n'ai peut-être juste pas mis que le service n'était pas public, mais j'imagine que c'est un détail (je n'ai plus le code sous la main en ce moment).

Vu que la classe est instanciée, on est OK de ce côté-là.

Je n'ai pas l'impression de voir quelque chose de faux ni dans ton voteur, ni dans la conf. Du coup j'ai une autre question, comment l'invoques-tu ? via isGranted/denyAccessUnlessGranted, ou is_granted dans un Template ? Comment es-tu sûr que la classe est instanciée mais que la méthode support n'est pas appelée ? (Tiens d'ailleurs, ça me fait me demander, Est-ce qu'il y a mieux pour débugger ce genre de trucs que de faire du die à l'ancienne ?)

Je souhaite utiliser isGranted('unAttributAuHasardParceQuIlEnFautUnApparemment', $lObjetAccess) dans mon contrôleur, l'idée étant de jeter une 403 à l'accès en édition et/ou à l'affichage.
En fait, je finis par me demander s'il ne manque pas une autre partie de configuration ou que simplement mon appel avec un attribut bidon ne fait pas tout foirer, parce que si j'active les lignes qui font cette vérification, je n'ai plus possibilité de visualiser ou d'éditer les détails de n'importe-quel objet, j'ai une 403 qui sort, même si je suis censé avoir les droits.
Autre élément parce que je ne peux pas trop demander sans tout fournir : les rôles enregistrés dans l'objet sont vides par défaut, je devrais peut-être mettre au moins IS_AUTHENTICATED_FULLY, je rajouterai cet élément "par défaut" demain.

Pour savoir que la classe est bien instanciée et les deux méthodes pas appelées, j'utilise xdebug et je fais une exécution pas à pas dans Eclipse, avec des points d'arrêt dans les méthodes que j'aimerais voir appelées. Le script s'arrête donc bien dans le constructeur, mais ne passe jamais dans les deux méthodes, les deux points d'arrêt n'étant jamais rencontrés.

Je me disais bien que c'était très probablement une stupidité : le tag n'était pas bon, reste d'un copié-collé…

Heureusement que j'ai eu tout le week-end pour celle-là 

Merci à tous !

J'attends d'avoir testé avant, au cas où je pourrai le recycler, mais ce ne sera pas nécessaire, apparemment.

(topic solved man !)