fix(identity): remember para logins com OAuth

[joaopabdala]

O login com qualquer OAuth (Discord/GitHub/Twitch) não guardava a informação de login porque a variável remember não era passada no controller. Assim a sessão expirava com base no SESSION_LIFETIME — não sei qual o valor em prod, mas no .env.example está 120, ou seja, 2 horas após o login — e o usuário era deslogado, com o painel pedindo para reconectar o Discord.

Resolvi isso passando o remember como true por padrão no OAuth

• Auth::login($result->user);

• Auth::login($result->user, remember: true);

Testei localmente deletar a tabela de sessões
(php artisan tinker --execute="DB::table('sessions')->delete();") para reproduzir as informações do perfil desaparecendo, mas não consegui reproduzir essa parte. Ainda assim, o remember por padrão deve resolver esse comportamento.

Closes #409

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro

Run ID: f9fd4e64-2fb8-4866-b807-d5d674a863f5

📥 Commits

Reviewing files that changed from the base of the PR and between 81045cc and 36fd210.

📒 Files selected for processing (1)

• app-modules/identity/src/Auth/Http/Controllers/OAuthController.php

---

📝 Walkthrough

Walkthrough

OAuthController::getAuthenticate() now calls Auth::login() with the remember option enabled, making OAuth-authenticated sessions persistent.

Suggested reviewers: danielhe4rt

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title matches the main change: enabling remember-me for OAuth logins.
Description check	✅ Passed	The description accurately describes the OAuth remember-me fix and its intent.
Linked Issues check	✅ Passed	The change directly addresses the reported session loss by making OAuth logins persistent with remember-me.
Out of Scope Changes check	✅ Passed	The PR only changes OAuth login persistence and adds no unrelated code.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[YuriSouzaDev]

LGTM!

[davicbtoliveira]

LGTM

[GustavoSimao]

L G T M!!!

[sirelves]

O fluxo OAuth não oferece uma opção de "Lembrar de mim". Sem remember, a autenticação depende exclusivamente da sessão e o usuário precisa autenticar novamente após o SESSION_LIFETIME. Como o objetivo desse fluxo é manter o usuário autenticado entre sessões, usar remember: true torna o comportamento consistente com a expectativa de um login via OAuth, portanto, LGTM!

[Danilo-Sam]

LGTM