`’` 在 Oops 页面的主题标题中

Moin · 2026 年6 月 28 日 13:47

在 oops 页面的热门主题列表（例如 https://meta.discourse.org/oops）中，部分标题显示为 ’ 而非 '。

Lilly · 2026 年6 月 28 日 17:03

这很有趣（而且可能有关联）

sam · 2026 年6 月 29 日 04:01

嗯，这很可能来自一个安全修复，由于注入问题，我们需要格外小心。

zogstrip · 2026 年6 月 29 日 14:17

你觉得这个怎么样 @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 · 2026 年7 月 6 日 06:00

此主题在 3 天后自动关闭。不再允许新的回复。

话题		回复	浏览量
<span class="search-highlight"> showing up in search results Bug	2	349	2023 年11 月 28 日
Discourse escaping & in Onebox Bug onebox	3	148	2026 年1 月 16 日
HTML entities not being parsed in featured topic title on profile Bug	4	646	2020 年2 月 4 日
Notifications menu shows raw backslash escapes in titles Bug	10	1394	2017 年3 月 13 日
Pasting a link in the composer's title field trims the title if there's a quote character in it Bug	2	533	2022 年6 月 8 日

`&rsquo;` 在 Oops 页面的主题标题中

相关话题

`’` 在 Oops 页面的主题标题中