[wp-trac] [WordPress Trac] #33970: Huge inflation of post_meta for oEmbed caches

[WordPress Trac]

#33970: Huge inflation of post_meta for oEmbed caches
--------------------------+------------------------------
 Reporter:  whyisjake     |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Embeds        |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------------------

Comment (by whyisjake):

 Replying to [comment:2 helen]:
 > @whyisjake Can you log the HTTP requests for these? You'll probably have
 to hook in somewhere that gets the `$url` and `$attr` so you can determine
 the key (`oembed_result` maybe), I'm not recalling anything that receives
 the key as context. If there are hooks or more context for hooks that we
 could use in core to better enable this, we can look at that too.

 Yeah, I can setup some logging. What is so strange to me is that the keys
 seem to be totally random. They don't match the results at all.

 > Still want to know what's going on, but maybe we could delete all the
 `{{unknown}}` responses on post save at least. I'll try to remember if
 there was a reason we didn't go that route when we made those changes in
 4.0.

 Yeah, it seems like it would be easy to hook into the delete cache method
 when saving a good response. I'll whip up  .diff.

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