Hi, I am using graphql-kotlin client. I am having ...

# graphql-kotlin

[Dušan Salay]

Hi, I am using graphql-kotlin client.
I am having issue with parsing this graphql response which sadly has non standart UpperCases of attributes:

{
  "Data": {
    "CreateIssuedInvoice": {
      "ID": "62d0e9c6-1c54-4324-8ac5-716701b325be",
      "Stav": 0,
      "CisloDokladu": "240010989"
    }
  },
  "Status": 1,
  "Message": "",
  "StackTrace": ""
}

but default deserialisation of GraphQLWebClient is unable to parse it.
I assume that’s is because of first uppercase letter? Is here any possible workaround how to fix that?

[Dušan Salay]

adding @get:JsonProperty on all response fields helped.
like: @get:JsonProperty("CreateIssuedInvoice")
Do you think that it will be possible to add that also for generated responses? As you did it here for requests.

[Dušan Salay]

in my case I also had to add

.configure(MapperFeature.ACCEPT_CASE_INSENSITIVE_PROPERTIES, true)

for Jackson object mapper to pass Data attribute with first uppercase.

[Dariusz Kuc]

👋 so the Data response is invalid per GraphQL spec and it should be data but I think annotating the response fields should be fine

[Dariusz Kuc]

PRs are always welcome

[Dušan Salay]

alright, thanks. I will prepare PR.

[Dušan Salay]

Hi, I kicked up first draft:
https://github.com/ExpediaGroup/graphql-kotlin/pull/1976
Can you check if change is ok? 🙏

[Dariusz Kuc]

👋 I was just thinking about it and wondering why do you actually need to annotate the response?

[Dariusz Kuc]

are you serializing it AGAIN after fetching it through a client?

[Dariusz Kuc]

*as otherwise it "should" be working fine

[Dariusz Kuc]

**I think

[Dušan Salay]

Hi, sry I was off this weekend.
I am just using deseralization of response in GraphQLWebClient. But Without @JsonProperty annotation it does not work. Names of properties in my response are really non-standart. Like Firstuppercase, ALLUPPERCASE or FirstUpAndThenCamelCase

[Dušan Salay]

But I assume there is no big disadvantage to have the annotation above every field?

[Dariusz Kuc]

guess it should be fine

[Dariusz Kuc]

i'll get to the review tomorrow