sandstorm / NeosTwoFactorAuthentication

Extend the Neos Backend Login with 2FA
MIT License
12 stars 8 forks source link

BUGFIX: Autostart session upon authentication #10

Closed bwaidelich closed 1 year ago

bwaidelich commented 2 years ago

Adds a @Flow\Session(autoStart=true) annotation to the OTP authentication action in order to start a session in case it hasn't been started already.

Fixes: #9

Benjamin-K commented 1 year ago

Looks good by reading. Can this be merged? @skurfuerst, @Pingu501 ?

Pingu501 commented 1 year ago

@bwaidelich @Benjamin-K finally looked into this issue. Sadly the provided fix does not resolve the problem. We had a deeper look into it and I think we have an idea what it is about.

Benjamin-K commented 1 year ago

This should now be fixed by #14 i think. So i would close this PR.