Make WordPress Core

Opened 15 years ago

Closed 14 years ago

Last modified 14 years ago

#20102 closed enhancement (fixed)

https oEmbed

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

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 15 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)

#1 @SergeyBiryukov
15 years ago

  • Milestone Awaiting Review3.4

#2 @SergeyBiryukov
15 years ago

  • Component GeneralEmbeds

#3 @ocean90
15 years ago

  • Milestone 3.4Awaiting 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 Awaiting Review3.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 Vimeo https oEmbedhttps oEmbed

#15 @ryan
14 years ago

  • Owner set to ryan
  • Resolutionfixed
  • Status newclosed

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.