Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 1573 Archived crash with html sanitizer on regex recursion #1574

Closed
wants to merge 1 commit into from

Conversation

DLalot
Copy link

@DLalot DLalot commented Jan 6, 2023

Just catching the exception to avoid looping (400 crashed mails received just for one bad email)
It's a very basic solution to consider that problem as a simple archive problem.
Should be better to just put it in spool bad without annoying the listmaster. But it's a little bit too difficult for me.

@DLalot
Copy link
Author

DLalot commented Jan 6, 2023

With the patch, sympa just said that something went wrong. But I believe that this phishing could be distributed in tons of lists. Should we say something ? From my point of view, we should just log it but without sending emails to listmaster..

Sympa n'a pas pu archiver le message
'[email protected],1389,8655'.
Celui-ci est déplacé dans le répertoire '/home/sympa/spool/outgoing/bad'.
Consultez les logs pour plus de détails.

@ikedas
Copy link
Member

ikedas commented Jan 19, 2023

Superseded by #1575

@ikedas ikedas closed this Jan 19, 2023
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.

2 participants