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

0010204: out of memory in html purifier #5093

Closed
Gloirin opened this Issue Jun 9, 2018 · 6 comments

Comments

Projects
None yet
1 participant
@Gloirin

Gloirin commented Jun 9, 2018

Reported by pschuele on 2 Sep 2014 08:45

Version: Collin (2013.10.8)

sometimes big mails cause out of memory problems in html purifier.

maybe we can try to skip purifying the mail (just allow plaintext part if available) for those mails. we need a way to detect / estimate the required memory for a mail and throw an exception if the memory is about to run out.

@Gloirin

This comment has been minimized.

Show comment
Hide comment
@Gloirin

Gloirin commented Jun 10, 2018

Related to #5091

@Gloirin

This comment has been minimized.

Show comment
Hide comment
@Gloirin

Gloirin Jun 11, 2018

Comment posted by weberho on 2 Sep 2014 12:06

There is another HTML cleaner available which could be worth testing:
https://packagist.org/packages/htmlawed/htmlawed

Gloirin commented Jun 11, 2018

Comment posted by weberho on 2 Sep 2014 12:06

There is another HTML cleaner available which could be worth testing:
https://packagist.org/packages/htmlawed/htmlawed

@Gloirin

This comment has been minimized.

Show comment
Hide comment
@Gloirin

Gloirin Jun 11, 2018

Comment posted by pschuele on 2 Sep 2014 12:29

ah, thanks for the information. we'll have a look.

Gloirin commented Jun 11, 2018

Comment posted by pschuele on 2 Sep 2014 12:29

ah, thanks for the information. we'll have a look.

@Gloirin

This comment has been minimized.

Show comment
Hide comment
@Gloirin

Gloirin Jun 11, 2018

Comment posted by mspahn on 23 Sep 2014 12:08

http://htmlpurifier.org/docs/dev-optimization.html

Gloirin commented Jun 11, 2018

Comment posted by mspahn on 23 Sep 2014 12:08

http://htmlpurifier.org/docs/dev-optimization.html

@Gloirin

This comment has been minimized.

Show comment
Hide comment
@Gloirin

Gloirin Jun 11, 2018

Comment posted by mspahn on 23 Sep 2014 12:30

Changed for testing purposes the cleaning level to medium.
https://gerrit.tine20.org/tine20/#/c/2892/

This should be discussed.

Gloirin commented Jun 11, 2018

Comment posted by mspahn on 23 Sep 2014 12:30

Changed for testing purposes the cleaning level to medium.
https://gerrit.tine20.org/tine20/#/c/2892/

This should be discussed.

@Gloirin

This comment has been minimized.

Show comment
Hide comment
@Gloirin

Gloirin Jun 11, 2018

Comment posted by fabiodepin on 15 Oct 2014 10:56

see 0010200.

Gloirin commented Jun 11, 2018

Comment posted by fabiodepin on 15 Oct 2014 10:56

see 0010200.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment