Combine the noscript/mglyph prevention blocks

These were on two separate branches, so let's be smart.
rails · Dec 1, 2024 · a7b0cfe · a7b0cfe
1 parent 5658335
commit a7b0cfe
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/rails/html/scrubbers.rb b/lib/rails/html/scrubbers.rb
@@ -149,11 +149,11 @@ def validate!(var, name)
               warn("WARNING: 'malignmark' tags cannot be allowed by the PermitScrubber and will be scrubbed")
               var.delete("malignmark")
             end
-          end
 
-          if var && name == :tags && var.include?("noscript")
-            warn("WARNING: 'noscript' tags cannot be allowed by the PermitScrubber and will be scrubbed")
-            var.delete("noscript")
+            if var.include?("noscript")
+              warn("WARNING: 'noscript' tags cannot be allowed by the PermitScrubber and will be scrubbed")
+              var.delete("noscript")
+            end
           end
 
           var