修复 wx-java-open-multi-spring-boot-starter 硬编码 HttpClient4 导致启动失败的问题#4060
Open
Copilot wants to merge 2 commits into
Open
修复 wx-java-open-multi-spring-boot-starter 硬编码 HttpClient4 导致启动失败的问题#4060Copilot wants to merge 2 commits into
Copilot wants to merge 2 commits into
Conversation
- 在 WxOpenMultiProperties.ConfigStorage 中新增 HttpClientType 枚举（APACHE_HTTP / HTTP_COMPONENTS）及 httpClientType 字段，默认值为 HTTP_COMPONENTS - 修改 AbstractWxOpenConfiguration.wxOpenService() 根据 httpClientType 选择对应的 WxOpenServiceApacheHttpClientImpl 或 WxOpenServiceHttpComponentsImpl - 修改 AbstractWxOpenConfiguration.configHttp() 仅在 httpClientType 为 APACHE_HTTP 时配置 DefaultApacheHttpClientBuilder，消除对 HttpClient4 的强制依赖 Closes #4059
Copilot
AI
changed the title
[WIP] Fix HttpClient version dependency issue in wx-java-open-multi-spring-boot-starter
修复 wx-java-open-multi-spring-boot-starter 硬编码 HttpClient4 导致启动失败的问题
Jul 1, 2026
There was a problem hiding this comment.
Pull request overview
该 PR 针对 wx-java-open-multi-spring-boot-starter 在未引入 HttpClient4 依赖时仍会硬编码初始化 DefaultApacheHttpClientBuilder、从而导致启动 NoClassDefFoundError 的问题，通过引入可配置的 HttpClient 类型选择，将 HttpClient4 相关初始化改为按需执行，确保默认（HttpClient5）路径不再触发类加载失败。
Changes:
- 在
WxOpenMultiProperties.ConfigStorage中新增httpClientType配置项及枚举（默认HTTP_COMPONENTS）。 AbstractWxOpenConfiguration.wxOpenService()根据httpClientType选择不同的WxOpenService实现（HttpClient4 vs HttpClient5）。configHttp()仅在选择 HttpClient4 时才配置DefaultApacheHttpClientBuilder，避免默认路径触发 HttpClient4 相关类加载。
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| spring-boot-starters/wx-java-open-multi-spring-boot-starter/src/main/java/com/binarywang/spring/starter/wxjava/open/properties/WxOpenMultiProperties.java | 增加 httpClientType 配置入口与枚举，默认使用 HttpClient5（HTTP_COMPONENTS）。 |
| spring-boot-starters/wx-java-open-multi-spring-boot-starter/src/main/java/com/binarywang/spring/starter/wxjava/open/configuration/services/AbstractWxOpenConfiguration.java | 根据配置选择 service 实现，并将 HttpClient4 超时配置改为仅在需要时执行，修复默认启动失败。 |
Comment on lines
76
to
+80
| public WxOpenService wxOpenService(WxOpenConfigStorage configStorage, WxOpenMultiProperties wxOpenMultiProperties) { | ||
| WxOpenService wxOpenService = new WxOpenServiceImpl(); | ||
| WxOpenMultiProperties.HttpClientType httpClientType = wxOpenMultiProperties.getConfigStorage().getHttpClientType(); | ||
| WxOpenService wxOpenService; | ||
| if (httpClientType == WxOpenMultiProperties.HttpClientType.APACHE_HTTP) { | ||
| wxOpenService = new WxOpenServiceApacheHttpClientImpl(); |
🤖 Augment PR SummarySummary: 修复 wx-java-open-multi-spring-boot-starter 在未引入 HttpClient4 依赖时启动报 NoClassDefFoundError 的问题。
🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
| if (httpClientType == WxOpenMultiProperties.HttpClientType.APACHE_HTTP) { | ||
| wxOpenService = new WxOpenServiceApacheHttpClientImpl(); | ||
| } else { | ||
| wxOpenService = new WxOpenServiceHttpComponentsImpl(); |
There was a problem hiding this comment.
AbstractWxOpenConfiguration.java:82 这里默认从返回 WxOpenServiceImpl 改为 WxOpenServiceHttpComponentsImpl，可能影响依赖具体实现类型（强转/instanceof/按类注入）的用户代码。即使当前 WxOpenServiceImpl 只是别名类，这种实现类型变化仍有兼容性风险。
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
AbstractWxOpenConfiguration.configHttp()无条件实例化DefaultApacheHttpClientBuilder（HttpClient 4.x），而默认服务实现WxOpenServiceImpl实际使用 HttpClient 5.x，导致未显式引入 HttpClient4 依赖的项目启动时抛出NoClassDefFoundError: org/apache/http/conn/HttpClientConnectionManager。变更内容
WxOpenMultiProperties：新增HttpClientType枚举（APACHE_HTTP/HTTP_COMPONENTS），并在ConfigStorage中添加httpClientType字段，默认值为HTTP_COMPONENTSAbstractWxOpenConfiguration：wxOpenService()根据httpClientType动态选择WxOpenServiceApacheHttpClientImpl或WxOpenServiceHttpComponentsImplconfigHttp()仅在httpClientType == APACHE_HTTP时才配置DefaultApacheHttpClientBuilder默认行为与原来一致（HttpClient 5.x），无需修改现有配置。如需切换至 HttpClient 4.x：