html: ensure comments are correctly scrubbed
authorWilliam Pitcock <nenolod@dereferenced.org>
Tue, 23 Oct 2018 00:48:49 +0000 (00:48 +0000)
committerWilliam Pitcock <nenolod@dereferenced.org>
Tue, 23 Oct 2018 00:48:49 +0000 (00:48 +0000)
lib/pleroma/html.ex

index fdc5e7dab9930c4adf66cad34fbd951d17dad544..a7338eac3681b6bd4804cf32674d57f5865da1cc 100644 (file)
@@ -43,8 +43,6 @@ defmodule Pleroma.HTML.Scrubber.TwitterText do
   require HtmlSanitizeEx.Scrubber.Meta
   alias HtmlSanitizeEx.Scrubber.Meta
 
-  alias Pleroma.HTML
-
   Meta.remove_cdata_sections_before_scrub()
   Meta.strip_comments()
 
@@ -83,8 +81,6 @@ defmodule Pleroma.HTML.Scrubber.Default do
   require HtmlSanitizeEx.Scrubber.Meta
   alias HtmlSanitizeEx.Scrubber.Meta
 
-  alias Pleroma.HTML
-
   @markup Application.get_env(:pleroma, :markup)
   @uri_schemes Application.get_env(:pleroma, :uri_schemes, [])
   @valid_schemes Keyword.get(@uri_schemes, :valid_schemes, [])
@@ -181,6 +177,8 @@ defmodule Pleroma.HTML.Transform.MediaProxy do
     {"img", attributes, children}
   end
 
+  def scrub({:comment, children}), do: ""
+
   def scrub({tag, attributes, children}), do: {tag, attributes, children}
   def scrub({tag, children}), do: children
   def scrub(text), do: text