restTemplate

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来验证我得猜想：

猜想被证实；所以面对高并发的问题，还是仔细点的好，不然各种诡异问题让人焦头烂额；