`’` en los títulos de los temas en la página de error

Moin · 28 Junio, 2026 13:47

En la lista de temas populares de la página oops (por ejemplo, https://meta.discourse.org/oops), algunos títulos muestran ’ en lugar de '.

Lilly · 28 Junio, 2026 17:03

esto es gracioso (y probablemente relacionado)

sam · 29 Junio, 2026 04:01

Sí, esto probablemente provenga de una corrección de seguridad. Debemos tener especial cuidado aquí debido a las inyecciones.

zogstrip · 29 Junio, 2026 14:17

Qué te parece esta, @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.

Tema		Respuestas	Vistas
<span class="search-highlight"> showing up in search results Bug	2	347	28 Noviembre 2023
Discourse escaping & in Onebox Bug onebox	3	147	16 Enero 2026
HTML entities not being parsed in featured topic title on profile Bug	4	646	4 Febrero 2020
Notifications menu shows raw backslash escapes in titles Bug	10	1392	13 Marzo 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 Junio 2022

`&rsquo;` en los títulos de los temas en la página de error

Temas relacionados

`’` en los títulos de los temas en la página de error