Added support for kotlinx.serialization

[werner77]

This PR adds support for kotlinx.serialization via an implementation of ModelConverter.

[JesusMcCloud]

I might be doing things wrong, but when I added your code and explicitly calling ModelConverters.getInstance().addConverter(KotlinxSerializationTypeConverter(Json, false)) before runApplication, I still only get the default, messed-up Jackson serialization in Swagger-UI (and in the generated OpenAPI JSON), which obviously entails that custom serializers and@SerialNameannotations are ignored as well…

How can I debug this? How can I help?

[JesusMcCloud]

The last commit helped, but it's quite not there yet:

For example: we're using custom serializers for X509Certificate and it does show up correctly in the schemas. However, the request and responses, which contain certificates still use the default (faulty) serializers. Maybe the cause of this behaviour is related to X509Certificate being a platform type

[werner77]

Yes right now it only considers types which are annotated with '@serializable'. I can work on a fix for that.

[JesusMcCloud]

Put differently, What's missing is properties annotated with @Serializable(with=…. Another Thing: Does your code dealt with the (experimental) @Serializer(forClass = …) Annotation?

[bnasslahsen]

@werner77,

Have you finished your implementation ?
If it's the case. Could you also consider adding corresponding unit tests to the project?

[werner77]

When I have time (hopefully soon) I will fix the open issues and add some unit tests.

[bnasslahsen]

closed as no activity