WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#20102 closed enhancement (fixed)

https oEmbed

Reported by: gluten's profile gluten Owned by: ryan's profile ryan
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.3.1
Component: Embeds Keywords: has-patch commit
Focuses: Cc:

Description

The new version of vimeo.com uses https urls, which currently do not trigger WordPress's oEmbed embedding.

Attachments (4)

vimeo-oembed-https.diff​ (1.1 KB) - added by gluten 14 years ago.
vimeo-oembed-https.2.diff​ (1.1 KB) - added by johnjamesjacoby 14 years ago.
Patch was stale. Refreshed for whitespace. (wcgr)
vimeo-oembed-https.3.diff​ (4.6 KB) - added by johnjamesjacoby 14 years ago.
Massaged to handle https/http for all providers
vimeo-oembed-https.4.diff​ (4.5 KB) - added by johnjamesjacoby 14 years ago.
Cleans up version 3 logic (wcgr)

Download all attachments as: .zip

Change History (20)

@gluten
14 years ago

#1 @SergeyBiryukov
14 years ago

  • Milestone changed from Awaiting Review to 3.4

#2 @SergeyBiryukov
14 years ago

  • Component changed from General to Embeds

#3 @ocean90
14 years ago

  • Milestone changed from 3.4 to Awaiting Review

Same problem as with YouTube https embed, see #18719.

#4 @Mamaduka
14 years ago

  • Cc georgemamadashvili@… added

#5 @yoavf
14 years ago

  • Cc yoavf added

#6 @yoavf
14 years ago

  • Milestone changed from Awaiting Review to 3.5

#7 @hd-J
14 years ago

  • Cc jeremy@… added

#8 @nacin
14 years ago

I feel like oEmbed in WordPress should automatically handle https links for all providers, including those that are not regular expressions (​http://example.com/*).

@johnjamesjacoby
14 years ago

Patch was stale. Refreshed for whitespace. (wcgr)

#9 @bradparbs
14 years ago

Applied JJJ's patch, works! Rock on WCGR! Props JJJ and Mark Jaquith!

#10 @Tyrun
14 years ago

Applied JJJ's patch as well and it works. Thanks for showing us how @ WCGR!

@johnjamesjacoby
14 years ago

Massaged to handle https/http for all providers

#11 @johnjamesjacoby
14 years ago

vimeo-oembed-https.3.diff updates all the regular expressions to handle https. Updates the simple URL processor to handle https in the regular expression it generates. Props WCGR.

@johnjamesjacoby
14 years ago

Cleans up version 3 logic (wcgr)

#12 @nacin
14 years ago

  • Keywords commit added

Looks good.

#13 @simonwheatley
14 years ago

  • Cc simon@… added

Always nice to see someone has already created the patch you were about to create. :)

#14 @nacin
14 years ago

  • Summary changed from Vimeo https oEmbed to https oEmbed

#15 @ryan
14 years ago

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

In [21915]:

Handle https for all oembed providers. Props johnjamesjacoby, gluten. fixes #20102

#16 @camiloclc
14 years ago

youtu.be also seems to accept https, although it redirects to the http version when accessed via browser.

Last edited 14 years ago by camiloclc (previous) (diff)
Note: See TracTickets for help on using tickets.