add support for the teleport-javascript library

Author: erik-krogh

javascript/change-notes/2021-06-24-json.md

@@ -4,4 +4,5 @@ lgtm,codescanning
     [json2csv](https://npmjs.com/package/json2csv),
     [json5](https://npmjs.com/package/json5),
     [prettyjson](https://npmjs.com/package/prettyjson),
-    [flatted](https://npmjs.com/package/flatted)
+    [flatted](https://npmjs.com/package/flatted),
+    [teleport-javascript](https://npmjs.com/package/teleport-javascript)

javascript/ql/src/semmle/javascript/JsonParsers.qll

@@ -26,7 +26,7 @@ private class PlainJsonParserCall extends JsonParserCall {
   PlainJsonParserCall() {
     exists(DataFlow::SourceNode callee | this = callee.getACall() |
       callee = DataFlow::globalVarRef("JSON").getAPropertyRead("parse") or
-      callee = DataFlow::moduleMember(["json3", "json5", "flatted"], "parse") or
+      callee = DataFlow::moduleMember(["json3", "json5", "flatted", "teleport-javascript"], "parse") or
       callee = DataFlow::moduleImport("parse-json") or
       callee = DataFlow::moduleImport("json-parse-better-errors") or
       callee = DataFlow::moduleImport("json-safe-parse") or

javascript/ql/src/semmle/javascript/JsonStringifiers.qll

@@ -11,7 +11,8 @@ class JsonStringifyCall extends DataFlow::CallNode {
   JsonStringifyCall() {
     exists(DataFlow::SourceNode callee | this = callee.getACall() |
       callee = DataFlow::globalVarRef("JSON").getAPropertyRead("stringify") or
-      callee = DataFlow::moduleMember(["json3", "json5", "flatted"], "stringify") or
+      callee =
+        DataFlow::moduleMember(["json3", "json5", "flatted", "teleport-javascript"], "stringify") or
       callee =
         DataFlow::moduleImport([
             "json-stringify-safe", "json-stable-stringify", "stringify-object",

javascript/ql/test/library-tests/TaintTracking/BasicTaintTracking.expected

@@ -93,6 +93,7 @@ typeInferenceMismatch
 | json-stringify.js:2:16:2:23 | source() | json-stringify.js:21:8:21:46 | new jso ... source) |
 | json-stringify.js:2:16:2:23 | source() | json-stringify.js:24:8:24:43 | json5.s ... ource)) |
 | json-stringify.js:2:16:2:23 | source() | json-stringify.js:27:8:27:47 | flatted ... ource)) |
+| json-stringify.js:2:16:2:23 | source() | json-stringify.js:30:8:30:49 | telepor ... ource)) |
 | json-stringify.js:3:15:3:22 | source() | json-stringify.js:8:8:8:31 | jsonStr ... (taint) |
 | nested-props.js:4:13:4:20 | source() | nested-props.js:5:10:5:14 | obj.x |
 | nested-props.js:9:18:9:25 | source() | nested-props.js:10:10:10:16 | obj.x.y |

javascript/ql/test/library-tests/TaintTracking/json-stringify.js

@@ -25,4 +25,7 @@ function foo() {
 
   const flatted = require('flatted');
   sink(flatted.stringify(flatted.parse(source))); // NOT OK
+
+  const teleport = require('teleport-javascript');
+  sink(teleport.stringify(teleport.parse(source))); // NOT OK
 }