search

stof / StofDoctrineExtensionsBundle

Integration bundle for DoctrineExtensions by l3pp4rd in Symfony
https://symfony.com/bundles/StofDoctrineExtensionsBundle/current/index.html
MIT License
1.89k stars 379 forks source link

BlameListener should set user on login #374

Open mathroc opened 6 years ago

mathroc commented 6 years ago

BlameListener should listen to Symfony\Component\Security\Http\SecurityEvents::INTERACTIVE_LOGIN at least and the set the user when someone login

otherwise entities create on registration or login can't have blameable fields

I also listen to FOS\UserBundle\FOSUserEvents::SECURITY_IMPLICIT_LOGIN

alternatively, maybe Gedmo\Blameable\BlameableListener should be overwritten and injectd with the token storage and use that in getFieldValue

barat commented 6 years ago

I second this. Was trying to sort it out, and found this issue.