diff --git a/javascript/ql/lib/semmle/javascript/frameworks/Spife.qll b/javascript/ql/lib/semmle/javascript/frameworks/Spife.qll index a97eb98dc29..f2e738d80b6 100644 --- a/javascript/ql/lib/semmle/javascript/frameworks/Spife.qll +++ b/javascript/ql/lib/semmle/javascript/frameworks/Spife.qll @@ -9,15 +9,6 @@ import semmle.javascript.frameworks.HTTP * Provides classes for working with [Spife](https://github.com/npm/spife) applications. */ module Spife { - /** - * An API graph entry point ensuring all tagged template exprs are part of the API graph - */ - private class TaggedTemplateEntryPoint extends API::EntryPoint { - TaggedTemplateEntryPoint() { this = "TaggedTemplateEntryPoint" } - - override DataFlow::SourceNode getASource() { result.asExpr() instanceof TaggedTemplateExpr } - } - /** * A call to a Spife method that sets up a route. */ diff --git a/javascript/ql/test/library-tests/frameworks/Spife/tests.expected b/javascript/ql/test/library-tests/frameworks/Spife/tests.expected index dc4c3451ce0..1f7ecf26ef8 100644 --- a/javascript/ql/test/library-tests/frameworks/Spife/tests.expected +++ b/javascript/ql/test/library-tests/frameworks/Spife/tests.expected @@ -1,83 +1,3 @@ -WARNING: Unused method getCredentialsHeader (tests.ql:70,26-46) -test0 -| lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | content-type | application/json | -| lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | content-type | application/json | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | content-type | application/json | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-credentials | true | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-headers | Content-Type, Authorization, Content-Length, X-Requested-With | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-methods | GET, POST, PUT, DELETE, OPTIONS | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-origin | * | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | content-type | text/html | -| lib/views/index.js:53:1:55:1 | functio ... ource\\n} | content-type | application/json | -| lib/views/index.js:56:1:58:1 | functio ... ition\\n} | content-type | application/json | -| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | content-type | application/json | -| lib/views/index.js:65:14:65:87 | reply.h ... /html') | content-type | text/html | -| lib/views/index.js:67:14:67:92 | reply.h ... ain' }) | content-type | text/plain | -| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | content-type | application/json | -| lib/views/index.js:77:14:77:92 | reply.h ... ain' }) | content-type | text/plain | -| lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | content-type | application/json | -| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | content-type | application/json | -| lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | content-type | application/json | -| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | content-type | application/json | -| lib/views/index.js:107:12:107:86 | reply(m ... tml' }) | content-type | text/html | -| lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | content-type | application/json | -test -| lib/views/index.js:45:36:45:38 | "*" | -test2 -| lib/views/index.js:43:16:43:29 | req.query.name | -| lib/views/index.js:65:27:65:57 | '
' + ... '
' | -| lib/views/index.js:107:18:107:24 | message | -test3 -| lib/views/index.js:43:16:43:29 | req.query.name | -| lib/views/index.js:65:27:65:57 | '' + ... '
' | -| lib/views/index.js:107:18:107:24 | message | -test4 -| lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | content-type | application/json | -| lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | content-type | application/json | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | content-type | application/json | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-credentials | true | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-headers | Content-Type, Authorization, Content-Length, X-Requested-With | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-methods | GET, POST, PUT, DELETE, OPTIONS | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-origin | * | -| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | content-type | text/html | -| lib/views/index.js:53:1:55:1 | functio ... ource\\n} | content-type | application/json | -| lib/views/index.js:56:1:58:1 | functio ... ition\\n} | content-type | application/json | -| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | content-type | application/json | -| lib/views/index.js:65:14:65:87 | reply.h ... /html') | content-type | text/html | -| lib/views/index.js:67:14:67:92 | reply.h ... ain' }) | content-type | text/plain | -| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | content-type | application/json | -| lib/views/index.js:77:14:77:92 | reply.h ... ain' }) | content-type | text/plain | -| lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | content-type | application/json | -| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | content-type | application/json | -| lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | content-type | application/json | -| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | content-type | application/json | -| lib/views/index.js:107:12:107:86 | reply(m ... tml' }) | content-type | text/html | -| lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | content-type | application/json | -test5 -| lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | content-type | -| lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | content-type | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | content-type | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-credentials | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-headers | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-methods | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-origin | -| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | content-type | -| lib/views/index.js:53:1:55:1 | functio ... ource\\n} | lib/views/index.js:53:1:55:1 | functio ... ource\\n} | content-type | -| lib/views/index.js:56:1:58:1 | functio ... ition\\n} | lib/views/index.js:56:1:58:1 | functio ... ition\\n} | content-type | -| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | content-type | -| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | lib/views/index.js:65:14:65:87 | reply.h ... /html') | content-type | -| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | lib/views/index.js:67:14:67:92 | reply.h ... ain' }) | content-type | -| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | content-type | -| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | lib/views/index.js:77:14:77:92 | reply.h ... ain' }) | content-type | -| lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | content-type | -| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | lib/views/index.js:86:1:91:1 | functio ... ition\\n} | content-type | -| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | lib/views/index.js:90:10:90:55 | reply(m ... sage }) | npm-notice | -| lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | content-type | -| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | content-type | -| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | lib/views/index.js:105:12:105:57 | reply(m ... sage }) | npm-notice | -| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | lib/views/index.js:107:12:107:86 | reply(m ... tml' }) | content-type | -| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | lib/views/index.js:107:12:107:86 | reply(m ... tml' }) | npm-notice | -| lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | content-type | passingPositiveTests | PASSED | candidateHandler | lib/views/index.js:82:32:82:56 | // test ... Handler | | PASSED | cleartextStorageSink | lib/views/index.js:57:115:57:193 | // test ... inition |