WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#33783 closed task (blessed) (fixed)

Improve oEmbed support for Photobucket

Reported by: johnbillion Owned by: johnbillion
Milestone: 4.4 Priority: low
Severity: normal Version:
Component: Embeds Keywords: 2nd-opinion has-patch
Focuses: Cc:

Description

Previously: #28379

Photobucket should be removed as an oEmbed provider. Finding an oEmbeddable URL for an image on Photobucket is all but impossible, and both the site and its oEmbed endpoint perform redirects on redirects on redirects.

If anyone objects, speak now or forever hold your peace.

Attachments (1)

33783.diff (2.1 KB) - added by johnbillion 6 years ago.

Download all attachments as: .zip

Change History (10)

@johnbillion
6 years ago

#1 @johnbillion
6 years ago

  • Keywords has-patch added

#2 @pento
6 years ago

I'm more inclined to fix it than remove it. It's still pretty popular on WP.com, with ~500k embeds per day.

#3 @atomicjack
6 years ago

Out of interest, if this were to be removed, what happens to content already included the Photobucket oEmbed URLs? Do their images break?

#4 @johnbillion
6 years ago

  • Keywords 2nd-opinion added

When photobucket.com/oembed?url=http://i123.photobucket.com[...] is requested, it redirects to api.photobucket.com which in turn redirects to api123.photobucket.com.

We could cut out one of the redirects by switching the oEmbed endpoint to api.photobucket.com, which it clearly should be anyway.

Sending the main photo page's URL to the oEmbed endpoint results in a 404. You need to use the 'Direct' URL from the 'Share Links' section on the page. @pento / @jkudish Does the partnerships team at WP.com have a contact at Photobucket? It'd be good if these URLs were embeddable. I suspect that WP.com's implementation supports the main photo page URL.

#5 @Viper007Bond
6 years ago

+1 to fix. As annoying as this is, it's still a widely used service that we should work to support.

This ticket was mentioned in Slack in #core by sergey. View the logs.


6 years ago

#7 @wonderboymusic
6 years ago

  • Owner set to johnbillion
  • Status changed from new to assigned

#8 @johnbillion
6 years ago

  • Summary changed from Remove oEmbed support for Photobucket to Improve oEmbed support for Photobucket

#9 @johnbillion
6 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 34526:

Update the oEmbed endpoint URL for Photobucket.

Fixes #33783

Note: See TracTickets for help on using tickets.