No Code Attached Yet bug
avatar brianteeman
brianteeman
15 Sep 2026

What happened?

using tinymce it should be possible to embed a youtube video

however it does not work unless you

  1. use joomla filters not tinymce filters
  2. AND disable sandbox OR add youtube.com to the Sandbox Iframe Exclusions List even though the description says youtube.com is already allowed

Version

6.1

Expected result

See video

thorium_j3GKtdG02D.mp4

Actual result

No response

System Information

No response

Additional Comments

No response

avatar brianteeman brianteeman - open - 15 Sep 2026
avatar joomla-cms-bot joomla-cms-bot - change - 15 Sep 2026
Labels Added: No Code Attached Yet bug
avatar joomla-cms-bot joomla-cms-bot - labeled - 15 Sep 2026
avatar mariantanase
mariantanase - comment - 7 Oct 2026

Confirm the issue.
It works with Joomla Text Filter (On), and the domain 'youtube.com' added to the Sandbox Iframe Exclusions List.

avatar mariantanase
mariantanase - comment - 7 Oct 2026

Confirm the issue.
It works with Joomla Text Filter (On), and the domain 'youtube.com' added to the Sandbox Iframe Exclusions List.

avatar brianteeman
brianteeman - comment - 10 Oct 2026

the problem is coming from how we add the current site to the samdboxIframesEclusions

we should be adding the current site but instead we are replacing the tinymce defaults with the current site

avatar ConfidantCommunications
ConfidantCommunications - comment - 29 Oct 2026

Also to note: when editing the TinyMCE plugin settings it says:
Additional domain names can be added to be excluded from iframe sandboxing. The current domain and this list of common domains are excluded by default. 'youtube.com', 'youtu.be', 'vimeo.com', 'player.vimeo.com', 'dailymotion.com', 'embed.music.apple.com', 'open.spotify.com', 'giphy.com', 'dai.ly', 'codepen.io'.

This was not true. I had to make my own entries for "Sandbox Iframe Exclusions List" matching youtube.com and youtube-nocookies.com and then it worked.

avatar brianteeman
brianteeman - comment - 29 Oct 2026

That's the point of this pr. Before it then none of the named sites were allowed, they are now. Youtube-nocookies.com is not in that list

avatar ConfidantCommunications
ConfidantCommunications - comment - 29 Oct 2026

I was having trouble with youtube.com before adding it to the exclusion list.

avatar brianteeman
brianteeman - comment - 29 Oct 2026

Before or after applying this pr?

avatar brianteeman
brianteeman - comment - 29 Oct 2026

Sorry I just realised you are commenting on the issue that reports the problem and not on the pull request that fixes it

avatar ConfidantCommunications
ConfidantCommunications - comment - 29 Oct 2026

Yes, I did not review the pull request. I just happened to discover this bug today. :)

avatar brianteeman
brianteeman - comment - 29 Oct 2026

if you test #48603 then we can get it merged

Add a Comment

Login with GitHub to post a comment