Moet Discourse moeite doen om een levensvatbaar commentaarplatform te worden?

Falco · 17 maart 2026 om 20:40

Thanks heaps for the report @Slaune, I managed to repro and fix it now

github.com/discourse/discourse

FIX: Handle blank-slug topics in full app embed mode (#38668)

main ← fix/embed-blank-slug-redirect-loop

merged 06:37PM - 17 Mar 26 UTC

xfalcox

+58 -3

## Summary Follow-up to #36613 (full Ember app embed mode). When an embedd…ed page's title can't be parsed by `ruby-readability`, `TopicEmbed.import` creates a topic with a blank title and slug. This causes: - **Redirect loop**: `Topic.url` generates `/t//123` (double slash), and `TopicsController#show` endlessly redirects trying to add the missing slug - **Broken topic**: The topic has no title or meaningful content ### Changes - **`topic_embed.rb`**: Fall back to the embed URL as the topic title when the parsed title is blank - **`topic.rb`**: `Topic.url` omits the slug segment when slug is blank (`/t/123` instead of `/t//123`) - **`topics_controller.rb`**: Skip canonical slug redirect when the topic has no slug (nothing to redirect to)

Topic		Antwoorden	Weergaven
Using Discourse to power comments in an event system or blog? Support	29	9546	25 augustus 2023
Embed Discourse comments with on-page commenting? Feature	32	7944	19 januari 2019
WP-discourse: comments are not pulled over WordPress	21	6146	20 april 2017
Embedding, but don't automatically create a topic until a user chooses to comment Feature pr-welcome	56	8413	4 december 2025
Federation support for Discourse Feature completed	122	38232	17 mei 2024

Moet Discourse moeite doen om een ​​levensvatbaar commentaarplatform te worden?

Gerelateerde topics

Moet Discourse moeite doen om een levensvatbaar commentaarplatform te worden?