Skip to content

Commit 1593573

Browse files
committed
JS: Remove use of types in ClassValidator
1 parent 8bed653 commit 1593573

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

javascript/ql/lib/javascript.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ import semmle.javascript.NodeJS
4747
import semmle.javascript.NPM
4848
import semmle.javascript.Paths
4949
import semmle.javascript.Promises
50-
import semmle.javascript.CanonicalNames
50+
deprecated import semmle.javascript.CanonicalNames
5151
import semmle.javascript.RangeAnalysis
5252
import semmle.javascript.Regexp
5353
import semmle.javascript.Routing

javascript/ql/lib/semmle/javascript/frameworks/ClassValidator.qll

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
*/
44

55
import javascript
6+
private import semmle.javascript.internal.TypeResolution
7+
private import semmle.javascript.internal.UnderlyingTypes
68

79
/**
810
* Provides predicates for reasoning about sanitization via the `class-validator` library.
@@ -50,7 +52,7 @@ module ClassValidator {
5052

5153
pragma[noinline]
5254
private ClassDefinition getClassReferencedByPropRead(DataFlow::PropRead read) {
53-
read.getBase().asExpr().getType().unfold().(ClassType).getClass() = result
55+
UnderlyingTypes::nodeHasUnderlyingClassType(read.getBase().asExpr(), result.flow())
5456
}
5557

5658
/**

0 commit comments

Comments
 (0)