Re: Upgrade graphgql-java to 17.4

2023-02-13 Thread Andreas Schaefer
Hi Just an updated on the development of moving to a newer version of grqphql-java: Testing with Sling Graphql Core and AEM I did test graphql-java 17.4 and 20.0 and both versions seems to work just fine even though 20.0 does lead to a PAX testing issue in Sling Graphql Core but I think that’s

Re: Upgrade graphgql-java to 17.4

2023-02-03 Thread Andreas Schaefer
Right now I am having issues with the SelectionSetWrapper and SelectedFieldWrapper. The way graphql-java is providing the DataFetchingFieldSelectionSet is quite different and I am not sure how to handle it. The big difference is that there are no sub fields anymore and also no InlineFragment

Re: Upgrade graphgql-java to 17.4

2023-02-03 Thread Radu Cotescu
Hi Andy, > On 3 Feb 2023, at 16:40, Andreas Schaefer wrote: > > Hi > > I am tasked with upgrading grqaphql-java for AEM to at least 17.4 because of > a vulnerability https://nvd.nist.gov/vuln/detail/CVE-2022-37734 > > I saw that Radu created a ticket for upgrading to graphql-java 17: >

Upgrade graphgql-java to 17.4

2023-02-03 Thread Andreas Schaefer
Hi I am tasked with upgrading grqaphql-java for AEM to at least 17.4 because of a vulnerability https://nvd.nist.gov/vuln/detail/CVE-2022-37734 I saw that Radu created a ticket for upgrading to graphql-java 17: https://github.com/graphql-java/graphql-java/releases/tag/v17.0 I assigned that