bterkuile/fizzy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove @regexp_timeout_reported from AutoLinkScrubber

Browse Source 
No longer needed since the scrubber is not memoized across calls and the
text node length guard prevents the performance issue.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Mike Dalessio
2026-03-20 19:53:02 -04:00
parent 87c659cd67
commit 8c3461a1d6
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/auto_link_scrubber.rb
+1 -3
View File
@@ -22,13 +22,12 @@ class AutoLinkScrubber < Loofah::Scrubber
def initialize def initialize
@direction = :top_down @direction = :top_down
@regexp_timeout_reported = false
end end
def scrub(node) def scrub(node)
return Loofah::Scrubber::STOP if EXCLUDED_ELEMENTS.include?(node.name) return Loofah::Scrubber::STOP if EXCLUDED_ELEMENTS.include?(node.name)
if node.text? && !@regexp_timeout_reported if node.text?
replacement = autolink_text_node(node) replacement = autolink_text_node(node)
node.replace(replacement) if replacement node.replace(replacement) if replacement
end end
@@ -78,7 +77,6 @@ class AutoLinkScrubber < Loofah::Scrubber
links links
rescue Regexp::TimeoutError => error rescue Regexp::TimeoutError => error
Sentry.capture_exception error if Fizzy.saas? Sentry.capture_exception error if Fizzy.saas?
@regexp_timeout_reported = true
[] []
end end