Patrick Strawderman
Patrick Strawderman
You're right. I've added support for it in #1800
Hey @ehardy, thank you for the PR and apologies for the slow movement on this. There is a larger refactor of the `InputObjectMapper` that I hope to land soon that...
@ehardy FYI the aforementioned refactoring has landed see #1735
I confirmed locally with the example Kotlin apps that exceptions thrown from suspend functions are no longer wrapped in `InvocationTargetException`.
Could we take a step back and discuss how this is intended to work end-to-end? What it sounds like to me in this proposal is that the `DefaultInputObjectMapper` would need...
I'd just mention that you can do this today by just checking the `DataFetchingEnvironment`. E.g., ```kotlin dfe.getArgument("input").containsKey("foo") ``` would tell you whether a null value was supplied explicitly or not....
> * DGS runtime: DefaultInputObjectMapper shouldn't be changed to accommodate this feature, especially calling a private method using reflection > * DGS CodeGen: > > * Avoid bitset, instead create...
Oh, one other issue, is that mapping for Kotlin types doesn't use setters at all currently. There is a [ticket open](https://github.com/Netflix/dgs-framework/issues/1057) for that, I think we can fix it in...
> There will be absolutely NO changes to ObjectMappers. Any JSON deserializer (not only Jackson, but any other) will work with this out of the box. All changes are encapsulated...
> > Oh, one other issue, is that mapping for Kotlin types doesn't use setters at all currently. There is a [ticket open](https://github.com/Netflix/dgs-framework/issues/1057) for that, I think we can fix...