调用微信接口异常重试逻辑问题

[foreveryang321]

简要描述

调用微信接口响应超时异常，会导致重复调用接口，从而导致重复发送模板消息、客服消息等

模块版本情况

• WxJava 模块名: weixin-java-mp
• WxJava 版本号: 4.0.8.B

详细描述

重试逻辑有问题，问题代码

errorCode == -1 时会重试

https://github.com/Wechat-Group/WxJava/blob/15bd95c3cbbfcd9a5eee93e209207cd245f4e81a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/BaseWxMpServiceImpl.java#L336-L353

请求超时（连接超时、响应超时），WxError.getErrorCode() 默认是 -1

https://github.com/Wechat-Group/WxJava/blob/15bd95c3cbbfcd9a5eee93e209207cd245f4e81a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/BaseWxMpServiceImpl.java#L401-L404

https://github.com/Wechat-Group/WxJava/blob/15bd95c3cbbfcd9a5eee93e209207cd245f4e81a/weixin-java-common/src/main/java/me/chanjar/weixin/common/error/WxErrorException.java#L25-L28

以下异常定义

// errorCode == -1 
throw new WxErrorException("请先设置WebhookKey");

基于以上代码，可以发现

• IOException异常会导致重试（猜测这个并不需要重试，会导致响应超时异常也重复调用接口，影响：调用发送模板、发送群推等接口会导致用户收到多条）
• throw new WxErrorException("异常信息")定义异常会导致重试
• 微信官方接口返回{"errcode":-1,"errmsg":"system error rid: 60891646-03eed615-7f6889ef"}会重试（猜测实现重试是这个意图）

日志

此处复现的是连接超时导致重试（响应超时也会存在重试问题）

2021-04-29 11:59:20.825 ERROR 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 
【请求地址】: https://api.weixin.qq.com/cgi-bin/material/get_materialcount?access_token=token脱敏
【请求参数】：null
【异常信息】：java.net.SocketException: Connection reset
2021-04-29 11:59:20.827  WARN 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 微信系统繁忙，1000 ms 后重试(第1次)
2021-04-29 11:59:26.836 ERROR 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 
【请求地址】: https://api.weixin.qq.com/cgi-bin/material/get_materialcount?access_token=token脱敏
【请求参数】：null
【异常信息】：java.net.SocketException: Connection reset
2021-04-29 11:59:26.836  WARN 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 微信系统繁忙，2000 ms 后重试(第2次)
2021-04-29 11:59:33.840 ERROR 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 
【请求地址】: https://api.weixin.qq.com/cgi-bin/material/get_materialcount?access_token=token脱敏
【请求参数】：null
【异常信息】：java.net.SocketException: Connection reset
2021-04-29 11:59:33.840  WARN 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 微信系统繁忙，4000 ms 后重试(第3次)
2021-04-29 11:59:42.848 ERROR 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 
【请求地址】: https://api.weixin.qq.com/cgi-bin/material/get_materialcount?access_token=token脱敏
【请求参数】：null
【异常信息】：java.net.SocketException: Connection reset
2021-04-29 11:59:42.848  WARN 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 微信系统繁忙，8000 ms 后重试(第4次)
2021-04-29 11:59:55.866 ERROR 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 
【请求地址】: https://api.weixin.qq.com/cgi-bin/material/get_materialcount?access_token=token脱敏
【请求参数】：null
【异常信息】：java.net.SocketException: Connection reset
2021-04-29 12:00:26.431  WARN 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 微信系统繁忙，16000 ms 后重试(第5次)
2021-04-29 12:01:10.443 ERROR 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 
【请求地址】: https://api.weixin.qq.com/cgi-bin/material/get_materialcount?access_token=token脱敏
【请求参数】：null
【异常信息】：java.net.SocketException: Connection reset
2021-04-29 12:01:10.445  WARN 6517 --- [sync-scheduler1] m.c.w.mp.api.impl.BaseWxMpServiceImpl    : 重试达到最大次数【5】

修复建议

https://github.com/Wechat-Group/WxJava/blob/15bd95c3cbbfcd9a5eee93e209207cd245f4e81a/weixin-java-common/src/main/java/me/chanjar/weixin/common/error/WxErrorException.java#L11-L13
https://github.com/Wechat-Group/WxJava/blob/15bd95c3cbbfcd9a5eee93e209207cd245f4e81a/weixin-java-common/src/main/java/me/chanjar/weixin/common/error/WxErrorException.java#L25-L28
初始化默认值-1，改成其他（比如-99、-10000等），保证不和微信接口官方返回错误码重复即可

[binarywang]

欢迎自行修复，如果测试没问题，直接提交PR即可