[wp-trac] [WordPress Trac] #34335: URL shorteners ( redirects) should to be supported by the embed handling for couple of hopes

[WordPress Trac]

#34335: URL shorteners ( redirects) should to be supported by the embed handling
for couple of hopes
--------------------------+------------------------------
 Reporter:  pbearne       |       Owner:
     Type:  defect (bug)  |      Status:  reopened
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Embeds        |     Version:  4.4
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------------------
Changes (by swissspidy):

 * keywords:  2nd-opinion =>
 * status:  closed => reopened
 * resolution:  maybelater =>
 * milestone:   => Awaiting Review


Comment:

 @stephenharris Good catch. However, the URL part is only there if it was
 found during oEmbed discovery, but not when `wp_oembed_add_provider()` is
 used for registering a provider. So yes, that line is necessary.

 ''But'' we could try to extract any `url` param from the `$provider`
 variable and only use `add_query_arg()` if the `url` param isn't already
 there.

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