WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#36189 closed task (blessed) (fixed)

Allow HTTPS URLs for dai.ly to be embedded

Reported by: johnbillion Owned by: johnbillion
Priority: normal Milestone: 4.5
Component: Embeds Version: 3.6
Severity: normal Keywords: https has-patch commit
Cc: Focuses:

Description

Dailymotion has recently implemented a valid security certificate for the dai.ly domain, which means embeds for HTTPS URLs on dai.ly should be supported in core.

Attachments (1)

36189.diff​ (1.8 KB ) - added by johnbillion 11 years ago.

Download all attachments as: .zip

Change History (6)

@johnbillion
11 years ago

#1 @johnbillion
11 years ago

  • Keywords has-patch added

#2 @kirasong
11 years ago

  • Owner set to johnbillion
  • Status new → assigned

This ticket was mentioned in ​Slack in #core by chriscct7. ​View the logs.
11 years ago

#4 @jeremyfelt
11 years ago

  • Keywords commit added

Looks good here. Interesting that the HTTPS dai.ly URLs redirect to HTTP, though it doesn't matter much since we go to HTTPS for the embed.

#5 @johnbillion
11 years ago

  • Resolution → fixed
  • Status assigned → closed

In 36951:

Embeds: Add support for dai.ly embeds over HTTPS.

Fixes #36189

Note: See TracTickets for help on using tickets.