Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion weixin-java-cp/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@
<dependency>
<groupId>org.bouncycastle</groupId>
<artifactId>bcprov-jdk18on</artifactId>
<version>1.80</version>
<version>1.84</version>

@augmentcode augmentcode Bot Apr 18, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

weixin-java-cp/pom.xml:99 — This bumps bcprov-jdk18on to 1.84, but the root pom.xml still manages bcpkix-jdk18on at 1.80; mixing BouncyCastle versions can lead to dependency convergence issues or runtime NoSuchMethodError when multiple modules are used together. Consider aligning the BouncyCastle artifact versions across the build to avoid a split-brain dependency tree.

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

</dependency>
Comment on lines 96 to 99

Copilot AI Apr 19, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里把 bcprov-jdk18on 升到 1.84,但根 pom.xml 的 dependencyManagement 里仍固定了 bcpkix-jdk18on 为 1.80。由于其他模块(pay/channel/miniapp 等)依赖 bcpkix-jdk18on(并通过 dependencyManagement 解析版本),当用户同时引入 weixin-java-cp 与这些模块时,Maven 可能会产生 BouncyCastle 版本分裂(bcprov 1.84 vs bcpkix 1.80 及其传递依赖),存在运行期 NoSuchMethodError / ClassNotFound 的风险。建议将 BouncyCastle 版本在父 pom 统一管理(例如抽成 bouncycastle.version 属性,并同时升级 bcpkix-jdk18on 与 bcprov-jdk18on 到同一版本),然后在此处去掉显式 或对齐到同一版本来源。

Copilot uses AI. Check for mistakes.

<dependency>
Expand Down