`’` nos títulos dos tópicos na página de erro

Moin · 28 Junho , 2026 13:47

Na lista de tópicos populares na página de erros (por exemplo, https://meta.discourse.org/oops), alguns títulos exibem ’ em vez de '.

Lilly · 28 Junho , 2026 17:03

isso é engraçado (e provavelmente relacionado)

sam · 29 Junho , 2026 04:01

sim, isso provavelmente veio de uma correção de segurança, precisamos ter cuidado extra aqui devido a injeções

zogstrip · 29 Junho , 2026 14:17

O que você acha dessa aqui, @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.

zogstrip · 6 Julho , 2026 06:00

Este tópico foi fechado automaticamente após 3 dias. Não é mais possível enviar novas respostas.

Tópico		Respostas	Vistas
<span class="search-highlight"> showing up in search results Bug	2	350	28 Novembro , 2023
Discourse escaping & in Onebox Bug onebox	3	155	16 Janeiro , 2026
HTML entities not being parsed in featured topic title on profile Bug	4	646	4 Fevereiro , 2020
Notifications menu shows raw backslash escapes in titles Bug	10	1394	13 Março , 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 Junho , 2022

`&rsquo;` nos títulos dos tópicos na página de erro

Related topics

`’` nos títulos dos tópicos na página de erro