Make WordPress Core

Opened 4 weeks ago

Closed 2 weeks ago

Last modified 2 weeks ago

#66078 closed task (blessed) (fixed)

Remove someecards.com as an oEmbed provider

Reported by: johnbillion Owned by: whyisjake
Priority: normal Milestone: 7.1.3
Component: Embeds Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Change History (12)

#1 @suryakantupadhyay
4 weeks ago

Confirmed the listed someecards.com oEmbed endpoints return 404.

I also checked the card pages for a discovery link. There is no <link rel="alternate" type="application/json+oembed">, only og: and twitter:card metadata, and both /v2/oembed/ and /oembed/ return 404.

Looks like they dropped oEmbed support in their Next.js rebuild, so removing the provider seems appropriate.

This ticket was mentioned in ​PR #13459 on ​WordPress/wordpress-develop by ​@suryakantupadhyay.


4 weeks ago
#2

  • Keywords has-patch added

The Someecards oEmbed endpoint now returns a 404 response, and card pages no longer advertise an oEmbed discovery link.

This removes the provider patterns for someecards.com and some.ly from the oEmbed allow list and documents Someecards as no longer supported as of WordPress 7.1.1.

Trac ticket: https://core.trac.wordpress.org/ticket/66078

### Use of AI Tools

AI assistance: Yes
Tool(s): OpenAI Codex
Model(s): GPT-5.6 Sol light
Used for: Identifying the relevant code path, final implementation were reviewed and edited by me.

#3 @adamsilverstein
4 weeks ago

PR looks good. Do we need this fix in 7.1?

#4 @johnbillion
4 weeks ago

It doesn't especially need to be done in 7.1.1 but we have shipped fixes to and removals of of oEmbed providers in minors in the past, eg #55997, #61941.

#5 @adamsilverstein
4 weeks ago

Ok, thanks for clarifying.

7.1.1 RC is in a couple of hours - punting this to 7.1.2 for continued work since it isn't quite ready.

#6 @adamsilverstein
4 weeks ago

  • Milestone 7.1.1 → 7.1.2

#7 @suryakantupadhyay
4 weeks ago

Updated PR to list 7.1.2 as the removal version.

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


3 weeks ago

#9 @johnbillion
2 weeks ago

  • Milestone 7.1.2 → 7.1.3

#10 @whyisjake
2 weeks ago

  • Owner set to whyisjake
  • Status new → accepted

#11 @whyisjake
2 weeks ago

  • Resolution → fixed
  • Status accepted → closed

In 63904:

Embeds: Remove someecards.com from the oEmbed allow list.

The someecards.com oEmbed endpoint returns a 404 Not Found response and card pages no longer expose an oEmbed discovery link, so the provider can no longer be used. This removes the someecards.com and some.ly patterns from the list of trusted providers.

Follows [60345].

Props johnbillion, suryakantupadhyay, adamsilverstein.
Fixes #66078.

#12 @whyisjake
2 weeks ago

In 63905:

Embeds: Remove someecards.com from the oEmbed allow list.

The someecards.com oEmbed endpoint returns a 404 Not Found response and card pages no longer expose an oEmbed discovery link, so the provider can no longer be used. This removes the someecards.com and some.ly patterns from the list of trusted providers.

Follow-up to r60345.

Reviewed by whyisjake.
Merges r63904 to the 7.1 branch.

Props johnbillion, suryakantupadhyay, adamsilverstein.
Fixes #66078.

Note: See TracTickets for help on using tickets.