-
-
Notifications
You must be signed in to change notification settings - Fork 734
Closed
Labels
a: utilityRelated to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)Related to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)p: 2 - normalNormal PriorityNormal Priorityt: featureNew feature or requestNew feature or request
Description
We're currently filtering malicious filetypes in our filetype filter - to be specific, we're filtering all filetypes we haven't specifically whitelisted. This does not, however, apply to URLs. That means it's easy enough to get around our filter by posting the file in a DM or on a different server, and then just linking the content instead.
This also allows users to post URLs that literally point at viruses, which isn't something we should permit.
Let's write a second filter that uses the same whitelist and same logics to filter URLs as well.
Let's keep this one simple - no need to download the file and inspect it to determine exactly what its filetype is. Just filter by extension, the way we're doing now.
Metadata
Metadata
Assignees
Labels
a: utilityRelated to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)Related to utility commands: (bot, eval, extensions, jams, reminders, snekbox, utils)p: 2 - normalNormal PriorityNormal Priorityt: featureNew feature or requestNew feature or request