From 6758654c4e28dc0589147c9e962c9d4c1fd44705 Mon Sep 17 00:00:00 2001 From: Masahiro Wakame Date: Wed, 13 Oct 2021 22:19:36 +0900 Subject: [PATCH] raise panic when nested @requires are used on federation (#1655) --- plugin/federation/federation.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/plugin/federation/federation.go b/plugin/federation/federation.go index 7d9abc9774..cdcea8c4f0 100644 --- 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{