Aperçu et résultat Markdown différents

Arkshine · Mai 5, 2023, 1:28

J’ai jeté un coup d’œil rapide par curiosité. Il semble y avoir un problème avec la bibliothèque Nokogiri.

D’après ce que Discourse utilise ici :

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 utilise le parseur HTML4 de Nokogiri.

Cela pourrait être corrigé en utilisant Loofah.html5_fragment tant que Nokogiri >= 1.14.0 et Loofah >= 2.21.0. Discourse utilise déjà Nokogiri::HTML5.fragment; cela aurait du sens.

Note : Loofah 2.21.0 n’est pas encore sorti ; actuellement en RC1.

Sujet		Réponses	Vues
HTML headings with line breaks -> no heading and inconsistency between preview and rendered post Bug	9	1284	Juin 22, 2023
"Traditional markdown linebreaks" not reflected in preview Bug	4	1341	Juillet 3, 2017
Strangely ordered markdown + HTML is shown differently in post vs. preview Support	7	628	Février 2, 2023
Multi-Line Strikethrough Not Covering Quotes Support	24	4570	Septembre 4, 2020
Quoting not working Support	74	3979	Avril 6, 2021

Aperçu et résultat Markdown différents

Sujets connexes