Make WordPress Core

Opened 11 years ago

Closed 10 years ago

#24170 closed enhancement (fixed)

oEmbed / Vimeo: Support URLs

Reported by: danielbachhuber's profile danielbachhuber Owned by: nacin's profile nacin
Milestone: 3.9 Priority: low
Severity: minor Version:
Component: Embeds Keywords: has-patch 2nd-opinion
Focuses: Cc:


Vimeo also supports URLs like for its oEmbed endpoint. It would be nice if the core regex also handled these URLs.

Attachments (1)

24170.1.diff (1.2 KB) - added by danielbachhuber 11 years ago.

Download all attachments as: .zip

Change History (8)

#1 @Ipstenu
10 years ago

  • Keywords has-patch added

#2 @Ipstenu
10 years ago

player.vimeo is their old URL - - How much is this used that it's still needed?

#3 @danielbachhuber
10 years ago

Sorry, I don't remember the original context in which this came up.

#4 @nacin
10 years ago

  • Keywords 2nd-opinion added
  • Milestone changed from Awaiting Review to Future Release

I don't know if they're still used, though they still work.

If we can find them in use, we should support them. If they're no longer used, I don't think it's worth it.

#5 @Ipstenu
10 years ago

I found a handful of reports in the forums about this exact problem, so since Vimeo is phasing it out, I think it's a coin toss here :/ So few people appear to have this problem. I think we should patch it, since it's rare enough that if it does happen, the odds of someone recognizing why AND knowing the 'fix' is to use the non-player URL would be rarer.

#6 @nacin
10 years ago

  • Milestone changed from Future Release to 3.9

I didn't realize vimeo was already a regex. No harm in just allowing any subdomain.

#7 @nacin
10 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 27225:

Allow any subdomain for embeds.

props danielbachhuber.
fixes #24170.

Note: See TracTickets for help on using tickets.