[wp-trac] [WordPress Trac] #55860: Add Pocket Casts as an oEmbed provider

[WordPress Trac]

#55860: Add Pocket Casts as an oEmbed provider
-------------------------+-----------------------------
 Reporter:  mattwondra   |       Owner:  (none)
     Type:  enhancement  |      Status:  reopened
 Priority:  normal       |   Milestone:  Future Release
Component:  Embeds       |     Version:
 Severity:  normal       |  Resolution:
 Keywords:               |     Focuses:
-------------------------+-----------------------------

Comment (by mattwondra):

 > do we intentionally keep the lists in sync (even if the occasional embed
 provider doesn't strictly need to be added to the allow list)

 @pento @peterwilsoncc In my testing I actually found a user-facing reason
 to put providers on the allowlist — HTML from trusted providers isn't
 altered:

 https://core.trac.wordpress.org/browser/trunk/src/wp-
 includes/embed.php#L894

 For example, the Pocket Casts embed has several links that open in a new
 tab, and a responsive layout with rounded corners — all of which was
 broken until I added it to the allowlist. So though auto-discovery could
 technically embed _some_ version of the Pocket Casts player, it wasn't the
 best experience:

 [[Image(https://cldup.com/7kkm_Z259W.png)]]

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