[wp-trac] [WordPress Trac] #45142: Filter HTML response in oEmbed proxy controller

WordPress Trac noreply at wordpress.org
Mon Oct 22 10:13:53 UTC 2018


#45142: Filter HTML response in oEmbed proxy controller
------------------------------------+-----------------------
 Reporter:  swissspidy              |       Owner:  (none)
     Type:  defect (bug)            |      Status:  new
 Priority:  normal                  |   Milestone:  5.0
Component:  Embeds                  |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:  rest-api
------------------------------------+-----------------------
Changes (by swissspidy):

 * keywords:  needs-patch => has-patch dev-feedback
 * focuses:   => rest-api


Comment:

 In [attachment:"45142.diff"]:

 * Extract most logic from `wp_filter_pre_oembed_result()` into new
 `get_oembed_response_data_for_url()` function so it can be more easily
 reused.
 * Short-circuit proxy response for internal URLs using
 `get_oembed_response_data_for_url()`
 * Add same `oembed_result` filter as for other embeds

 @pento @imath would love to get your thoughts on this.

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


More information about the wp-trac mailing list