Issue with x.com previews

MarcP · May 18, 2024, 4:48pm

Everything seems fine on opengraph.xyz - not sure if this is on Discourse end?

But:

https://x.com/X/status/1791399086182031741

Posting old twitter.com link does work.

Firepup650 · May 18, 2024, 4:52pm

If I had to guess, I’d say twitter.com had a special oneboxing rule, and that rule now needs to be copied to x.com.

Arkshine · May 18, 2024, 9:05pm

Indeed.

github.com

discourse/discourse/blob/main/lib/onebox/engine/twitter_status_onebox.rb#L12


      
          
          module Onebox
            module Engine
              class TwitterStatusOnebox
                include Engine
                include LayoutSupport
                include HTML
                include ActionView::Helpers::NumberHelper
          
                matches_regexp(
                  %r{^https?://(mobile\.|www\.)?twitter\.com/.+?/status(es)?/\d+(/(video|photo)/\d?+)?+(/?\?.*)?/?$},
                )
                always_https
          
                def http_params
                  { "User-Agent" => "DiscourseBot/1.0" }
                end
          
                def to_html
                  raw.present? ? super : ""
                end

jancernik · May 23, 2024, 2:34pm

Thanks for the report! We merged a fix for this today

JammyDodger · May 26, 2024, 2:58pm

This topic was automatically closed after 3 days. New replies are no longer allowed.

Topic		Replies	Views
Twitter link preview broken Support	1	931	June 10, 2020
Twitter API – My Discourse API App for Tweet previews was suspended Support	3	715	April 15, 2023
Weird preview in Facebook share Bug	5	1160	July 30, 2015
How can I show X/Twitter posts on my site? Support	7	357	March 13, 2024
Twitter login doesn't work on meta Site feedback	5	1172	June 22, 2024

Issue with x.com previews

Related topics