Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#36189 closed task (blessed) (fixed)

Allow HTTPS URLs for dai.ly to be embedded

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


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 5 years ago.

Download all attachments as: .zip

Change History (6)

5 years ago

#1 @johnbillion
5 years ago

  • Keywords has-patch added

#2 @mikeschroder
5 years ago

  • Owner set to johnbillion
  • Status changed from new to assigned

This ticket was mentioned in Slack in #core by chriscct7. View the logs.

5 years ago

#4 @jeremyfelt
5 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
5 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 36951:

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

Fixes #36189

Note: See TracTickets for help on using tickets.