Early return in shouldIncludeNode if directives empty

99designs · Apr 17, 2019 · a74abc4 · a74abc4
1 parent 1c95d42
commit a74abc4
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/graphql/exec.go b/graphql/exec.go
@@ -111,6 +111,10 @@ func getOrCreateField(c *[]CollectedField, name string, creator func() Collected
 }
 
 func shouldIncludeNode(directives ast.DirectiveList, variables map[string]interface{}) bool {
+	if len(directives) == 0 {
+		return true
+	}
+
 	skip, include := false, true
 
 	if d := directives.ForName("skip"); d != nil {