Ruby: drop SanitizerIn from ClearTextLoggingQuery

Author: alexrford

ruby/ql/lib/codeql/ruby/security/CleartextLoggingCustomizations.qll

@@ -37,11 +37,6 @@ module CleartextLogging {
    */
   abstract class Sanitizer extends DataFlow::Node { }
 
-  /**
-   * A node that receives sanitized sensitive information.
-   */
-  abstract class SanitizerIn extends DataFlow::Node { }
-
   /**
    * Holds if `re` may be a regular expression that can be used to sanitize
    * sensitive data with a call to `sub`.
@@ -83,7 +78,7 @@ module CleartextLogging {
    * Logger.new(STDOUT).info password
    * ```
    */
-  private class MaskingReplacerSanitizedNode extends SanitizerIn {
+  private class MaskingReplacerSanitizedNode extends Sanitizer {
     MaskingReplacerSanitizedNode() {
       exists(Ssa::Definition def |
         exists(MaskingReplacerSanitizer maskCall |

ruby/ql/lib/codeql/ruby/security/CleartextLoggingQuery.qll

@@ -28,10 +28,6 @@ class Configuration extends TaintTracking::Configuration {
     node instanceof CleartextLogging::Sanitizer
   }
 
-  override predicate isSanitizerIn(DataFlow::Node node) {
-    node instanceof CleartextLogging::SanitizerIn
-  }
-
   override predicate isAdditionalTaintStep(DataFlow::Node nodeFrom, DataFlow::Node nodeTo) {
     CleartextLogging::isAdditionalTaintStep(nodeFrom, nodeTo)
   }