logued out do not why

[nahuelmellusso]

• Laravel Version: 6.0
• Adldap2-Laravel Version: 6.0
• PHP Version: 7.3
• LDAP Type: OpenLDAP

I am making a login like this :

protected function attemptLogin(Request $request)
    {

        $credentials = $request->only($this->username(), 'password');

        $username = $credentials[$this->username()];
        $password = $credentials['password'];
        $user_format = env('LDAP_USER_FORMAT', 'cn=%s,'.env('LDAP_BASE_DN', ''));
        $userdn = sprintf($user_format, $username);

        if(Adldap::auth()->attempt($userdn, $password, $bindAsUser = true)) {
            // the user exists in the LDAP server, with the provided password

            $user = \App\User::where($this->username(), $username)->first();

            if (!$user) {
                // the user doesn't exist in the local database, so we have to create one

                $user = new \App\User();
                $user->username = $username;
                $user->password = '';

                // you can skip this if there are no extra attributes to read from the LDAP server
                // or you can move it below this if(!$user) block if you want to keep the user always
                // in sync with the LDAP server
                $sync_attrs = $this->retrieveSyncAttributes($username);
                foreach ($sync_attrs as $field => $value) {
                    $user->$field = $value !== null ? $value : '';
                }
            }

            // by logging the user we create the session, so there is no need to login again (in the configured time).
            // pass false as second parameter if you want to force the session to expire when the user closes the browser.
            // have a look at the section 'session lifetime' in `config/session.php` for more options.
            $this->guard()->login($user, false);

            return true;
           dd(auth()->check()) ----> return true
        }

        // the user doesn't exist in the LDAP server or the password is wrong
        // log error
        return false;
    }

but imediatly redirecting to login page and auth()->check() return false.

why ?

[stevebauman]

Hi @nahuelmellusso,

You haven't saved the user you just constructed (look at the last line of code below):

// the user doesn't exist in the local database, so we have to create one

$user = new \App\User();
$user->username = $username;
$user->password = '';

// you can skip this if there are no extra attributes to read from the LDAP server
// or you can move it below this if(!$user) block if you want to keep the user always
// in sync with the LDAP server
$sync_attrs = $this->retrieveSyncAttributes($username);

foreach ($sync_attrs as $field => $value) {
    $user->$field = $value !== null ? $value : '';
}

$user->save(); // Create the user.

This will resolve your issue.

[nahuelmellusso]

but the user exist ! El vie., 15 nov. 2019 a las 17:43, Steve Bauman (<notifications@github.com>) escribió:

…

Hi @nahuelmellusso <https://github.com/nahuelmellusso>, You haven't saved the user you just constructed: // the user doesn't exist in the local database, so we have to create one$user = new \App\User();$user->username = $username;$user->password = '';// you can skip this if there are no extra attributes to read from the LDAP server// or you can move it below this if(!$user) block if you want to keep the user always// in sync with the LDAP server$sync_attrs = $this->retrieveSyncAttributes($username);foreach ($sync_attrs as $field => $value) { $user->$field = $value !== null ? $value : '';}$user->save(); // Create the user. This will resolve your issue. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#811?email_source=notifications&email_token=AB5QKHXHJIWRWFS3XRNA7NTQT4CWVA5CNFSM4JOAALW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEGVDYI#issuecomment-554521057>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB5QKHVQVHLNJ6SVUITKLDLQT4CWVANCNFSM4JOAALWQ> .

[stevebauman]

but the user exist !

Okay, for testing purposes, remove that code block to only allow existing users in your local database to authenticate.

What session driver are you using? If you're using the file session driver, check to see if a file is created in your storage/framework/sessions folder.

[nahuelmellusso]

ok ,
session driver is file ., and session file is created in storage/framework/sessions folder.

this code is never used , because user exist , anyway i have commented

if (!$user) {
                // the user doesn't exist in the local database, so we have to create one

                $user = new \App\User();
                $user->username = $username;
                $user->password = '';

                // you can skip this if there are no extra attributes to read from the LDAP server
                // or you can move it below this if(!$user) block if you want to keep the user always
                // in sync with the LDAP server
                $sync_attrs = $this->retrieveSyncAttributes($username);
                foreach ($sync_attrs as $field => $value) {
                    $user->$field = $value !== null ? $value : '';
                }
            }

but still redirecting to login view , and loging out user :(

[nahuelmellusso]

i resolved with this ->

protected function authenticated(Request $request, $user)
    {
        //
        return redirect()->route('home') ;
    }

thanks for your help !