aerialship / SamlSPBundle

SAML 2.0 Symfony SP Bundle - new version available at
http://www.lightsaml.com/SP-Bundle/
MIT License
63 stars 43 forks source link

Refactor to fos user bundle approach. #81

Closed BernardoSilva closed 9 years ago

BernardoSilva commented 9 years ago

This is inspered on https://github.com/FriendsOfSymfony/FOSUserBundle/tree/master/Doctrine suggested by @stof on https://github.com/aerialship/SamlSPBundle/pull/71#issuecomment-95113848

stof commented 9 years ago

The SSOStateEntity should not be in the Doctrine namespace: it is not tied to the Doctrine ORM at all. And there should be no SSOStateDocument at all. The same class should be used.

BernardoSilva commented 9 years ago

I think you can close this ticket as well @tmilos . With mongoDB PR we ended up refactoring the whole bundle :p