Revert "DEV: Add logging for stack level too deep exception in HtmlToMarkdown"

We can do this in a better way by storing an IncomingEmail record.

Follow-up-to: 4a9ee25c56
This commit is contained in:
Daniel Waterworth 2020-07-09 13:38:01 +01:00
parent 32ee9fae40
commit 3b368a48d1

View File

@ -6,30 +6,21 @@ require "securerandom"
class HtmlToMarkdown class HtmlToMarkdown
def initialize(html, opts = {}) def initialize(html, opts = {})
@html = html
@opts = opts @opts = opts
# we're only interested in <body> # we're only interested in <body>
@doc = Nokogiri::HTML5(html).at("body") @doc = Nokogiri::HTML5(html).at("body")
end
def to_markdown
remove_not_allowed!(@doc) remove_not_allowed!(@doc)
remove_hidden!(@doc) remove_hidden!(@doc)
hoist_line_breaks!(@doc) hoist_line_breaks!(@doc)
remove_whitespaces!(@doc) remove_whitespaces!(@doc)
end
def to_markdown
traverse(@doc) traverse(@doc)
.gsub(/\n{2,}/, "\n\n") .gsub(/\n{2,}/, "\n\n")
.strip .strip
rescue SystemStackError => e
Discourse.warn_exception(
e,
message: "stack level too deep in HtmlToMarkdown",
env: { html: @html },
)
raise
end end
private private