Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#33783 closed task (blessed) (fixed)

Improve oEmbed support for Photobucket

Reported by: johnbillion's profile johnbillion Owned by: johnbillion's profile 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 10 years ago.

Download all attachments as: .zip

Change History (10)

@johnbillion
10 years ago

#1 @johnbillion
10 years ago

  • Keywords has-patch added

#2 @pento
10 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
10 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
10 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
10 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.


10 years ago

#7 @wonderboymusic
10 years ago

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

#8 @johnbillion
10 years ago

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

#9 @johnbillion
10 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.