restTemplate大面积出现java.lang.IllegalArgumentException: ‘messageConverters’ must not contain null elements的问题
前文
最近在调用三方接口使用的restTemplate,之前一直没什么问题,近期公司的业务扩大,单量较多,所以就出现了大批量调用三方服务。随之一些隐藏的问题就出现了。
业务描述
公司主要做店铺,订单,仓储等等业务,我们有一个物流服务 主要是请求三方物流商获取运单号和面单信息,在订单服务使用mq发送消息去物流服务,然后用restTemplate去请求三方。restTemplate我们是使用的注入的方式
@Autowired
private RestTemplate restTemplate;
查看后台日志出现大面积这样的错误:
根据上面的报错信息不难看出我给restTemplate传过去有null,可是我看我得代码里面 不可能给他的参数有null啊,我都是经过判空处理的(图里面的参数我已经封装好了),上图:
根据报错的信息去层层查找。
根据报错的位置,我们上源码,红框里面是异常位置451行:
这里没什么问题,再去看看821行:
这里也没问题 再看991:
有个getMessageConverters()方法 我们进去看下:
可以看出往list里面已经初始化了变量。所以肯定不会存在null的情况;
接着往下走,再看这几行代码:
可以看出 我们的异常就是从这里抛出来的。‘messageConverters’ must not contain null elements。
再把这个方法点进去:
Assert.noNullElements(messageConverters, “‘messageConverters’ must not contain null elements”);
这里可以看出,它是把里面的list进行了遍历,如果某一个节点为null那么就会抛出异常。
到这里问题已经很明显了。就是List<HttpMessageConverter<?>> messageConverters 这里面的某一个元素为null导致的;我们再网往上看这里面初始化的代码:
这里也不可能为null啊。都是new了对象在里面;再回头看看自己的代码:
发现在代码里面加入了getMessageConverters()这行代码,可是这行代码加上也不至于让list里面的某一个元素为null啊,它每次调用的时候都会在该list的第一个位置加一个对象放在里面,list容量扩大。乍一看也没什么问题;可是我们要注意有两个关键点:1、ArrayList是线程不安全的 2、大批量的调用;于是我猜想会不会因为并发调用list.add()导致里面的元素更改或者直接add(null) 了 。现在结合这两点我们写一个demo来验证我得猜想:
猜想被证实;所以面对高并发的问题,还是仔细点的好,不然各种诡异问题让人焦头烂额;