impresscms icon indicating copy to clipboard operation
impresscms copied to clipboard

Update HTTMLPurifier to support HTML5

Open fiammybe opened this issue 6 years ago • 6 comments

Verify that the values that we have in the ACP are still in line with the values that are supported by HTMLPurifier in its latest version.

fiammybe avatar Jun 12 '18 09:06 fiammybe

OK, so I thought that we didn't have HTML5 support in our library because we were behind in what HTMLPurifier could offer, but it turns out that the vanilla HTMLPurifier does not officially support HTML5.

I found this : https://github.com/xemlock/htmlpurifier-html5, which offers a HTML5-compatible HTMLPurifier.

We will have to adapt our declaration of the HTMLPurifier library to match this new one, should we decide to use it.

fiammybe avatar Jun 12 '18 09:06 fiammybe

This might even be a good improvement for the 1.3 branch, given that HTML5 isn't that novel anymore and widely usd :-)

fiammybe avatar Nov 14 '18 21:11 fiammybe

This issue was automatically marked as stale

github-actions[bot] avatar May 28 '21 07:05 github-actions[bot]

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or assign sombody or this will be closed in 5 days.

github-actions[bot] avatar Dec 11 '22 02:12 github-actions[bot]

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or assign sombody or this will be closed in 5 days.

github-actions[bot] avatar Feb 11 '23 02:02 github-actions[bot]

ezyang's HTMLPurifier library does include another library to extend coverage to HTML5 tags - see libraries/htmlpurifier/standalone/Lexer/PH5P.php. The 4.15.0 version has PHP8.1 fixes in this file.

skenow avatar Sep 01 '23 21:09 skenow

in that case, I think we can close this issue.

fiammybe avatar Aug 21 '24 10:08 fiammybe