[wp-trac] [WordPress Trac] #59299: Add Felt.com to the list of oEmbed providers
WordPress Trac
noreply at wordpress.org
Fri Sep 22 18:56:47 UTC 2023
#59299: Add Felt.com to the list of oEmbed providers
-------------------------+------------------------------
Reporter: javiarce | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Embeds | Version:
Severity: normal | Resolution:
Keywords: | Focuses:
-------------------------+------------------------------
Comment (by swissspidy):
> I am able to use Notion's oEmbed discovery to automatically embed a Felt
map and all the links for fine. Is the link functionality being broken
something we can fix on our end?
WordPress oEmbed discovery support is very strict, and iframes are
rigorously sandboxed by adding `sandbox="allow-scripts"
security="restricted"` attributes. That's why the link is not working
automatically.
Links to the same origin as the iframe are supported via postMessage calls
using a secret that's passed to the iframe via a query parameter. While
that's technically possible for you to add, it also complicates things :-)
Plus, I now see that there also some CORS issues because of the
sandboxing.
Long story short, I think adding it to the allowlist is the most
straightforward way.
That said, it feels like for now that's best done via a dedicated
WordPress plugin that adds such support. Such a plugin could also provide
a dedicated block for the block editor as well.
Perhaps this is something you could look into?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/59299#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list