add the com.alibaba.fastjson2 support for HttpMessage Converter.
Comment From: pivotal-cla
@mek1986 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.
Comment From: pivotal-cla
@mek1986 Thank you for signing the Contributor License Agreement!
Comment From: bclozel
@mek1986 thanks for the PR, but discussing it beforehand would have been a good first step. I'm not sure the Spring wants to add another variant for this. Spring Boot starters are well designed for this and I think we might reduce the auto-detection in Framework in the future anyway. We will discuss that as a team and get back to you.
Comment From: mek1986
@mek1986 thanks for the PR, but discussing it beforehand would have been a good first step. I'm not sure the Spring wants to add another variant for this. Spring Boot starters are well designed for this and I think we might reduce the auto-detection in Framework in the future anyway. We will discuss that as a team and get back to you.
ok,thank you.
Comment From: wenshao
fastjson is very widely used in China, and fastjson2 is an upgraded version of fastjson. Spring's built-in support for fastjson will make it more convenient for Chinese users.
Comment From: bclozel
It looks like fastjson2 already offers excellent support for Spring Framework 5 and 6. Maybe the only missing part would be a Spring Boot starter to auto-configure this?
I'm not sure merging this would be an improvement. Instead of improving support it would be harder for the fastjson team to evolve the converter along the core support. Spring Framework has backwards compatibility constraints and cannot adopt the latest changes in libraries without raising the baseline.
Maybe we can link to fastjson support in the reference docs directly? This would make fastjson more discoverable to users worldwide (not just china) and would avoid the downsides of merging the support here.
Comment From: bclozel
We've discussed this and we think that a link from the Spring Boot community starters page would be a better fit, if in the future the fastjson team decides to release such an artifact with their project.