l3-team / CasGuardBundle

PHPCas wrapper for Symfony3, Symfony4, Symfony 5, Symfony 6, Symfony 7
8 stars 4 forks source link

Documentation (README.md) - cas logout : manque une ligne instanciation de phpCas #2

Open MUbedaSJ opened 1 year ago

MUbedaSJ commented 1 year ago

Bonjour,

Et tout d'abord, un grand BRAVO pour ce bundle bien documenté et versionné pour sf5.x et sf6.x Le CAS est une fonctionnalité très utilisée dans le monde universitaire, mais malheureusement peu connu dans le monde du développement en général...

Dans la route "logout" , il manque une ligne d'initialisation/instanciation de phpClient() Sinon une erreur de CAS_OutOfSequenceBeforeClientException apparait : this method cannot be called before phpCAS::client() or phpCAS::proxy()

https://github.com/l3-team/CasGuardBundle/blob/4d52027e4c8c7299c5f4e06b0583032d6724509a/README.md?plain=1#L508

Comme dans la doc, il est préconnisé de définir dans les "parameters" de services.yaml (cas_host, casport, cas...) Voici la ligne que j'ai rajouté dans la route "logout" de mon controller, juste avant l'appel de la fonction "\phpCAS::logoutWithRedirectService(...)" : \phpCAS::client(CAS_VERSION_2_0, $this->getParameter('cas_host'), $this->getParameter('cas_port'), is_null($this->getParameter('cas_path')) ? '' : $this->getParameter('cas_path'), true);

Ligne inspirée/issue de (+ ajout prefix 'cas_' pour les paramètres) https://github.com/l3-team/CasGuardBundle/blob/4d52027e4c8c7299c5f4e06b0583032d6724509a/Security/CasAuthenticator.php#L56

Encore un grand merci ! Good job -) !