RicoSuter / Namotion.Messaging

.NET abstractions and implementations for message brokers, event queues and data ingestion services.
MIT License
46 stars 8 forks source link

Rename Namotion.Messaging.Json to Namotion.Messaging.Newtonsoft.Json #18

Open RicoSuter opened 3 years ago

SeanFarrow commented 2 years ago

As someone who is about to use this and requires System.Text.Json support, I'd be happy to do this as well as making it possible to provide serializer settingsto the currently implemented message retriever and publisher.

Ideally, we should do the following: Firstly, Rename the project in this issue. Secondly, extract any implementations of IMessageReceiver and IMessagePublisher to a new project, tentatively Namotion.Messaging.Json. This project also needs a way of being able to inject serializer settings.

To do this I am suggesting an interface such as IProvideJsonSerializerSettings. This interface should have a method GetSerializerSettings, Which is generic and returns the settings for either System.Text.Json or NewtonSoft.Json, depending on the implementation.

We then need another project Namotion.Messaging.Json.SystemTextJson which has an implementation of the previously described interface.

Does that sound a good idea?