ImpersonateManager does not leave the impersonation

[kejedi]

I'm on Laravel 11

When I click Leave in the banner, nothing happens.

I've narrowed it down to the ImpersonateManager leave method not working.

Which as to do with the package this package uses.

[kejedi]

For now I've just overwritten the route to log the user right out:

Route::get('filament-impersonate/leave', function () {
    Auth::logout();

    return redirect('/login');
})->name('filament-impersonate.leave');

[kejedi]

A better workaround for now:

Route::middleware(config('filament-impersonate.leave_middleware'))
    ->get('filament-impersonate/leave', function () {
        $manager = app('impersonate');

        if ($manager->isImpersonating()) {
            $user = $manager->findUserById($manager->getImpersonatorId());

            Auth::quietLogout();

            Auth::quietLogin($user);

            $manager->clear();
        }

        return redirect('/');
    })
    ->name('filament-impersonate.leave');

[AurelDemiri]

I have this issue as well using Laravel 11.

What "fixed" it for me was removing my global scopes (not what I wanted to do) so I just added a check for the leave route:

    protected static function booted(): void
    {
        // Workaround for filament impersonate leave bug.
        if (request()?->route()?->getName() !== 'filament-impersonate.leave') {
            static::addGlobalScope(new ViewableByCurrentUser());
        }
    }

[manbhatia]

I have this issue as well using Laravel 11.

What "fixed" it for me was removing my global scopes (not what I wanted to do) so I just added a check for the leave route:

    protected static function booted(): void
    {
        // Workaround for filament impersonate leave bug.
        if (request()?->route()?->getName() !== 'filament-impersonate.leave') {
            static::addGlobalScope(new ViewableByCurrentUser());
        }
    }

Worked for me as well. Thanks!

[jszobody]

Ah, global scopes would make sense. Interesting.

Any thoughts on how this package could provide an option or hook for this? I'm wondering if the leave route should use withoutGlobalScope() somewhere.