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

WordPress Trac noreply at wordpress.org
Mon Oct 22 22:17:27 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 needs-        |     Focuses:  rest-api
  testing needs-unit-tests                       |
-------------------------------------------------+-------------------------

Comment (by swissspidy):

 > About my thoughts: I think it's a good idea, correct me if I'm wrong but
 this means having the new `get_oembed_response_data_for_url()` would make
 it directly usable with a WordPress URL, right ? It seems very interesting
 :)

 Yep, that is an intended nice effect of that patch :-)

 ----

 As for the patch, I tried to add some tests in
 [attachment:"45142.2.diff"], but since it's already late here I couldn't
 make the last test work that should verify the newly added `oembed_result`
 filter in the proxy endpoint.

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


More information about the wp-trac mailing list