记一次异步处理导致Jetty Request对象泄漏
最近排查一个bug，发现了一系列有意思的东西，对「自定义线程池」、「Jetty线程模型」都有了一些新的认识。
本文预计阅读时间10分钟，包括：
问题表现
常见原因筛查
根因与源码分析
最佳实践
一些小TIPS
1、问题表现
预发环境偶发请求失败异常，服务端显示错误信息为：
Required String parameter 'seriesbaid' is not present对应controller的api为
乍一看，是一个非常简单的异常，请求参数里面没有带seriesbaid，导致失败。
但是，经过确认，前端请求参数已经携带了seriesbaid，而且为“偶发失败”，并不是常见的参数传递问题。
2、常见原因筛查
2.1 网关参数丢失？
由于前端请求到达后端服务中会经过网关，所以一开始怀疑是否网关丢失了传递参数。
经过 调用链分析，在偶发的失败的请求中，也确认已经传递了querystring。所以网关没有丢失参数传递。
2.2 特殊字符导致参数转换失败 ？
既然已经传递了querystring到后端服务，那么一种常见的原因，由于queryString中带有特殊字符而导致解析成queryParam失败了。
会是这个问题吗？
我们通过在服务中继承一个spring-web的OncePerRequestFilter，对请求参数进行打印。
在偶发的失败的请求中，找到了以下日志
2021-12-29 15:36:05,536 INFO [com.xxx.interceptor.RequestLoggingFilter] - shouldLog - swanparameter:traceId:fb2266d3687911ecb5f3cf045ea19ac3; query:seriesbaid=3FO4K4SLX2IW&x_plugin=custom&x_bz=&locale=zh_CN&x_resourceId=&x_resourceVersion=; parameterMap:{};比较遗憾，我们确认了请求中确实有querystring而没有成功解析为queryParam，但是这个querystring中，并没有期望的特殊字符，讲道理是可以解析成功的。
既然常见原因无法解释，只能去源码捞一把了。
2.3 去源码捞一把
我们的网络容器使用了jetty，所以HttpServletRequest的实现是jetty的Request类。
Request类中，对queryString的解析是在 getParameters() 的时候。
我们发现，当异常请求进来的时候，这里的判断
_queryParameter竟然不是null，而是一个空对象。
而正常请求，这里判断_queryParameter为null，然后进行解析。
所以，还是要从源码去分析了。
3、根因与源码分析
3.1 _queryParamter为什么不是null了？
我们通过在Request类中设置多个断点，找到了原因。整理过程如下图所示。
1）同步请求A快速完成返回。
当请求A进来，在一次Http请求结束后（controller方法返回客户端），会进行相应的recycle()操作，这里包括Requst对象执行recycle()方法，清理相关参数，包括_queryParameters。
2）异步任务延迟响应，在recycle()后重新设置了_queryParameter属性。
在请求A执行过程中，使用「自定义线程池」异步执行了一个方法B（方法较慢）。方法B中，从RequestContextHolder中获取了HttpServletRequest，然后通过request.getParameter()获取请求头。
因为此时_queryParameters为null，因此extractQueryParameters()方法就解析了一个空的对象放进去。
3）新请求C进入，返回异常。
当新的请求C进入后端服务，拿到了同一个Request对象，由于此时_queryParameters不为null，因此跳过了extractQueryParameters()，导致应该解析的queryString无法被解析，controller抛出异常。
总结：一旦主线程执行完毕，完成recycle过程，而异步线程执行较慢，异步线程中的任何request.getParameter()行为会破坏Request对象的recycle，导致_queryParameters属性为空对象而不是null，从而导致新的请求失败。
3.2 异步线程中，RequestContextHolder还能拿到Request对象？（根本原因）
我们知道RequestContextHolder是基于ThreadLocal实现的。因此，在异步线程中，是无法直接通过
RequestContextHolder.getRequestAttributes()获取主线程的HttpServletRequest。
问题出在了「自定义线程池」
ThreadPoolExecutorWithMonitor中。
里面自定义实现了一个内部类DecorateRunnableTask来处理任务。
内部类DecorateRunnableTask继承了内部类DecorateTask，保存了主线程的RequestAttributes对象。
然后在异步线程执行前，通过before()方法设置到了当前线程的RequestContextHolder中。
总结：给异步线程传递RequestAttributes对象，是造成Request对象泄漏的根本原因！
3.3 两个请求，为什么会共享一个Request对象？
本来上面的分析基本已经找到了Bug的原因，但是我仔细想了下，又觉得有点奇怪。
两个请求，为什么会共享一个Request对象？
如果是使用了相关池化技术，那怎么能在两个请求找到同一个对象，然后稳定复现呢？因此，又继续去研究了下jetty的相关内容。
jetty 9.x整体架构图：
SelectorManager + ManagedSelector +QueuedThreadPool 组成了「Reactor线程模型」。对于一个http请求，SelectorManager分配给某一个ManagedSelector创建HttpConnection对象，然后在QueuedThreadPool中执行相应的IO操作。
HttpConnection对象持有HttpChannel对象，HttpChannel中持有了Request对象（就是HttpServletRequest）。
网关到后端服务之间使用的是Http请求，默认为长连接，因此，在短时间内的新的请求（长连接结束前），会复用同一个HttpConnection对象。
4、最佳实践
不要给异步线程传递RequestAttributes对象并进行保存。
如果需要相关请求参数，可以新建上下文对象存储参数后进行传递。或者使用TransmittableThreadLocal。
5、一些小TIPS
5.1 jetty源码不匹配
在对jetty的Request类进行debug时，一开始这里遇到一个小坑，idea一直源码匹配不上。从github上把 jetty源码拉下来，按照引入的jetty版本进行本地mvn install，还是不一致。
根据pom的依赖分析，可以看到引入的jetty版本为9.4.12。
后来突然想起来，这个项目虽然是springboot项目，但是并不是打成jar包通过内置jetty容器启动的。而是打成了war包，本地通过jetty-maven-plugin的jetty:run启动的。这里使用的jetty版本为9.4.9。
所以，我们需要按照jetty-maven-plugin的版本来选择jetty的源码。
5.2 「偶发问题」难以复现
考虑到篇幅原因与阅读体验，本文在排查过程中，没有展开说明一个非常困难的地方————本地如何稳定复现「偶发问题」异常请求。
真实排查过程中，本地稳定复现耗费了大量时间。如果不是本地可以稳定复现，后面的debug也无从谈起。
后面主要根据代码的近期变更情况，发现了一个异步请求的引入，将异步改为同步后，发现就不会再出现这个问题了。
所以才从异步请求出发，多次尝试后，进行了稳定复现。
所以本次排查的一个重要收获，就是对于一些故障的排查，可以考虑从近期的「各种变更」中去寻找线索。
往期热门笔记合集推荐：
原创：阿丸笔记（微信公众号：aone_note），欢迎 分享，转载请保留出处。
扫描下方二维码可以关注我哦～
如果有任何疑问或者建议，欢迎 写留言 或者 微信 和我联系哦～