Support for JSON input type in Smallrye Graphql

[pipinet]

https://github.com/graphql-java/graphql-java-extended-scalars

[debu999]

I did create on same lines 👍
#1773

[debu999]

Is there any update on this? @t1

[jmartisk]

With @mskacelik we tried experimenting with this, but hit an issue because JsonObject extends Map<String, JsonValue>, and for maps we already have a custom handling logic that is incompatible with this.

For example, a map containing a key "a" and value "b" is currently mapped to a GraphQL object like this

[
  {
     "key": "a",
     "value": "b"
  }
]

whereas the logical JSON object would be

{ "a": "b"}

we weren't yet sure how to properly resolve this conflict without breaking backwards compatibility.

[debu999]

May be a user defined property to enable map mapper will be good. Since everyone using smallrye-graphql are used to Map = [{key: a, value: b}] they should do a parsing as needed.
It might be something for inputs though to do parsing a JSON

[phillip-kruger]

Users can provide their own mapper for Maps, overriding the default built-in one

[cristianonicolai]

Also have a use case where we would need to allow for generic JSON to be passed on as part of the query input. We use dynamic variable names and paths that users could filter on, so using JSON scalar could be of benefit here.

[debu999]

I was reading how it was done in spqr this seems to be a mapper for GSON and JACKSON.
leangen/graphql-spqr@5244588