Skip to content

Commit fd4233e

Browse files
committed
Moved apollo modeling to MaD
1 parent 84ffbbe commit fd4233e

File tree

3 files changed

+14
-38
lines changed

3 files changed

+14
-38
lines changed

javascript/ql/lib/ext/apollo-server.model.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,12 @@ extensions:
55
data:
66
- ["@apollo/server", "Member[ApolloServer,ApolloServerBase].Argument[0].AnyMember.AnyMember.AnyMember.Parameter[1]", "remote"]
77

8+
- addsTo:
9+
pack: codeql/javascript-all
10+
extensible: sinkModel
11+
data:
12+
- ["@apollo/server", "Member[gql].Argument[0]", "sql-injection"]
13+
814
- addsTo:
915
pack: codeql/javascript-all
1016
extensible: typeModel
@@ -13,3 +19,9 @@ extensions:
1319
- ["@apollo/server", "apollo-server-express", ""]
1420
- ["@apollo/server", "apollo-server-core", ""]
1521
- ["@apollo/server", "apollo-server", ""]
22+
- ["@apollo/server", "@apollo/apollo-server-express", ""]
23+
- ["@apollo/server", "apollo-server-express", ""]
24+
- ["@apollo/server", "@apollo/server", ""]
25+
- ["@apollo/server", "@apollo/apollo-server-core", ""]
26+
- ["ApolloServer", "@apollo/server", "Member[ApolloServer]"]
27+
- ["GraphQLApollo", "@apollo/server", "Member[gql]"]

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

Lines changed: 0 additions & 36 deletions
This file was deleted.

javascript/ql/lib/semmle/javascript/security/CorsPermissiveConfigurationCustomizations.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
*/
66

77
import javascript
8-
private import semmle.javascript.frameworks.Apollo
98
private import semmle.javascript.frameworks.Cors
109

1110
/** Module containing sources, sinks, and sanitizers for overly permissive CORS configurations. */
@@ -109,7 +108,8 @@ module CorsPermissiveConfiguration {
109108
*/
110109
class CorsApolloServer extends Sink, DataFlow::ValueNode {
111110
CorsApolloServer() {
112-
exists(Apollo::ApolloServer agql |
111+
exists(API::NewNode agql |
112+
agql = ModelOutput::getATypeNode("ApolloServer").getAnInstantiation() and
113113
this =
114114
agql.getOptionArgument(0, "cors").getALocalSource().getAPropertyWrite("origin").getRhs()
115115
)

0 commit comments

Comments
 (0)