raise panic when nested @requires are used on federation (#1655)

99designs · Oct 13, 2021 · 6758654 · 6758654
1 parent f6c35be
commit 6758654
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/plugin/federation/federation.go b/plugin/federation/federation.go
@@ -243,7 +243,12 @@ func (f *federation) setEntities(schema *ast.Schema) {
 					if dir == nil {
 						continue
 					}
-					fields := strings.Split(dir.Arguments[0].Value.Raw, " ")
+					args := dir.Arguments[0].Value.Raw
+					if strings.Contains(args, "{") {
+						// TODO: see. https://github.com/99designs/gqlgen/issues/1138
+						panic("Nested fields are not currently supported in @requires declaration.")
+					}
+					fields := strings.Split(args, " ")
 					requireFields := []*RequireField{}
 					for _, f := range fields {
 						requireFields = append(requireFields, &RequireField{