Markdown-Vorschau und Ergebnis unterscheiden sich

Arkshine · 5. Mai 2023 um 01:28

Ich habe aus Neugier kurz nachgeschaut. Es scheint ein Problem mit der Nokogiri-Bibliothek zu sein.

Soweit ich das hier bei Discourse sehe:

lib/pretty_text.rb

7ff8e5580


      
            doc = Nokogiri::HTML5.fragment(sanitized)
          
            add_nofollow = !options[:omit_nofollow] && SiteSetting.add_rel_nofollow_to_user_content
            add_rel_attributes_to_user_content(doc, add_nofollow)
            strip_hidden_unicode_bidirectional_characters(doc)
            sanitize_hotlinked_media(doc)
          
            add_mentions(doc, user_id: opts[:user_id]) if SiteSetting.enable_mentions
          
            scrubber = Loofah::Scrubber.new { |node| node.remove if node.name == "script" }
            loofah_fragment = Loofah.fragment(doc.to_html)
            loofah_fragment.scrub!(scrubber).to_html
          end
          
          def self.strip_hidden_unicode_bidirectional_characters(doc)
            return if !DANGEROUS_BIDI_REGEXP.match?(doc.content)
          
            doc
              .css("code,pre")
              .each do |code_tag|
                next if !DANGEROUS_BIDI_REGEXP.match?(code_tag.content)

Loofah.fragment verwendet den HTML4-Parser von Nokogiri.

Dies könnte mit Loofah.html5_fragment behoben werden, solange Nokogiri >= 1.14.0 und Loofah >= 2.21.0 ist. Discourse verwendet bereits Nokogiri::HTML5.fragment; das wäre sinnvoll.

Hinweis: Loofah 2.21.0 ist noch nicht veröffentlicht; derzeit in RC1.

Thema		Antworten	Aufrufe
HTML headings with line breaks -> no heading and inconsistency between preview and rendered post Bug	9	1285	22. Juni 2023
"Traditional markdown linebreaks" not reflected in preview Bug	4	1342	3. Juli 2017
Strangely ordered markdown + HTML is shown differently in post vs. preview Support	7	628	2. Februar 2023
Multi-Line Strikethrough Not Covering Quotes Support	24	4570	4. September 2020
Quoting not working Support	74	3980	6. April 2021

Markdown-Vorschau und Ergebnis unterscheiden sich

Verwandte Themen