-
Notifications
You must be signed in to change notification settings - Fork 25
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
Video Embeds Stripped Out Regardless of HTML Purifier Config #59
Comments
Hi, thanks for reaching out. This seems to be related to: ezyang/htmlpurifier#302. CKEditor 5 uses |
@i-just Ahh, yep. Dang. Well thank you for that update, I appreciate it! |
Craft 3.7.66 and 4.3.9 are out with an update to the HTML Purifier config to allow |
We're on Craft 4.5.5 but YouTube video's are always stripped out when using HTML purifier, regardless of what config is used. |
"HTML.SafeIframe": true,
"URI.SafeIframeRegexp": "%^(https?:)?//(www\\.youtube\\.com/embed/|www\\.youtube\\.com/watch\\?v=.+|youtu\\.be/.+|www\\.youtube(?:-nocookie)?\\.com/embed/|player\\.vimeo\\.com/video/)%" Add this to your htmlpurifier json config @FrDH and try again |
Description
The
Default.json
config for HTML Purifier that is included on installation has the following lines, which should prevent YouTube and Vimeo video embeds from being stripped out of a CKEditor field on save, but embeds from either platform are still being stripped out.I wasn't totally sure if this issue belongs here, or in the Craft starter, or with HTML Purifier itself, but for reference, I tested this with Redactor, and it wasn't an issue there, the embeds remained after save.
Steps to reproduce
ddev composer require craftcms/ckeditor -w && ddev exec php craft plugin/install ckeditor
https://cdn.ckeditor.com/ckeditor5/32.0.0/classic/ckeditor.js
as the CKEditor Build URL.Default
config.Additional info
The text was updated successfully, but these errors were encountered: