We’re trying to implement Discourse embed comments to our documentation website, and we’re facing an inconsistent error with the message “Start discussion” and “Error embedding”. It’s not a consistent error, we couldn’t identify a pattern. There are pages rendering the comments normally, other page…

Ok, new info. I just remembered that I had previously created a test category and all topics had already been populated. I now see that some pages point to the old topics. Maybe they conflict, so we’d need to purge all old topics and the old category itself. Using .* seems to work now.

We’re currently on Discourse 1.9.0.beta13 - rolling back to 1.8 or upgrading to 1.19.0.beta14 ( Releases · discourse/discourse · GitHub ) could be a possible fix?

It’s look like your Allowed Hosts or Path Whitelist not matching the referrer URL in Discourse (in your case docs.gitlab.com and /some/path optional). Can you please double check the setting? Are you running Discourse with our standard docker installation method ?

This is what we have: [com-2017-11-04-07-57-51] and yes, we are running Discourse the official way via Docker. Maybe we should add a path whitelist?

[image] Comments not embedding, 500 Error I bet you have Path Whitelist of * and it should be .* I think so. You should add “Path Whitelist” as .*

The change should be instant, right? I tried with many variations, but all fail. .* .*\.html\z The interesting thing is that it fails for specific pages for specific browsers. I also checked if there’s an X-Frame-Options header that would deny the iframe, but there isn’t: curl -I https://docs.git…

That is the problem. Every time after a URL embedded then it will be stored in DB with referenced topic_id. Let me know if you are unable to purge it.

@vinothkannans seems that works now, but we’ve noticed that in some cases the iframe is stuck at “Loading discussion…”. A page reload fixes it, though. I cannot find anything related in /logs. I deleted all old topics and then the category itself, so there shouldn’t be any problems. Could this be a…

We tested internally, and could not reproduce the “Loading discussion…” error! Consider this topic solved! Thanks all!

Awesome, thanks everyone! :heart:

Discourse embed: inconsistent errors

Support

vinothkannans (Vinoth Kannan) November 4, 2017, 6:34am 5

I think so. You should add “Path Whitelist” as .*

Topic		Replies	Views
"Error Embedding" comments Support	16	2329	May 4, 2022
Cannot get embedding to work Support	11	2612	August 31, 2021
Comments not embedding, 500 Error Feature pr-welcome	15	2022	October 3, 2017
Embedding error Support	13	4036	November 21, 2020
Discourse embed blocked on "Loading Discussion..." Support embedding	5	2619	October 13, 2015

Discourse embed: inconsistent errors

Related topics