-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
fix: Updated dependencies to resolve security vulnerabilities #1046
fix: Updated dependencies to resolve security vulnerabilities #1046
Conversation
Thanks @upendrao for the contribution! |
@upendrao I've merged your PR and for some reason build in develop branch is now failing: https://github.com/kobylynskyi/graphql-java-codegen/actions/runs/4436709200/jobs/7785474765 |
@kobylynskyi I'm looking at the failure and shall submit a PR on that. |
Hi @kobylynskyi ! Can I ask you a question here? |
hey @upendrao |
Hi,
If I use 'extend type Query {' then I think I need to implement that query
on my own as it wont exist on GraphQL server.
For example I have a query as follows then I want to generate Java/Kotlin
code for that query so that I can easily deserialize the response into it.
query {
listDomains(
input: {}
) {
start
count
total
domains {
urn
properties {
name
}
}
}
}
The types defined in the base schema (ex: Domain here) have a mix of
nullable and non-nullable fields.
But in a query projection we (want to)list only a subset of the fields of a
specific type.
I am experiencing a challenge while deserializing such types when the
projection does not include a non-nullable field.
Is there any fix for this on the json codec?
Best Regards,
Upendra
…On Fri, Mar 31, 2023 at 3:30 PM Bogdan Kobylynskyi ***@***.***> wrote:
hey @upendrao <https://github.com/upendrao>
few questions:
1 - There's one folder with all the schemas and you created another file
with your own queries?
2 - Does your schema file have an extend keyword? If you have queries
spread across multiple files you would want to have extend type Query {
... } in every file.
3 - If the answer to questions 1 and 2 is *yes*, but you still don't see
a code generated for your custom schema, then try keeping only your schemas
in the folder (deleting all other) and see if classes are generated. You
can investigate from there.
Let me know how it goes.
—
Reply to this email directly, view it on GitHub
<#1046 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA7BMZXCURMXBVX7LNWOWIDW63L7FANCNFSM6AAAAAAV5CYIEQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Description
Related to #1045
Updated following dependencies to resolve security vulnerabilities
com.graphql-java:graphql-java
com.fasterxml.jackson.core:jackson-databind
Updated unit tests to be able to parse test schemas with more tokens than the default size(15000)
Changes were made to: