Skip to content

Improve validations#33

Merged
thaiismeneses merged 5 commits intomainfrom
validations
Oct 3, 2025
Merged

Improve validations#33
thaiismeneses merged 5 commits intomainfrom
validations

Conversation

@AllanDantas21
Copy link
Copy Markdown
Owner

Implementação de algumas validações com base na RFC:

  • Adicionando SanitizeString para validar as mensagens recebidas no server, no RFC ele indica o máximo de 512 bytes/ por mensagem.
  • Corrigindo a validação dos parâmetros de entradas via argv
  • Adicionando validações no handler de NICK e USER, com base na RFC, por exemplo, limite de 10 caracteres e a proibição de alguns caracteres especiais.

@AllanDantas21 AllanDantas21 self-assigned this Oct 3, 2025
Copy link
Copy Markdown
Collaborator

@dwbessa dwbessa left a comment

Choose a reason for hiding this comment

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

Acho que a única mudança que precisa ser feita é a do bot, mas pode deixar que eu mesmo altero

Comment thread srcs/commands/PrivmsgCommand.cpp Outdated
@thaiismeneses thaiismeneses merged commit c9d7a8a into main Oct 3, 2025
2 checks passed
@thaiismeneses thaiismeneses deleted the validations branch October 3, 2025 23:51
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.

3 participants