-
Notifications
You must be signed in to change notification settings - Fork 35
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
Update HTTMLPurifier to support HTML5 #303
Comments
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. |
This might even be a good improvement for the 1.3 branch, given that HTML5 isn't that novel anymore and widely usd :-) |
This issue was automatically marked as stale |
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. |
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. |
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. |
in that case, I think we can close this issue. |
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.
The text was updated successfully, but these errors were encountered: