-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/delete cache #154
Feature/delete cache #154
Conversation
@@ -39,6 +39,8 @@ public function handle(Request $request, Closure $next) | |||
|
|||
$this->loginByUserId($user->id); | |||
} catch (Exception $e) { | |||
Cache::delete($md5Token); | |||
|
|||
Log::error($e->getMessage()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sabe dizer pq só nesse caso se usa Log ao invés de report?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não sei, talvez o @zerossB possa responder essa.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por que isso foi feito a muito tempo atrás e não atualizamos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alterado para o report
Não se esqueça de adicionar as mudanças na release
Qual problema esse Pull Request aborda?
Qual solução foi aplicada?
src/Middleware/AuthenticateHubMiddleware.php
src/Middleware/AuthenticateCheckHubMiddleware.php
Deletar do cache caso caia na exceção.
src/Http/Controllers/Auth/LogoutController.php
Buscar todas as sessões ativas do usuário e montar o md5 para deletar todas as sessões em cache.
Card(s) da tarefa
[Cole aqui o link do card relacionado a esse problema]
Grau de urgência
Esse Pull Request deverá aguardar para ser mergeado?
É necessário executar alguma etapa após o Pull Request ser mergeado?
Footnotes
Utilize grau de urgência baixo para alterações que podem ser revisadas no mesmo dia. ↩
Utilize grau de urgência alto para alterações que devem ser revisadas o quanto antes. ↩
Utilize a opção "Sim" para casos em que o merge das alterações trouxerem algum tipo de risco para a aplicação naquele momento. ↩