Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#35152 closed enhancement (fixed)

Remove Rdio embed support

Reported by: pento's profile pento Owned by: jorbin's profile jorbin
Milestone: 4.4.1 Priority: normal
Severity: normal Version:
Component: Embeds Keywords: fixed-major
Focuses: Cc:

Description

With Rdio closing down, we should remove oEmbed support for them from core, lest the domain become owned by someone more nefarious than a streaming music provider.

The codex page will also need to be updated.

Attachments (1)

35152.diff (2.3 KB) - added by swissspidy 9 years ago.

Download all attachments as: .zip

Change History (9)

#1 @pento
9 years ago

Given the timeline, I would not be entirely against pushing this into 4.4.1, too.

@swissspidy
9 years ago

#2 @swissspidy
9 years ago

  • Keywords has-patch added; needs-patch removed

#3 @DrewAPicture
9 years ago

  • Keywords needs-docs removed

#4 @jorbin
9 years ago

  • Milestone changed from 4.5 to 4.4.1

This absolutely seems like something that should be done for 4.4.1 to protect the safety of our users.

#5 @jorbin
9 years ago

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

In 36007:

Remove RDIO from oEmbed providers

RDIO is shutting down. https://www.rdio.com/farewell/

Props swissspidy
Fixes #35152

#6 @jorbin
9 years ago

  • Keywords fixed-major added; has-patch removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

#7 @dd32
9 years ago

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

In 36066:

Remove RDIO from oEmbed providers

RDIO is shutting down. https://www.rdio.com/farewell/

Merges[36007] to the 4.4 branch.
Props swissspidy.
Fixes #35152.

#8 @DrewAPicture
9 years ago

  • Keywords needs-codex removed

Removed from the Codex.

Note: See TracTickets for help on using tickets.