null values in body are not passed through to graphql backend

[wundergraph]

null values in body are not passed through to graphql backend

[jensneuse]

Sounds reasonable to me, although the reason is not performance. In some cases, you don't want to send nulls because the server handles them differently than empty fields, e.g. in a patch.

[jensneuse]

Thanks for reporting. We've got people who wanted to have the exact opposite behaviour. I guess we should make this configurable. Do you think this need to have granular configuration or could it be just at the datasource level? E.g. switch it on or off via boolean config param for the whole origin.

[greemo]

Hi Jens, thanks for the quick response.

In GraphQL, one can have different semantics for non-inclusion vs inclusion as null. Therefore I believe the default should be to pass nulls through. Otherwise an intentional feature of GraphQL backends is not available through Wundergraph.

I see this filtering null values as an additional filtering feature, rather than the way the system should behave by default.

If someone wishes to have this filtering nulls behaviour, and they want it in the Go codebase for performance reasons, rather than a hook, I can see the value in that. I imagine enabling the filter for the entire datasource would be sufficient. I would do it wherever it's easiest and most performant.

[greemo]

Yes, this is the reason I meant.
Regarding performance, I meant doing the filtering in the Go code instead of in the Node hooks, where it could also be done. :)

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[jensneuse]

Hey @spetrunin nice find :-)