前言
近项目组升级了SpringBoot
版本,由之前的2.0.4
升级到新版本2.7.5
,却引出了一个大Bug。
到底是怎么回事呢?
1.案发现场
有一天,项目组的同事反馈给我说,我之前有个接口在新的测试环境报错了,具体异常是:Missing argment level for method parameter of type Integer
。
我当时的反应有点懵,心想这个接口是一个老接口
,有一年多的时间都没改过了,怎么会出问题呢?
他说近期另外一个同事为了部署阿里云服务器
,把新测试环境SpringBoot
的版本升级到了新版
。
之后,在测试的过程中,发现我有个Get
请求接口报异常了。
该接口代码类似于这样:
在getCategory接口中,有两个参数:
- type表示大类,是必传的。
- level表示要返回几级分类,比如:4级分类,就传4,是非必传的,默认就是查4级分类。
就是这样一个接口的level参数,前端没有传参,例如:
结果被Spring MVC
拦截直接报错了。
2 报错的原因
从打印的异常信息看,现在level参数必须要传值了,之前是可传,可不传的。
我后来本打算自定义Spring的转换器,修改一下校验规则,跟老版本保持一致。
这样那些基本接口就不用改了。
但后来发现,被spring-web-5.3.23的源码无情的打脸了。
在org.springframework.web.method.annotation
包下的AbstractNamedValueMethodArgumentResolver
类的resolveArgument
方法中:
多了这样的校验。如果该参数为空,没有设置默认值,required属性为true,并且不是Optional类型,则执行handleMissingValueAfterConversion
方法。
该方法会调用handleMissingValue
方法,具体代码如图中所示:
后会抛出之前我看到的那个异常。
原因新版本的Spring中不允许Get接口的请求参数,在不使用@RequestParam注解时,值为空的情况出现了。
3 如何解决问题?
想要解决上面的报错问题,其实很简单,只需在level参数前加@RequestParam
注解,并且设置required
属性为false
。
例如:
但是后面发现,项目中不只我这一个接口要调整,其他好多同事的接口,也有类似的问题,需要修改的接口很多。
这个改动的工作量不小。
哭晕在测试。。。
后话
很多人中招,所以非常有必要把这个问题分享给大家,防微杜渐。
我之前level参数不加@RequestParam
注解,也没设置required
属性,当时持有的心态是Spring有默认值
,有些注解不加程序也能正常运行,既然这样就可以少写点代码,并且在当时的版本测试过,没有出现过什么问题。
这种情况其实是Spring框架的一个bug,已经在新版本中被修复了。。。
赶紧review一下你们的代码,看看有没有类似的用法,不然迟早有一天也会中招。