Make WordPress Core

Opened 5 years ago

Closed 9 months ago

#45610 closed enhancement (wontfix)

oEmbed Tweets from Twitter with external links uses t.co

Reported by: micasuh's profile micasuh Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.0
Component: Embeds Keywords: needs-patch close
Focuses: Cc:

Description

Currently, when embedding a Tweet that has an external link, WordPress imports the string value from the link element with the type="application/json+oembed" attribute that includes a URL shortener t.co.

My understanding is that t.co is also a tracker for Twitter's analytics and it doesn't seem necessary for WordPress to support this.

It makes sense for WordPress' oEmbed feature to use the data attribute "data-expanded-url" that contains the original URL.

Change History (4)

#1 @chriscct7
5 years ago

data-expanded-url is not available for all tweets. It's noted by Twitter to be only available (for example; there are other restrictions) for tweets that are made within the last 13 months, not older tweets.

#2 @chriscct7
5 years ago

  • Keywords needs-patch added; oembed twitter removed

#3 @swissspidy
9 months ago

  • Keywords close added

#4 @johnbillion
9 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Special case handling such as this is not practical to maintain. There are tracking links everywhere, Twitter should not be treated differently.

Closing this off due to lack of interest and the above.

Note: See TracTickets for help on using tickets.