Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#41019 closed task (blessed) (fixed)

oEmbed: Update VideoPress oEmbed URL

Reported by: jnylen0's profile jnylen0 Owned by: swissspidy's profile swissspidy
Milestone: 4.8.1 Priority: normal
Severity: normal Version:
Component: Embeds Keywords: good-first-bug has-patch commit fixed-major
Focuses: Cc:

Description

Per https://developer.wordpress.com/docs/oembed-provider-api/ the https://public-api.wordpress.com/oembed/1.0 URL format is deprecated in favor of https://public-api.wordpress.com/oembed. Let's fix this in core:

https://core.trac.wordpress.org/browser/tags/4.8/src/wp-includes/class-oembed.php#L98

Attachments (1)

41019.patch (1.3 KB) - added by NomNom99 7 years ago.
Updated the URL structure

Download all attachments as: .zip

Change History (8)

#1 @dd32
7 years ago

#41005 was marked as a duplicate.

@NomNom99
7 years ago

Updated the URL structure

#2 @swissspidy
7 years ago

  • Keywords has-patch commit added; needs-patch removed

If the for param is still supported, this seems good to go.

#3 @jnylen0
7 years ago

Yes, for is still supported (required). +1 for commit.

#4 @swissspidy
7 years ago

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

In 40901:

Embeds: Update oEmbed endpoint URL for VideoPress.

Props NomNom99.
Fixes #41019.

#5 @swissspidy
7 years ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening for 4.8.1

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


7 years ago

#7 @SergeyBiryukov
7 years ago

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

In 41025:

Embeds: Update oEmbed endpoint URL for VideoPress.

Props NomNom99.
Fixes #41019.

Merges [40901] to the 4.8 branch.

Note: See TracTickets for help on using tickets.