Have an hierarchy for the MessageConverter based-ObjectMapper in Spring AMQP similar to Spring Web

[mhewedy]

Currently, the Jackson2JsonMessageConverter is tied with the JSON and we don't have a Jackson2 hierarchy abstraction as AbstractJackson2HttpMessageConverter of spring-mvc module.

We need to implement Jackson2XmlMessageConverter and create a new abstraction AbstractJackson2HttpMessageConverter to act as a parent for the new Jackson2XmlMessageConverter and the existing Jackson2JsonMessageConverter.

[artembilan]

Well, from big height your concern isn't clear and there are some typos in your post, but still I can guess that you would like to have a variant of the Jackson2JsonMessageConverter which can work exactly with XML. In other words you want to have an hierarchy for the ObjectMapper here in Spring AMQP similar what we have in Spring Web with the AbstractJackson2HttpMessageConverter.

As you said yourself the current Jackson2JsonMessageConverter is tied with the JSON, but that is exactly in its name. So, what we need is to extract some abstract AbstractJackson2MessageConverter and implement a Jackson2XmlMessageConverter.

If that is what you have in mind, then it is a good suggestion and contribution is always welcome.

Otherwise your issue is not clear. Sorry...

[mhewedy]

That is exactly what I meant, sorry for my question being not clear enough.

I would prepare and send a PR. Thanks.

[artembilan]

Good to hear.
Please, edit your issue to reflect the reality of the request.

My main concern that we can't modify an AbstractJackson2HttpMessageConverter here because it is a part of Spring Web, not Spring AMQP.

[mhewedy]

Noted; thanks.

…

On Wed, 11 Jul 2018, 4:26 pm Artem Bilan, ***@***.***> wrote: Good to hear. Please, edit your issue to reflect the reality of the request. My main concern that we can't modify an AbstractJackson2HttpMessageConverter here because it is a part of Spring Web, not Spring AMQP. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#768 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABCSYY2qJfTu_eC2J6rSBqccslQJWJAfks5uFfzogaJpZM4VGsbh> .