[wp-trac] [WordPress Trac] #56733: oEmbed: Add support for Tumblr reader URLs

[WordPress Trac]

#56733: oEmbed: Add support for Tumblr reader URLs
------------------------------------+-----------------------
 Reporter:  pento                   |       Owner:  pento
     Type:  defect (bug)            |      Status:  reopened
 Priority:  normal                  |   Milestone:  6.1
Component:  Embeds                  |     Version:  trunk
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch needs-review  |     Focuses:
------------------------------------+-----------------------
Changes (by pento):

 * keywords:  has-patch => has-patch needs-review
 * status:  closed => reopened
 * version:   => trunk
 * resolution:  fixed =>


Comment:

 Re-opening this ticket, as the original fix was incorrect: Tumblr just
 changed their URL structure. 🙃

 I think it's reasonable to change the behaviour to rely on Tumblr's oEmbed
 endpoint to tell us whether this is a valid embed URL or not. (For
 example, trying to
 [https://www.tumblr.com/oembed/1.0?url=https://www.tumblr.com/oembed/1.0
 embed the embed endpoint] correctly returns an error.)

 This brings Gutenberg and WordPress core behaviour into line (once
 [https://github.com/WordPress/gutenberg/pull/44854 GB44854] is merged).

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