Skip to content

Security updates - #186

Merged
Xarlos89 merged 5 commits into
masterfrom
security-updates
Oct 3, 2026
Merged

Xarlos89 merged 5 commits into
masterfrom
security-updates

Conversation

@Xarlos89

Copy link
Copy Markdown
Collaborator

Description of your PR

A few things.

  • Fixed issue where a spam message is TOO long to drop into the quarantine channel
  • All commands that delete messages now log the message AND images
  • revised permissions for mod commands + added ability to enable/disable verification

Added

  • image logging
  • revised mod permission structure
  • new verification toggle

Fixed

  • fixed some issues with logging after admin commands
  • fixed logging images

Issue link

Closes #155
Closes #102

Checklist

  • Does an issue of this PR exist and have you linked it?
  • Have your ran pre-commit over your PR?
  • Was this PR branched off from master?
  • Is this PR merging to master?

@Xarlos89
Xarlos89 requested review from a team September 29, 2026 09:27
@Xarlos89

Copy link
Copy Markdown
Collaborator Author
image image image image image

@Xarlos89
Xarlos89 removed the request for review from a team September 29, 2026 09:52
@Jiggly-Balls Jiggly-Balls added chore repo/project maintenance enhancement Updates or enhancement to existing features. security Auth, permissions, or exploit risk labels Oct 1, 2026
Comment thread src/bot/cogs/moderation/admin_ban.py Outdated
Comment thread src/bot/cogs/_checks.py
Comment thread src/bot/cogs/_checks.py
@Xarlos89
Xarlos89 requested a review from Jiggly-Balls October 3, 2026 17:03
@Xarlos89
Xarlos89 merged commit 953a7ae into master Oct 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore repo/project maintenance enhancement Updates or enhancement to existing features. security Auth, permissions, or exploit risk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Ban user deletes all of that users messages FEATURE UPDATE: log deleted messages from purge_messages and ban_user

2 participants