[wp-trac] [WordPress Trac] #41746: oEmbed does not respect canonical provider url parameter

WordPress Trac noreply at wordpress.org
Sun Sep 10 19:21:14 UTC 2017


#41746: oEmbed does not respect canonical provider url parameter
------------------------------------------------+--------------------------
 Reporter:  dougal                              |       Owner:
     Type:  defect (bug)                        |      Status:  new
 Priority:  normal                              |   Milestone:  Awaiting
Component:  Embeds                              |  Review
 Severity:  normal                              |     Version:  trunk
 Keywords:  has-patch dev-feedback 2nd-opinion  |  Resolution:
                                                |     Focuses:
------------------------------------------------+--------------------------

Comment (by dougal):

 Also, to clarify, the actual bug here is that in cases where we are
 already using discovery, and the service is already giving us a URL in the
 discovered provider URL, we override it with whatever original URL we were
 spidering in the first place. Clear as mud? Let's break it down:

 User: Hey WordPress, I need you to oEmbed URLA.
 WP: Hmm, I don't already have a provider URL in my whitelist for that,
 let's try discovery!
 WP: Hey, Service -- what's the oEmbed provider URL for URLA? (<spiders
 URLA>)
 Service: The provider address for URLA is {ProvUrl}?url=URLB
 WP: Thanks! But I'll just toss out that querystring, and rebuild it...
 WP: The provider URL is now {ProvUrl}?url=URLA! I'll fetch some JSON from
 there...
 Service: oEmbed for URLA? No such thing!
 User: Dangit, why won't this URL embed correctly?

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


More information about the wp-trac mailing list