Skip to content
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

Ajustes necessários no firewall interno (limite de usuários e publicações por IP) #1463

Open
aprendendofelipe opened this issue Jul 4, 2023 · 1 comment · May be fixed by #1638
Open
Labels
back Envolve modificações no backend segurança Melhoria de segurança

Comments

@aprendendofelipe
Copy link
Collaborator

O TabNews possui um firewall que limita a criação de usuários e publicações vindas do mesmo IP em um curto intervalo de tempo, mas agora precisamos fazer algumas melhorias nele:

  1. A mensagem de erro deve ser mais clara sobre o impacto de ter sido pego pelo firewall. Por exemplo, precisamos avisar que não apenas a mensagem atual não foi publicada, mas também algumas mensagens anteriores podem ter perdido o status de publicadas.
  2. Ao invés de mudar os conteúdos para "draft", talvez precisamos excluí-los ou criar um status próprio para esses casos.
  3. Independentemente da exclusão ou de um status específico, devemos aplicar todos os efeitos que ocorreriam se a publicação fosse excluída, como a perda dos TabCoins.
  4. Cuidar para não criar efeitos em publicações que já estão excluídas.

Talvez existam outras melhorias necessárias, então precisamos rever:

@Rafatcb
Copy link
Collaborator

Rafatcb commented Feb 25, 2024

  1. Ao invés de mudar os conteúdos para "draft", talvez precisamos excluí-los ou criar um status próprio para esses casos.

Acho que podemos usar um status como review para futuramente termos uma lista onde um "moderador" pode tomar uma ação conforme o conteúdo (apagar, permitir a publicação, contatar o autor etc.). Esse status pode ser reaproveitado em situações onde, por exemplo:

  • Conteúdos barrados pelo firewall (este issue).
  • O autor do conteúdo não é confiável.
  • O conteúdo recebeu X sinalizações.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back Envolve modificações no backend segurança Melhoria de segurança
Projects
None yet
2 participants