Skip to content

Commit a80b691

Browse files
author
Alvaro Muñoz
committed
Remove unnecessary TaggedTemplateEntryPoint
1 parent 37ea3f2 commit a80b691

File tree

2 files changed

+0
-89
lines changed

2 files changed

+0
-89
lines changed

‎javascript/ql/lib/semmle/javascript/frameworks/Spife.qll

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,6 @@ import semmle.javascript.frameworks.HTTP
99
* Provides classes for working with [Spife](https://github.com/npm/spife) applications.
1010
*/
1111
module Spife {
12-
/**
13-
* An API graph entry point ensuring all tagged template exprs are part of the API graph
14-
*/
15-
private class TaggedTemplateEntryPoint extends API::EntryPoint {
16-
TaggedTemplateEntryPoint() { this = "TaggedTemplateEntryPoint" }
17-
18-
override DataFlow::SourceNode getASource() { result.asExpr() instanceof TaggedTemplateExpr }
19-
}
20-
2112
/**
2213
* A call to a Spife method that sets up a route.
2314
*/

‎javascript/ql/test/library-tests/frameworks/Spife/tests.expected

Lines changed: 0 additions & 80 deletions
Original file line numberDiff line numberDiff line change
@@ -1,83 +1,3 @@
1-
WARNING: Unused method getCredentialsHeader (tests.ql:70,26-46)
2-
test0
3-
| lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | content-type | application/json |
4-
| lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | content-type | application/json |
5-
| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | content-type | application/json |
6-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-credentials | true |
7-
| 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 |
8-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-methods | GET, POST, PUT, DELETE, OPTIONS |
9-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-origin | * |
10-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | content-type | text/html |
11-
| lib/views/index.js:53:1:55:1 | functio ... ource\\n} | content-type | application/json |
12-
| lib/views/index.js:56:1:58:1 | functio ... ition\\n} | content-type | application/json |
13-
| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | content-type | application/json |
14-
| lib/views/index.js:65:14:65:87 | reply.h ... /html') | content-type | text/html |
15-
| lib/views/index.js:67:14:67:92 | reply.h ... ain' }) | content-type | text/plain |
16-
| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | content-type | application/json |
17-
| lib/views/index.js:77:14:77:92 | reply.h ... ain' }) | content-type | text/plain |
18-
| lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | content-type | application/json |
19-
| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | content-type | application/json |
20-
| lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | content-type | application/json |
21-
| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | content-type | application/json |
22-
| lib/views/index.js:107:12:107:86 | reply(m ... tml' }) | content-type | text/html |
23-
| lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | content-type | application/json |
24-
test
25-
| lib/views/index.js:45:36:45:38 | "*" |
26-
test2
27-
| lib/views/index.js:43:16:43:29 | req.query.name |
28-
| lib/views/index.js:65:27:65:57 | '<p>' + ... '</p>' |
29-
| lib/views/index.js:107:18:107:24 | message |
30-
test3
31-
| lib/views/index.js:43:16:43:29 | req.query.name |
32-
| lib/views/index.js:65:27:65:57 | '<p>' + ... '</p>' |
33-
| lib/views/index.js:107:18:107:24 | message |
34-
test4
35-
| lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | content-type | application/json |
36-
| lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | content-type | application/json |
37-
| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | content-type | application/json |
38-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-credentials | true |
39-
| 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 |
40-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-methods | GET, POST, PUT, DELETE, OPTIONS |
41-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | access-control-allow-origin | * |
42-
| lib/views/index.js:43:10:50:4 | reply(r ... n\\n\\n }) | content-type | text/html |
43-
| lib/views/index.js:53:1:55:1 | functio ... ource\\n} | content-type | application/json |
44-
| lib/views/index.js:56:1:58:1 | functio ... ition\\n} | content-type | application/json |
45-
| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | content-type | application/json |
46-
| lib/views/index.js:65:14:65:87 | reply.h ... /html') | content-type | text/html |
47-
| lib/views/index.js:67:14:67:92 | reply.h ... ain' }) | content-type | text/plain |
48-
| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | content-type | application/json |
49-
| lib/views/index.js:77:14:77:92 | reply.h ... ain' }) | content-type | text/plain |
50-
| lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | content-type | application/json |
51-
| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | content-type | application/json |
52-
| lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | content-type | application/json |
53-
| lib/views/index.js:100:1:109:1 | functio ... n\\n }\\n} | content-type | application/json |
54-
| lib/views/index.js:107:12:107:86 | reply(m ... tml' }) | content-type | text/html |
55-
| lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | content-type | application/json |
56-
test5
57-
| lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | lib/views/index.js:23:1:26:1 | functio ... tned)\\n} | content-type |
58-
| lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | lib/views/index.js:28:1:39:1 | functio ... eBody\\n} | content-type |
59-
| lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | lib/views/index.js:41:1:51:1 | functio ... \\n })\\n} | content-type |
60-
| 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 |
61-
| 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 |
62-
| 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 |
63-
| 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 |
64-
| 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 |
65-
| lib/views/index.js:53:1:55:1 | functio ... ource\\n} | lib/views/index.js:53:1:55:1 | functio ... ource\\n} | content-type |
66-
| lib/views/index.js:56:1:58:1 | functio ... ition\\n} | lib/views/index.js:56:1:58:1 | functio ... ition\\n} | content-type |
67-
| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | content-type |
68-
| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | lib/views/index.js:65:14:65:87 | reply.h ... /html') | content-type |
69-
| lib/views/index.js:60:1:70:1 | functio ... ext.'\\n} | lib/views/index.js:67:14:67:92 | reply.h ... ain' }) | content-type |
70-
| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | content-type |
71-
| lib/views/index.js:72:1:80:1 | functio ... ext.'\\n} | lib/views/index.js:77:14:77:92 | reply.h ... ain' }) | content-type |
72-
| lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | lib/views/index.js:82:1:84:1 | functio ... !xss\\n} | content-type |
73-
| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | lib/views/index.js:86:1:91:1 | functio ... ition\\n} | content-type |
74-
| lib/views/index.js:86:1:91:1 | functio ... ition\\n} | lib/views/index.js:90:10:90:55 | reply(m ... sage }) | npm-notice |
75-
| lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | lib/views/index.js:93:1:98:1 | functio ... !xss\\n} | content-type |
76-
| 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 |
77-
| 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 |
78-
| 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 |
79-
| 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 |
80-
| lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | lib/views/index.js:111:1:115:1 | functio ... \\n })\\n} | content-type |
811
passingPositiveTests
822
| PASSED | candidateHandler | lib/views/index.js:82:32:82:56 | // test ... Handler |
833
| PASSED | cleartextStorageSink | lib/views/index.js:57:115:57:193 | // test ... inition |

0 commit comments

Comments
 (0)