[wp-trac] [WordPress Trac] #59299: Add Felt.com to the list of oEmbed providers

WordPress Trac noreply at wordpress.org
Fri Oct 6 23:43:15 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 canduruk):

 Hi @swissspidy — wanted to check in here. What would be the next steps to
 get Felt included in this allowlist? Thanks a lot in advance for your
 consideration.

 Replying to [comment:10 canduruk]:
 > Replying to [comment:9 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?
 >
 >
 > Thanks for the detailed explanation and it all makes sense.
 >
 > We will look into the plugin route but I'll note that a big chunk of our
 WordPress requests come from end-users, such as teachers and students.
 These folks don't have access to the plugin system or aren't tech-savvy.
 >
 > I think being included in the allowlist would help these people
 immediately instead of those people having to use Notion or other tools to
 continue working with their Felt Maps.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/59299#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list