search

swooletw / laravel-swoole

High performance HTTP server based on Swoole. Speed up your Laravel or Lumen applications.
MIT License
4.04k stars 390 forks source link

Clear auth instance on every requests #507

Closed ad-mos closed 2 years ago

ad-mos commented 2 years ago

Currenly auth instance is setting after swoole reload/restart, so user auth shares across all other requests.

It's rollback for commit 3a93e92c8b8ad5eed8bbdf4398faa22bdef48b4f by @Arkanius Putting Auth instance in pre_resolved array makes it the same instance between different user requests. Tested on clean Laravel 8 install.

This commit fixes: 1) Logout user after swoole reload 2) Share auth instance across all requests

Take a look into documentation: image

And into code of SwooleTW\Http\Concerns\WithApplication

image

Arkanius commented 2 years ago

So, when you tested you got the same user at every requests?

ad-mos commented 2 years ago

Yes, because Auth is pre-resolved instance currently.

grvoyt commented 2 years ago

please accept this pull request. Its resolve problem with auth last user on every requests.

grvoyt commented 2 years ago

@

So, when you tested you got the same user at every requests?

I have the same problem. The first logged-in user for each request is stored in memory

Arkanius commented 2 years ago

Sorry for my late response guys. Have being really busy.

Please, if you could, check it out on master to see if your issue is solved.

I'll wait your responses to release the new version