feiniaojin / graceful-response

Spring Boot接口响应处理解决方案,提供统一返回值封装、全局异常处理、自定义异常错误码、参数校验增强、断言增强等功能
https://doc.feiniaojin.com
MIT License
1.09k stars 164 forks source link

Feign 响应解析 #6

Closed senpal220 closed 1 year ago

senpal220 commented 1 year ago

解析 Feign 的响应,直接声明 Payload 对应的 Bean 做为返回值。 ResponseStatus 没有扩展的需求,也不利于 Response 的反序列化(多态属性),由接口改为类。

feiniaojin commented 1 year ago

这个想法非常好,赞!不过不能直接合进来,因为有的用户工程里面是没有Feign的,也不是spring cloud的工程,我们不能强行给他们引进这些包。 我建议单独创建一个扩展的包,pom里面引入graceful-response,在这个包里面去完成本次PR的逻辑,这样能减少影响。 另外,我看到您对这个工程非常的投入,有没有兴趣加入啊?

senpal220 commented 1 year ago

@ConditionalOnClass 条件装配,有 feign 依赖才会启用。 我是发现了些没覆盖的使用场景才提出想法,^_^。

feiniaojin commented 1 year ago

问题不在于@ConditionalOnClass 条件装配,而是在于引入feign这个包。虽然有这个注解,但是pom打包的时候会引入,有很多应用不会用到feign,所以需要想办法拆成两个包

senpal220 commented 1 year ago

有编译期依赖,没有运行期依赖。 provided 依赖,打包不会把 feign 打进去。