Skip to content

Auth: indexar e rotacionar sessoes de refresh - #47

Merged
pedrobragabes merged 2 commits into
mainfrom
codex/issue-27-indexed-refresh
Jul 16, 2026
Merged

Auth: indexar e rotacionar sessoes de refresh#47
pedrobragabes merged 2 commits into
mainfrom
codex/issue-27-indexed-refresh

Conversation

@pedrobragabes

Copy link
Copy Markdown
Collaborator

Resumo

  • troca a busca O(N) por uma sessao unica com selector indexado por usuario
  • armazena somente SHA-256 do segredo, expiração e aplica rotacao CAS com deteccao de reuso
  • revoga logout mesmo com access corrompido, bloqueia o fallback em producao e documenta o rollout aditivo

Verificacao local

  • 41 testes determinísticos em 8 suites
  • Prisma validate
  • build + frontend:check
  • npm audit --omit=dev: 0 vulnerabilidades
  • revisao paralela de concorrencia, Prisma e rollout sem P1/P2

A suite integrada com PostgreSQL fica a cargo do GitHub Actions antes do merge.

Closes #27

@pedrobragabes

Copy link
Copy Markdown
Collaborator Author

CI #29519080413: 135/136 testes passaram. O unico erro vinha da ordem dos dois Set-Cookie de migracao: o jar do Superagent tratou a limpeza do path legado, emitida por ultimo, como remocao do refresh atual.

Corrigido em 346f854: limpeza legada primeiro, cookie atual por ultimo, selecao explicita do cookie nao vazio nos testes e regressao unitaria da ordem. Validacao local apos o ajuste: 42 testes deterministicos verdes em 8 suites. Aguardando a nova execucao PostgreSQL.

@pedrobragabes
pedrobragabes marked this pull request as ready for review July 16, 2026 17:21
Copilot AI review requested due to automatic review settings July 16, 2026 17:22
@pedrobragabes
pedrobragabes merged commit af36e09 into main Jul 16, 2026
1 check passed
@pedrobragabes
pedrobragabes deleted the codex/issue-27-indexed-refresh branch July 16, 2026 17:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auth: substituir refresh token O(N) por identificador de sessao

2 participants