hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2025-12-24 12:46:34 +01:00
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary TaggedTemplateEntryPoint

Browse Source
This commit is contained in:
Alvaro Muñoz
2022-10-25 11:44:45 +02:00
parent 37ea3f23f1
commit a80b691358
2 changed files with 0 additions and 89 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
javascript/ql/lib/semmle/javascript/frameworks/Spife.qll
View File

@@ -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.
*/

80
javascript/ql/test/library-tests/frameworks/Spife/tests.expected
View File

@@ -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 | '<p>' + ... '</p>' |
| 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 | '<p>' + ... '</p>' |
| 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 |