feiniaojin / graceful-response

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

添加@NotUseGracefulResponse注解, 对不需要处理的接口单独判断 #18

Closed TOP-LH closed 1 year ago

TOP-LH commented 1 year ago

添加@NotUseGracefulResponse注解, 对不需要处理的接口单独判断

feiniaojin commented 1 year ago

辛苦兄弟了。

这个地方原来之所以不考虑加排除,主要是因为在一个项目中,一般的接口都会统一格式,不统一格式的工程虽然不常见,但是也是有,这个PR补充了这部分的功能,非常好,为你点赞。

对这个功能,我的建议是:(1)如果要加这样的功能,那光是加了这个注解还不够,还需要提供包、类层面的配置,例如某个包路径下、某个类下的接口都过滤掉,因为一旦有这种情况,就不会只有一个方法需要排除,通常都会很多,逐个加不太现实;(2)注解的名字可以推敲一下,例如叫@ExcludeFromGracefulResponse

TOP-LH commented 1 year ago

可以, 这个不难, 我开发一下

TOP-LH commented 1 year ago

辛苦兄弟了。

这个地方原来之所以不考虑加排除,主要是因为在一个项目中,一般的接口都会统一格式,不统一格式的工程虽然不常见,但是也是有,这个PR补充了这部分的功能,非常好,为你点赞。

对这个功能,我的建议是:(1)如果要加这样的功能,那光是加了这个注解还不够,还需要提供包、类层面的配置,例如某个包路径下、某个类下的接口都过滤掉,因为一旦有这种情况,就不会只有一个方法需要排除,通常都会很多,逐个加不太现实;(2)注解的名字可以推敲一下,例如叫@ExcludeFromGracefulResponse

已优化, 你看看能否满足

  1. 包过滤在properties上添加了excludeFromGracefulResponsePackage字段, 支持list, 支持*号匹配
  2. 类我感觉没必要, 在使用包过滤和注解过滤就可以满足了
  3. 如果可以的话可以拉我到提供者, 我还有很多想法的 谢谢
TOP-LH commented 1 year ago

辛苦兄弟了。 这个地方原来之所以不考虑加排除,主要是因为在一个项目中,一般的接口都会统一格式,不统一格式的工程虽然不常见,但是也是有,这个PR补充了这部分的功能,非常好,为你点赞。 对这个功能,我的建议是:(1)如果要加这样的功能,那光是加了这个注解还不够,还需要提供包、类层面的配置,例如某个包路径下、某个类下的接口都过滤掉,因为一旦有这种情况,就不会只有一个方法需要排除,通常都会很多,逐个加不太现实;(2)注解的名字可以推敲一下,例如叫@ExcludeFromGracefulResponse

已优化, 你看看能否满足

  1. 包过滤在properties上添加了excludeFromGracefulResponsePackage字段, 支持list, 支持*号匹配
  2. 类我感觉没必要, 在使用包过滤和注解过滤就可以满足了
  3. 如果可以的话可以拉我到提供者, 我还有很多想法的 谢谢

最好测试一下功能