[wp-trac] [WordPress Trac] #21632: Adding Imgur as an oEmbed provider

WordPress Trac noreply at wordpress.org
Tue Feb 11 08:36:14 UTC 2014


#21632: Adding Imgur as an oEmbed provider
-------------------------------------+--------------------------
 Reporter:  bradparbs                |       Owner:  markjaquith
     Type:  enhancement              |      Status:  closed
 Priority:  normal                   |   Milestone:  3.9
Component:  Embeds                   |     Version:
 Severity:  normal                   |  Resolution:  fixed
 Keywords:  has-patch needs-refresh  |     Focuses:
-------------------------------------+--------------------------

Comment (by nacin):

 Replying to [comment:24 GaryJ]:
 > While it's a really low-cost addition to have this provider, is it
 really something that a significant proportion of users are going to be
 using? Or does that not apply here due to it being one line of code?

 These regular expressions are run only on URLs that have already been
 extracted from the post and made possible candidates for embedding.
 They're fast, there aren't many of them, and at the pace we add them,
 we're not going to be making any measurable dent in performance.

 That said, we do have some guidelines for when to add an oEmbed provider:
 http://make.wordpress.org/core/handbook/design-decisions/#whitelisting-
 oembed-providers. It wasn't linked here because these guidelines were
 written down in the time since this ticket first stalled about 17 months
 ago, but imgur fits those guidelines very well. The only thing holding
 them back originally was the shaky API endpoint.

 imgur is [http://en.wikipedia.org/wiki/Imgur#Popularity huge and growing].
 While you and I may not use it, a large swath of the internet does, and
 they use it to create content. And that section of the internet produces
 content that manages to get me to click on a link to an imgur.com page on
 average once a day, and I bet I see way more images elsewhere getting
 hotlinked from it. A lot of imgur users surely also have WordPress sites.
 It makes sense to add it.

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


More information about the wp-trac mailing list