チャンネル名に右角括弧があると、そのチャンネルからの引用がすべて壊れる

Firepup650 · 2024 年 5 月 17 日午後 3:09

偽のチャンネルデータを使用した例：

Someone random

#Some cool channel

この引用は正常に機能します

Someone random

#[Some cool channel]

この引用は正常に機能しません

zogstrip · 2024 年 5 月 24 日午前 8:57

Firepup650さん、レポートありがとうございます
ついにその時が来たようですね

github.com/discourse/discourse

app/assets/javascripts/discourse-markdown-it/src/features/bbcode-block.js

c39a4de13


      
              closed = true;
              break;
            }
          }
          
          if (closed) {
            length = i - start + 1;
          
            let raw = src.slice(start + tag.length + 1, i);
          
            // trivial parser that is going to have to be rewritten at some point
            if (raw) {
              let match, key, val;
          
              while ((match = ATTR_REGEX.exec(raw))) {
                if (match[1]) {
                  key = "_default";
                } else {
                  key = match[4];
                }

このPRがレビューされてマージされれば修正されます

github.com/discourse/discourse

FIX: BBCode tag parser

main ← fix-bbcode-parser

opened 08:48AM - 24 May 24 UTC

ZogStriP

+230 -344

Wasn't quite handling the cases where a closing bracket `]` was used in the valu…e of one of the attributes. ```markdown [chat quote=user channel="[broken]"] ``` Would not be correctly parsed because we would _greedily_ use the first `]` as the end of the tag even though it might be a valid character when inside proper quotes. https://github.com/discourse/discourse/blob/c39a4de139ed9bba4696d44b15cb7569038b5fde/app/assets/javascripts/discourse-markdown-it/src/features/bbcode-block.js#L62 Re-wrote the `parseBBCodeTag` to properly handle the following cases - A closing tag (aka `[/name]`) which are easy since they don't have any attributes - An old `[quote=...]` format we used that doesn't uses quotes but still has various attributes of the form `key:value` - All three valid BBCode opening tag formats we support - `[name]` without any attributes - `[name=foo]` with a default value - `[name foo=bar]` with some attributes Ended up having to fix/rewrite the few bbcode rules that were using the `parseBBCodeTag` function, namely `d-wrap` and `discourse-local-dates`. While working on this, I think I also found a way to get rid the of shims we had in place so that plugins could use the `parseBBCodeTag` function. Reference - https://meta.discourse.org/t/having-a-right-bracket-in-a-channel-name-breaks-all-quotes-from-that-channel/308439

Peter_Baehr · 2024 年 6 月 4 日午後 4:15

ファイルアップロードに関連するバグを見つけた後、この問題/修正に出くわしました。ファイル名に ‘[’ の前のない ‘]’ 文字が含まれていると、フォーマットが壊れてリンクが生成されません。BBCodeパーサーをここで修正すれば、その問題も解決されると思いますが、他の場所で解析される可能性もあるため、言及しておきます。

zogstrip · 2024 年 6 月 18 日午前 10:38

PRがマージされ、ここにデプロイされ、OPが再ベイクされました👍

JammyDodger · 2024 年 6 月 21 日午前 10:43

このトピックは3日後に自動的に閉じられました。新しい返信はもう許可されていません。

トピック		返信	表示
Fix quotes after phpBB import Support	81	6896	2024 年 6 月 8 日
Quotes in full name mess up quote markdown Bug	4	56	2025 年 12 月 18 日
BBCode [quote] tag and mixed newlines Feature	4	1321	2022 年 1 月 26 日
Tag name with single quote causes server error Bug	5	2422	2020 年 8 月 3 日
BBcode [url] deletes parts of a link Bug	2	891	2017 年 7 月 20 日

チャンネル名に右角括弧があると、そのチャンネルからの引用がすべて壊れる

関連トピック