aacotroneo/laravel-saml2

Laravel 8

tomasdelvechio opened this issue · 8 comments

tomasdelvechio commented

I could use this library with laravel 8. It could be clarified in the README that the support is for all Laravel versions.

yasharrow commented

I could use this library with laravel 8. It could be clarified in the README that the support is for all Laravel versions.

@tomasdelvechio Did you get any error like below? in Laravel8 + PHP8.0

image
image

tomasdelvechio commented

@yasharrow I dont remember any error, but I check this tomorrow

yasharrow commented

@yasharrow I dont remember any error, but I check this tomorrow

@tomasdelvechio Did you get a chance to check on above?

tomasdelvechio commented

@tomasdelvechio Did you get a chance to check on above?

Sorry, I forgot to comment. I don't get this error in my installation.

hackerkok commented

@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?

tomasdelvechio commented

@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.

sandervdo commented

@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.

hackerkok commented

@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.
(Sorry for the late reply)

similar issue here #39 (comment)

  • 👍1