Skip to content
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

Class 'OneLogin_Saml2_Auth' not found #132

Open
cbzink opened this issue Apr 11, 2018 · 9 comments
Open

Class 'OneLogin_Saml2_Auth' not found #132

cbzink opened this issue Apr 11, 2018 · 9 comments

Comments

@cbzink
Copy link

cbzink commented Apr 11, 2018

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
Copy link
Contributor

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
Copy link

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
Copy link
Contributor

@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
Copy link
Contributor

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
Copy link

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
Copy link

dcaswel commented Apr 16, 2018

@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
Copy link

lvidal1 commented Apr 17, 2018

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

@jhoopes
Copy link

jhoopes commented Apr 25, 2018

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

@aacotroneo
Copy link
Owner

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants