search

mganss / HtmlSanitizer

Cleans HTML to avoid XSS attacks
MIT License
1.52k stars 198 forks source link

Refactor out defaults to separate file #369

Closed vanillajonathan closed 1 year ago

vanillajonathan commented 1 year ago

The HtmlSanitizer is over thousand lines long and mixes the concerns of logic and default options, so refactor out the default options to a separate file.

codecov[bot] commented 1 year ago

Codecov Report

Merging #369 (242add5) into master (84a7a7b) will increase coverage by 0.07%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #369      +/-   ##
==========================================
+ Coverage   94.44%   94.51%   +0.07%     
==========================================
  Files           5        6       +1     
  Lines         828      839      +11     
  Branches       85       85              
==========================================
+ Hits          782      793      +11     
  Misses         34       34              
  Partials       12       12
Impacted Files Coverage Δ
src/HtmlSanitizer/HtmlSanitizer.cs 93.16% <100.00%> (-3.02%) :arrow_down:
src/HtmlSanitizer/HtmlSanitizerDefaults.cs 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 84a7a7b...242add5. Read the comment docs.