使用微信支付公钥构建对应HTTP请求时，怎么关闭对应的HttpResponse

[pengwenclown]

错误描述

在使用APIv3 Java SDK时，SDK 未支持我需要的接口。这里推荐使用OkHttpClientAdapter实现类与使用 ApacheHttpClient 也就是ApacheHttpClientAdapter构建发送 HTTP 请求。我选择的是ApacheHttpClientAdapter创建对应的httpClient ，并且使用业务自定义的CloseableHttpClient(已创建连接池)参与构建。在请求微信时，相关请求都成功了。就是返回参数是要使用com.wechat.pay.java.core.http.HttpResponse接收，这里可以获取到对应的返回参数，但是没有提供对应的HttpResponse的关闭，在SDK的ApacheHttpClientAdapterTest测试类中也没有看到关闭HttpResponse

重现bug的步骤

这里是SDK推荐使用。
以下是提供的伪代码

就是针对这个httpResponse 数据是可以获取到的，那么对应的Response是否需要关闭，处理完响应后，是否关闭这个响应对象来释放底层的系统资源。
这里我是没有找到相关释放close()方法的
在SDK的ApacheHttpClientAdapterTest测试类中也是一样的只到获取数据，后续没有操作

预期行为

是否底层已经关闭？在使用com.wechat.pay.java.core.http.HttpResponse接收时，对应的响应已经关闭了？还是说有其他操作来关闭？

导致错误的代码片段

操作系统

WINDOS

Java 版本

JAVA8

wechatpay-java 版本

v0.2.16

其他信息

No response

[EmmetZC]

apacheHttpResponse 在创建时使用了 try-with-resources 表达式，会自动关闭对象，无需担心。

参见：

wechatpay-java/core/src/main/java/com/wechat/pay/java/core/http/apache/ApacheHttpClientAdapter.java

Lines 57 to 65 in c3ddd87

	public OriginalResponse innerExecute(HttpRequest wechatPayRequest) {
	try {
	CloseableHttpResponse apacheHttpResponse =
	apacheHttpClient.execute(buildApacheHttpRequest(wechatPayRequest));
	return assembleOriginalResponse(wechatPayRequest, apacheHttpResponse);
	} catch (IOException e) {
	throw new HttpException(wechatPayRequest, e);
	}
	}

[pengwenclown]

那就是说，我不用在去关注关闭Response这个动作了，只需要从com.wechat.pay.java.core.http.HttpResponse取数据就行，不用其他操作了。
我还有一个ApacheHttpClientAdapter发送post请求，请求参数中有中文，返回验签失败的问题。就跟#352的那个一样，麻烦看下，怎么处理。

[Nitsuya]

apacheHttpResponse 在创建时使用了 try-with-resources 表达式，会自动关闭对象，无需担心。

参见：

wechatpay-java/core/src/main/java/com/wechat/pay/java/core/http/apache/ApacheHttpClientAdapter.java

Lines 57 to 65 in c3ddd87

public OriginalResponse innerExecute(HttpRequest wechatPayRequest) {
try {
CloseableHttpResponse apacheHttpResponse =
apacheHttpClient.execute(buildApacheHttpRequest(wechatPayRequest));
return assembleOriginalResponse(wechatPayRequest, apacheHttpResponse);
} catch (IOException e) {
throw new HttpException(wechatPayRequest, e);
}
}

老哥, 麻烦看下TWR的语法糖, 你这样是关不掉的, 作为官方库, 犯这么低级的错误....
麻烦让高点的T级Code Review下~
还无需担心.. 给别人埋坑, 想着都头痛~

[EmmetZC]

抱歉，看错了，这个确实有问题。

OkHttpClient 写的没问题，ApacheClient 这个以为一样的实现没注意，不好意思。

[EmmetZC]

那就是说，我不用在去关注关闭Response这个动作了，只需要从com.wechat.pay.java.core.http.HttpResponse取数据就行，不用其他操作了。 我还有一个ApacheHttpClientAdapter发送post请求，请求参数中有中文，返回验签失败的问题。就跟#352的那个一样，麻烦看下，怎么处理。

请求中有中文的问题正在 #358 中修复。

这里提到的 Response 关闭的问题，正好提醒了我们 ApacheHttpResponse 的处理有误，正在 #360 中修复，修复完成后就如你所说，你从 HttpResponse 里头读取数据即可，不需要关闭。

[Nitsuya]

抱歉，看错了，这个确实有问题。

OkHttpClient 写的没问题，ApacheClient 这个以为一样的实现没注意，不好意思。

问题还有呢.

1. 默认的PoolingHttpClientConnectionManager最大连接池和单线路由默认很低, 而且又是static的, 建议改成lazy static, 不然用户自定义线程池, 你这里面又开一个.

2. 建议关闭Retry, 可能会有风险.

3. 建议关闭Keep-Alive功能, 约十年前吃过你龙哥服务器v2接口的亏.

--

另外建议把JSON部分抽出接口, GSON作为实现, 多数都是用的Jackson, 这也是快改完了懒得继续改下去的原因.

--

作为官方SDK, 请对用户负责, Apache的实现明显没有跑过全量API测试和压力测试, 不要刷KPI.

吐槽一下, 公钥迁移, 想着只用Core包平滑一下, 移花接木, 想偷懒两小时改完, 结果折磨两天, 开始没检索issues, 排查#352坑, 发现还有未关闭问题, 最后还是花了半天自己写了.