`’` dans les titres de sujets sur la page oops

Moin · Juin 28, 2026, 1:47

Dans la liste des sujets populaires de la page oops (par exemple https://meta.discourse.org/oops), certains titres affichent ’ au lieu de '.

Lilly · Juin 28, 2026, 5:03

c’est drôle (et probablement lié)

sam · Juin 29, 2026, 4:01

oui, cela a probablement été corrigé dans le cadre d’une correction de sécurité, il faut donc être particulièrement vigilant ici en raison des injections

zogstrip · Juin 29, 2026, 2:17

Qu’en penses-tu, @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.

Sujet		Réponses	Vues
<span class="search-highlight"> showing up in search results Bug	2	347	Novembre 28, 2023
Discourse escaping & in Onebox Bug onebox	3	147	Janvier 16, 2026
HTML entities not being parsed in featured topic title on profile Bug	4	646	Février 4, 2020
Notifications menu shows raw backslash escapes in titles Bug	10	1392	Mars 13, 2017
Pasting a link in the composer's title field trims the title if there's a quote character in it Bug	2	533	Juin 8, 2022

`&rsquo;` dans les titres de sujets sur la page oops

Sujets connexes

`’` dans les titres de sujets sur la page oops