-
Notifications
You must be signed in to change notification settings - Fork 237
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Laravel 8 #236
Comments
@tomasdelvechio Did you get any error like below? in Laravel8 + PHP8.0 |
@yasharrow I dont remember any error, but I check this tomorrow |
@tomasdelvechio Did you get a chance to check on above? |
Sorry, I forgot to comment. I don't get this error in my installation. |
@tomasdelvechio Trying to use the library with Laravel 8 too. Everything works, but session does not persist after user login in the event listener. Did you have any similar issues? How did you solve them? |
I didn't keep testing this library. I had only tested that it worked in Laravel 8 and returned the correct user after the SAML process. In fact I don't know how to link that with a Laravel session. |
@hackerkok have you checked this part of the README file? I forgot to do that step for some projects, resulting in the problem you describe. |
My problem was that I was not storing the user in the db. After storing the logged in user, my problem was solved. similar issue here #39 (comment) |
I could use this library with laravel 8. It could be clarified in the README that the support is for all Laravel versions.
The text was updated successfully, but these errors were encountered: