search

aacotroneo / laravel-saml2

A Laravel 5 package for Saml2 integration as a SP (service provider) based on the simple OneLogin toolkit
MIT License
566 stars 237 forks source link

Class 'OneLogin_Saml2_Auth' not found #132

Open cbzink opened 6 years ago

cbzink commented 6 years ago

Brand new Laravel app, ran composer require aacotroneo/laravel-saml2:dev-remove_mcrypt, php artisan vendor:publish, and setup the various configurations in config/saml2_settings.php.

Visiting /saml2/metadata results in the following error:

~/vendor/aacotroneo/laravel-saml2/src/Aacotroneo/Saml2/Saml2ServiceProvider.php

Line 78: return new OneLogin_Saml2_Auth($config);

"Class 'OneLogin_Saml2_Auth' not found"

This is using Laravel on macOS with Valet. Any help is appreciated!

danmichaelo commented 6 years ago

First thing to do when you get an error message of type "X not found" is to check if X is there or not ;) Try to search for "class OneLogin_Saml2_Auth" in your "vendor" directory. Hint. If it's there, but not loading, try composer dump-autoload –optimize, or delete the vendor directory and composer install.

marknduncan commented 6 years ago

Having the same issue on the remove-mcrypt branch, seems one of the OneLogin classes is incorrect now. Any chance this can be fixed?

danmichaelo commented 6 years ago

@marknduncan , did you read the previous comment? This doesn't really sound like a problem with aacotroneo/laravel-saml2. If it is, more information is needed to pinpoint the issue.

danmichaelo commented 6 years ago

Aha, now I see that you also posted #133 . Seems like they've changed naming from OneLogin_Saml2_Auth to OneLogin\Saml2\Auth in the 3.0.0 branch.

marknduncan commented 6 years ago

Ah yep sorry, meant to come back here and explain a bit more after I posted that and dug into it a bit more. Appreciate the response.

dcaswel commented 6 years ago

@danmichaelo I am running into this issue as well. Are you aware of someone working on fixing this to work with the current version of 3.0.0 or is everyone waiting for that to become an official release?

lvidal1 commented 6 years ago

Is anyone working on this? This issue is also happening while trying to use the remove_mcrypt branch.

jhoopes commented 6 years ago

Just Made a PR #134 correcting the issue with missing classes.

aacotroneo commented 6 years ago

Thanks @jhoopes I merged it... lets see how it goes.

Once 3.0 is out and ~stable we'll merge all to master and release a new version.