Skip to content

feat(seguranca): no-store, regras de senha e login sem pistas (tasks 47, 46, 45, 52) - #34

Merged
andynnnfw merged 25 commits into
mainfrom
feat/seguranca-login
Oct 9, 2026
Merged

feat(seguranca): no-store, regras de senha e login sem pistas (tasks 47, 46, 45, 52)#34
andynnnfw merged 25 commits into
mainfrom
feat/seguranca-login

Conversation

@sousa7tz

@sousa7tz sousa7tz commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

PR empilhado 1 de 6. Faça o merge na ordem: cada PR tem como base a branch do anterior e mostra só o diff do tema dele. Ao mergear um, o GitHub muda a base do próximo para a main.

Tasks

  • 47: Cache-Control: no-store em todo merchant/* e customer/* (RequestGuard::sendNoStore).
  • 46: PasswordPolicy, senha de 8 a 72 bytes no cadastro e na troca, e a nova não pode ser igual à atual. O login não confere tamanho.
  • 45 (parcial): e-mail inexistente leva o mesmo tempo (LoginGuard::DUMMY_HASH), token CSRF novo no login e teto por conta (50 erros em 15 min, somando todos os IPs), que só avisa no log. Falta escolher o CAPTCHA.
  • 52: ajustes das revisões dos PRs 28 a 32:
    • Money::toCents recusa negativo.
    • valor_pontos_demais tem mensagem própria.
    • A conferência do lançamento passou para o PointsLogModel.
    • deleteOrDeactivate roda numa transação.
    • Migrator::statements só separa no ; fora de texto e de comentário.
  • Também: commit de docs do CLAUDE.md (seção balcão, testes de máscara, comando /autopilot).

Pontos de atenção

  • 45: o aviso sai uma vez por janela, quando a contagem chega exatamente a 50, para não inundar o log.
  • 46: o maxlength="72" do HTML conta caracteres e o servidor conta bytes. Assim o navegador nunca barra uma senha válida, e o servidor explica quando acento ou emoji passam do limite.
  • 52: com o prêmio travado, um resgate concorrente com a exclusão do mesmo prêmio pode falhar por FK (500). É raro, e preferível a um resgate apontando para prêmio inexistente.

Como testar

  • composer test -- --filter "SecurityHeadersTest|PasswordPolicy|RegisterTest|ProfileTest|LoginGuard|LoginLockout|MoneyTest|MigratorStatements"
  • No navegador:
    • cadastro com senha de 7 caracteres
    • trocar a senha pela mesma em ?url=merchant/profile
    • F12 → Rede em ?url=merchant/dashboard para ver o no-store

🤖 Generated with Claude Code

sousa7tz and others added 20 commits October 6, 2026 16:05
@sousa7tz @claude
docs(claude): secao balcao, testes de mascara e comando /autopilot
387c08b 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
fix(seguranca): painel e consulta de saldo com cache-control no-store
3921456 
Task 47.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(senha): senha de 8 a 72 bytes e nova senha diferente da atual
e80a31b 
Task 46.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
fix(login): mesmo tempo para e-mail inexistente, csrf novo no login e…
ccef1ff 
… teto por conta

O teto por conta (50 erros em 15 min, todos os ips) so avisa no log:
o CAPTCHA ainda nao foi escolhido.

Task 45.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
refactor(revisoes): ajustes pendentes das revisoes dos prs 28 a 32
b18798b 
- Money::toCents recusa valor negativo (a tela tambem)
- compra que passa de 10.000 pontos pela regra tem mensagem propria
- conferencia do lancamento do cartao sai do controller para o PointsLogModel
- RewardModel::deleteOrDeactivate numa transacao com o premio travado
- Migrator so separa comandos no ';' fora de texto e de comentario
- comentarios desatualizados do index.php e do schema.sql

Task 52.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
refactor(lgpd): tira do customers as colunas de dado pessoal sem uso
f89fd7e 
Migration 007 remove cpf, email, birth_date e gender (e os indices):
o cliente e so o telefone.

Task 49.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(cadastro): aceita cnpj alfanumerico
72fbd49 
Digito verificador pela regra nova da Receita (codigo ascii - 48), cnpj
numerico continua valendo, mascara com letras, perfil formatado e ADR 001.

Task 57.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(balcao): estorno de resgate pelo extrato
13fff43 
Os pontos voltam ao saldo com um reversal no points_log, o total acumulado
nao muda e os relatorios deixam de contar o resgate desfeito.

Task 44.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(balcao): corrigir nome e trocar o telefone do cliente
c7ec9ee 
So o cartao desta loja muda (saldo, historico e consentimento vao junto);
telefone que ja tem cartao nesta loja e recusado.

Task 43.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(acessibilidade): estilos em classe, botao carregando e campo com…
0f6cdfe 
… erro marcado

- style="" das views vira classe (a barra de progresso usa passos de 5%)
- src/ts/forms.ts: botao mostra carregando e o formulario nao e enviado
  duas vezes; o campo apontado pelo erro fica marcado, com aria-invalid e foco
- foco do teclado sempre visivel

Task 18.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(seguranca): csp completa sem nada inline e hsts em https
dd37bd4 
- scripts inline viram src/ts/nav.ts e src/ts/customer.ts (compilados em public/js)
- onsubmit/onclick viram data-confirm (forms.js) e data-print (nav.js)
- RequestGuard manda CSP com default-src 'self' e, em https, Strict-Transport-Security

Task 48.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(relatorios): filtro por periodo e exportacao em csv
3ee1815 
- indicadores e historico por periodo (hoje, 7 dias, mes, intervalo); o saldo
  em circulacao continua o atual
- merchant/export baixa o historico do periodo ou a lista de clientes em csv
  (; + BOM, protegido contra injecao de formula), so desta loja

Task 59.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(operacao): rota de saude sem sessao que confere o banco
7a1cb79 
?url=health responde ok/erro em json antes do session_start. O healthcheck
do Docker e o monitor externo ficam para a parte de infraestrutura.

Task 56 (parte de codigo).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
chore(qualidade): phpstan nivel 5 com baseline e composer analyse
cec19fb 
A baseline guarda so os "might not be defined" das views (variaveis do
extract). CI, composer audit no workflow e Dependabot ficam pendentes.

Task 51 (parte de codigo).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(email): interface de envio com driver de log e sem provedor por …
5c0993b 
…padrao

MAIL_DRIVER vazio nao manda nada e avisa no log (sem o conteudo); "log"
grava o e-mail em MAIL_LOG_FILE para dev e testes. Provedor, SPF e DKIM
ficam para a escolha do time.

Task 55 (parte de codigo).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(senha): recuperacao de senha por e-mail
8f76199 
Token aleatorio de uso unico, valido por 1 hora e guardado so como hash
(migration 008). O pedido responde igual exista o e-mail ou nao e tem
limite por ip e por e-mail; trocar a senha derruba as sessoes abertas.

Task 21.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(conta): confirmacao do e-mail do lojista
66b36c6 
Cadastro manda um link (24 h, uso unico, so o hash no banco); enquanto nao
confirma, o lojista entra mas so ve a tela de confirmar, com reenvio.
Contas existentes contam como confirmadas (migration 009). Os links por
e-mail passam a herdar de MerchantTokenModel.

Task 50.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
fix(phpstan): tira da baseline o codigo inalcancavel do esqueci a senha
7a5510b 
handleForgot/handleReset tinham retorno never e o render ainda fazia
return depois: codigo novo nao entra na baseline.

Task 21.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
feat(admin): painel administrativo com login proprio
ee200ba 
Tabela admins (migration 010), conta criada por bin/create-admin.php,
sessao separada da do lojista. O admin ve as lojas (sem dados dos
clientes) e ativa ou desativa um lojista, que perde o acesso na hora.

Task 30.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz @claude
docs(adr): proposta de atendentes com login proprio
51731a7 
ADR 003 em status "proposta": tabela users com papel dono/atendente,
permissoes por rota, autor de cada movimentacao no points_log e
perguntas em aberto para o time. Sem codigo.

Task 58.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@sousa7tz sousa7tz mentioned this pull request Oct 7, 2026
Merged
@sousa7tz
sousa7tz requested a review from andynnnfw October 7, 2026 17:45
@andynnnfw
Merge pull request #39 from stargolab/feat/admin
3a401a7 
feat(admin): painel administrativo e adr de atendentes (tasks 30, 58)
@andynnnfw
Merge pull request #38 from stargolab/feat/email-conta
8d5e5fc 
feat(conta): e-mail transacional, esqueci a senha e confirmacao do e-mail (tasks 55, 21, 50)
@andynnnfw
Merge pull request #37 from stargolab/chore/operacao-qualidade
8d7283b 
chore(qualidade): rota de saude e phpstan nivel 5 (tasks 56, 51)
@andynnnfw
Merge pull request #36 from stargolab/feat/csp-relatorios
5468e97 
feat(seguranca): acessibilidade, csp completa e relatorios por periodo com csv (tasks 18, 48, 59)
@andynnnfw
Merge pull request #35 from stargolab/feat/balcao-cadastro
00206ce 
feat(balcao): customers so telefone, cnpj alfanumerico, estorno de resgate e correcao do cartao (tasks 49, 57, 44, 43)
andynnnfw
andynnnfw reviewed Oct 9, 2026
View reviewed changes

@andynnnfw andynnnfw 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.

LGTM

@andynnnfw
andynnnfw merged commit 176bb3b into main Oct 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andynnnfw andynnnfw andynnnfw left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sousa7tz @andynnnfw