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

[WordPress Trac]

#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 imath):

 Hi @swissspidy

 Thanks a lot for the ping. I've just tested the patch using branch 5.0 &
 Gutenberg master.
 - I've first tested without it: a self embed post is working as expected
 - then I've tested with it, running `grunt build` as `grunt watch` wasn't
 applying the part of the patch involving `src/wp-includes/class-wp-oembed-
 controller.php`: it worked as expected too.

 The only weird thing for both tests were the huge place taken by the
 block's placeholder > https://cloudup.com/iAIcSlHQMfj But I guess it's not
 related to this ticket.

 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
 :)

 Thanks a lot for your work on this.

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