limits for parsing files and archives

JonathanThorpe / spamassassin-vba-macro

SpamAssassin plugin for detecting VBA macros in Microsoft Office Documents

Apache License 2.0

11 stars 3 forks source link

limits for parsing files and archives #2

Closed meschansky closed 8 years ago

meschansky commented 8 years ago

Large attachments cause errors "exceeded time limit in Mail::SpamAssassin::Plugin::Check:"

limiting number of files within archive to process (default 3 files)
limit the buffer size to check (default 100k)
limit the archive part size to extract (default 1m)

Removed the check for content-transfer-encoding, as it opens a hole if a spammer uses non-base64 encoding (qp, binary). And the Mail::SpamAssassin::Message::Node::decode takes care of the CTE automatically.

JonathanThorpe commented 8 years ago

Thank you very much for your pull request! This will certainly introduce some much needed robustness to the plugin.