[wp-trac] [WordPress Trac] #45033: Add TIDAL to oEmbed whitelist

[WordPress Trac]

#45033: Add TIDAL to oEmbed whitelist
-------------------------+----------------------
 Reporter:  tidal        |       Owner:  (none)
     Type:  enhancement  |      Status:  closed
 Priority:  normal       |   Milestone:
Component:  Embeds       |     Version:
 Severity:  normal       |  Resolution:  wontfix
 Keywords:  needs-patch  |     Focuses:
-------------------------+----------------------
Changes (by peterwilsoncc):

 * keywords:  needs-patch reporter-feedback => needs-patch
 * status:  new => closed
 * resolution:   => wontfix
 * milestone:  Awaiting Review =>


Comment:

 I've tested this locally and confirmed auto-discovery works as the embed
 only uses [https://core.trac.wordpress.org/browser/tags/4.9.8/src/wp-
 includes/embed.php?marks=730-745#L730 allowed tags and attributes]. I'll
 close this ticket accordingly.

 In testing I didn't get the 500 error @swissspidy received so there might
 be a few bugs to iron out with the end-point at your end.

 While you're testing, switching the iframe to https always will ensure
 browsers don't block the embed on sites running over SSL.

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