-
Notifications
You must be signed in to change notification settings - Fork 174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Circular input property leads to stackoverflow #409
Labels
Comments
KammererTob
changed the title
Circular input property leads to Stackoverflow
Circular input property leads to stackoverflow
Jun 22, 2020
Confirmed bug. Contributions welcome. |
KammererTob
added a commit
to KammererTob/graphql-java-tools
that referenced
this issue
Jul 4, 2020
fyi, simple (non-mutual) recursion also triggers the bug: input CriterionInput {
and: [CriterionInput!]
or: [CriterionInput!]
not: CriterionInput
status: StatusCriterionInput
}
input StatusCriterionInput {
eq: String
} |
Can confirm. This is especially important for filtering purposes like recursive OR and AND patterns. |
vojtapol
added a commit
that referenced
this issue
Dec 14, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After updating graphql-java-tools to 6.1.0 i have encountered a stackoverflow exception on startup. I think that the problem was introduced in this commit: 10ee43f and happens because of the interaction between
determineInputType
andcreateInputObject
.Example schema:
According to the spec this should be allowed (graphql/graphql-spec#445, graphql/graphql-spec#189), which leads me to believe that this is a bug.
The text was updated successfully, but these errors were encountered: