Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#36189 closed task (blessed) (fixed)

Allow HTTPS URLs for to be embedded

Reported by: johnbillion's profile johnbillion Owned by: johnbillion's profile 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 domain, which means embeds for HTTPS URLs on should be supported in core.

Attachments (1)

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

Download all attachments as: .zip

Change History (6)

8 years ago

#1 @johnbillion
8 years ago

  • Keywords has-patch added

#2 @kirasong
8 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.

8 years ago

#4 @jeremyfelt
8 years ago

  • Keywords commit added

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

#5 @johnbillion
8 years ago

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

In 36951:

Embeds: Add support for embeds over HTTPS.

Fixes #36189

Note: See TracTickets for help on using tickets.