`’` in topic titles on oops page

zogstrip · 29 juni 2026 om 14:17

What do you think of this one @sam?

github.com/discourse/discourse

FIX: Prevent double-escaping of HTML entities in titles and meta descriptions (#41261)

main ← fix-double-escape-entities

merged 06:26PM - 02 Jul 26 UTC

ZogStriP

+167 -13

Previously, content that was already HTML-escaped — topic `fancy_title`s on the …404/oops pages, GitHub onebox labels, and topic/group/category `<meta>` descriptions — got escaped a second time, so readers (and crawlers) saw literal entities like `’` and `&` instead of `'` and `&`. This change escapes every value exactly once, by converting it to plain text before it reaches the renderer instead of flagging anything `html_safe`: - `Emoji.codes_to_img` escapes its text segments with `html_escape_once` so the already-escaped `fancy_title` and sanitized GitHub labels aren't escaped again. All of its callers render into element content, and the attribute escaping inside `emoji_img_tag` is untouched. - the new `ExcerptParser.to_plain_text` converts a stored excerpt (escaped text, the `…` truncation marker, and the hashtag placeholder markup the parser keeps) back to plain text, exposed as `Topic#plain_text_excerpt`. It's a tag-strip plus a memoized `htmlentities` decode — pure Ruby, a few microseconds per excerpt, no Nokogiri in the hot path. - `Category#plain_text_description` caches the pre-escape plain text that `description_text` previously discarded, and the category meta descriptions use it directly. This also fixes the double-escaped `og:description`/`twitter:description` on category pages, where `gsub_emoji_to_unicode` silently drops the `html_safe` flag of `description_text`. - the new `Group#bio_summary` mirrors `UserProfile#bio_summary`: a plain-text excerpt of the bio with links and images stripped and hashtags rendered as plain `#slug`. Marking the stored values `html_safe` would have worked for the column data — every writer of `topics.excerpt` escapes — but `topic.excerpt` can be swapped in-memory with a localized excerpt that isn't guaranteed to be escaped, so treating everything as unsafe plain text is both simpler and safer.

Topic		Antwoorden	Weergaven
<span class="search-highlight"> showing up in search results Bug	2	351	28 november 2023
Discourse escaping & in Onebox Bug onebox	3	156	16 januari 2026
HTML entities not being parsed in featured topic title on profile Bug	4	651	4 februari 2020
Notifications menu shows raw backslash escapes in titles Bug	10	1397	13 maart 2017
Pasting a link in the composer's title field trims the title if there's a quote character in it Bug	2	533	8 juni 2022

`&rsquo;` in topic titles on oops page

Gerelateerde topics

`’` in topic titles on oops page