Skip to content

аудит безопасности: пинги через вебхуки, лимит /clear#24

Merged
insvrg3ncy merged 1 commit into
developfrom
sec/audit-hardening
Jun 11, 2026
Merged

аудит безопасности: пинги через вебхуки, лимит /clear#24
insvrg3ncy merged 1 commit into
developfrom
sec/audit-hardening

Conversation

@iAmScienceMan

Copy link
Copy Markdown
Contributor

прошёлся по коду на предмет дыр:

  • вебхуки (пересылка, uwuify, /anonsay) и /say больше не могут пинговать @everyone и роли чужим текстом
  • /clear ограничен 200 сообщениями
  • понятная ошибка при пустом TOKEN

тесты и ruff проходят, на allowed_mentions добавил тесты

- глобальный allowed_mentions в боте: без everyone и ролей
- вебхук-сервис подставляет безопасные allowed_mentions по умолчанию
  (пересылка каналов, uwuify), /anonsay вообще без пингов
- /clear ограничен 200 сообщениями за раз
- бот падает с понятной ошибкой если TOKEN не задан
- тесты на allowed_mentions вебхуков
@iAmScienceMan iAmScienceMan added the bug Something isn't working label Jun 11, 2026
@iAmScienceMan iAmScienceMan self-assigned this Jun 11, 2026
@iAmScienceMan iAmScienceMan requested a review from insvrg3ncy June 11, 2026 13:11
@codecov

codecov Bot commented Jun 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
bot.py 0.00% 3 Missing ⚠️
cogs/moderation.py 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@insvrg3ncy insvrg3ncy merged commit c7d71e3 into develop Jun 11, 2026
5 checks passed
@insvrg3ncy insvrg3ncy deleted the sec/audit-hardening branch June 11, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants