Make WordPress Core

Opened 21 months ago

Closed 21 months ago

Last modified 17 months ago

#54789 closed defect (bug) (reported-upstream)

oEmbeds: allow embedding Twitter Profiles with a trailing slash

Reported by: jeherve's profile jeherve Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Embeds Keywords:
Focuses: Cc:


This is a follow-up to #38003.

#### Steps to reproduce

  1. Go to Posts > Add New
  2. On a new line, paste a Twitter profile URL, without a trailing slash, such as
  3. The URL will be transformed into an embed block (Twitter variation).
  4. Now, paste a Twitter profile URL with a trailing slash, e.g.
  5. You'll notice that the embed fails for this URL.

This appears to be because we do not currently support trailing slashes for profile URLs:

'#https?://(www\.)?twitter\.com/\w{1,15}$#i' => array( '', true ),

It seems that this was done on purpose when those regexes were added:

Turns out that Twitter's API doesn't like trailing slashes so I'm going to remove the optional slash from the regex.


Would it be possible to add the trailing slash today, or is this still an issue?

Attachments (1)

54789.diff (1.1 KB) - added by jeherve 21 months ago.
Allow trailing slash in Twitter Profile embeds

Download all attachments as: .zip

Change History (4)

21 months ago

Allow trailing slash in Twitter Profile embeds

#1 @skithund
21 months ago

It's still an issue. Twitter API doesn't like the trailing slash.

#2 @jeherve
21 months ago

  • Resolution set to reported-upstream
  • Status changed from new to closed

Ah, I missed that in my test. Good call. Closing this then. 👍

I've reported the issue to Twitter here:

#3 @desrosj
17 months ago

  • Milestone Awaiting Review deleted
Note: See TracTickets for help on using tickets.