修复公钥模式下平台证书自动更新导致的初始化失败问题 by Copilot · Pull Request #3854 · binarywang/WxJava

Copilot · 2026-01-19T03:28:49Z

问题描述

用户配置公钥模式（publicKeyId + publicKeyPath）并设置 strictlyNeedWechatPaySerial=true 时，SDK 在初始化和每次请求时都会报错"无可用的平台证书"（HTTP 404），虽然请求实际成功但日志持续报错。

根本原因：AutoUpdateCertificatesVerifier 构造函数立即尝试下载平台证书，对于只使用公钥的新商户号（无平台证书），下载失败导致初始化异常。

主要修改

1. AutoUpdateCertificatesVerifier 容错增强

构造函数：证书下载失败时记录警告而非抛出异常，允许后续重试
verify(): 无证书时返回 false 而非异常
getValidCertificate(): 抛出明确的配置提示异常

2. WxPayService 实现类异常处理

getWechatPaySerial(): 捕获证书获取异常，返回空串允许请求继续（WxPayServiceApacheHttpImpl 和 WxPayServiceHttpComponentsImpl）

3. 单元测试

新增 AutoUpdateCertificatesVerifierPublicKeyModeTest 验证公钥模式健壮性

影响范围

向后兼容：不影响现有用户
公钥模式：修复初始化失败，建议使用 fullPublicKeyModel=true 以跳过证书下载
混合模式：支持平台证书+公钥共存场景

// 推荐配置（仅使用公钥时）
wxPayConfig.setPublicKeyId(publicKeyId);
wxPayConfig.setPublicKeyPath(publicKeyPath);
wxPayConfig.setFullPublicKeyModel(true);  // 避免尝试下载平台证书

Original prompt

This section details on the original issue you should resolve

<issue_title>【4.7.7.B版本】无可用的平台证书，请在商户平台-API安全申请使用微信支付公钥。</issue_title>
<issue_description>使用版本
<binarywang.weixin.java.version>4.7.7.B</binarywang.weixin.java.version>
配置公钥时设置所有请求携带标识参数设置为true了
订单状态查询时会有这个报错但是实际上请求都成功了
回调也有这个问题
Comments on the Issue (you are @copilot in this section)

Custom agent used: 全部用中文
需要用中文，包括PR标题和分析总结过程

Fixes 【4.7.7.B版本】无可用的平台证书，请在商户平台-API安全申请使用微信支付公钥。 #3669

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: binarywang <1343140+binarywang@users.noreply.github.com>

Copilot

Pull request overview

此 PR 修复了在公钥模式下（使用 publicKeyId + publicKeyPath）配置并启用 strictlyNeedWechatPaySerial=true 时，SDK 初始化失败并在每次请求时都报错"无可用的平台证书"的问题。根本原因是 AutoUpdateCertificatesVerifier 构造函数立即尝试下载平台证书，对于仅使用公钥的新商户号，下载失败导致初始化异常。

主要改动：

增强了 AutoUpdateCertificatesVerifier 的容错性，允许证书下载失败而不影响初始化
改进了 WxPayService 实现类中的异常处理，使证书获取失败时请求仍可继续
新增单元测试验证公钥模式下的健壮性

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
AutoUpdateCertificatesVerifier.java	构造函数增加异常容错，verify() 和 getValidCertificate() 方法增加空值检查和有意义的错误提示
WxPayServiceHttpComponentsImpl.java	getWechatPaySerial() 方法捕获异常并返回空字符串，允许请求继续
WxPayServiceApacheHttpImpl.java	getWechatPaySerial() 方法捕获异常并返回空字符串，允许请求继续
AutoUpdateCertificatesVerifierPublicKeyModeTest.java	新增单元测试，验证公钥模式下构造函数、verify() 和 getValidCertificate() 的行为

Copilot · 2026-01-19T03:49:59Z

...ava/com/github/binarywang/wxpay/v3/auth/AutoUpdateCertificatesVerifierPublicKeyModeTest.java

+    AutoUpdateCertificatesVerifier verifier = null;
+    try {
+      verifier = new AutoUpdateCertificatesVerifier(
+        credentials,
+        invalidApiV3Key.getBytes(StandardCharsets.UTF_8),
+        60,
+        payBaseUrl,
+        null
+      );
+      // 如果没有抛出异常，测试通过
+      assertNotNull(verifier);
+    } catch (Exception e) {
+      fail("构造函数不应该抛出异常，但抛出了: " + e.getMessage());
+    }


测试方法在 try-catch 块中调用 fail()，但在成功路径上只是断言 verifier 不为空。实际上，如果构造函数成功返回，测试已经通过了（因为没有抛出异常）。可以简化这个测试，移除 try-catch 块，直接创建对象并断言不为空即可。TestNG 会自动捕获并报告任何未预期的异常。

Suggested change

AutoUpdateCertificatesVerifier verifier = null;

try {

verifier = new AutoUpdateCertificatesVerifier(

credentials,

invalidApiV3Key.getBytes(StandardCharsets.UTF_8),

60,

payBaseUrl,

null

);

// 如果没有抛出异常，测试通过

assertNotNull(verifier);

} catch (Exception e) {

fail("构造函数不应该抛出异常，但抛出了: " + e.getMessage());

}

AutoUpdateCertificatesVerifier verifier = new AutoUpdateCertificatesVerifier(

credentials,

invalidApiV3Key.getBytes(StandardCharsets.UTF_8),

60,

payBaseUrl,

null

);

// 如果没有抛出异常，测试通过

assertNotNull(verifier);

Copilot · 2026-01-19T03:49:59Z

...va-pay/src/main/java/com/github/binarywang/wxpay/v3/auth/AutoUpdateCertificatesVerifier.java

    } catch (IOException | GeneralSecurityException e) {
-      throw new WxRuntimeException(e);
+      log.warn("Auto update cert failed during initialization, will retry later, exception = {}", e.getMessage());
+      // 设置instant为null，在第一次使用时会触发重新下载


注释说"在第一次使用时会触发重新下载"，但实际行为是：只要 instant 为 null，每次调用 verify() 或 getValidCertificate() 时都会尝试重新下载证书，直到成功为止。建议修改注释为："设置 instant 为 null，后续每次使用时都会尝试下载证书直到成功"，以更准确地反映实际行为。

Suggested change

// 设置instant为null，在第一次使用时会触发重新下载

// 设置 instant 为 null，后续每次使用时都会尝试下载证书直到成功

Copilot · 2026-01-19T03:50:00Z

...ava/com/github/binarywang/wxpay/v3/auth/AutoUpdateCertificatesVerifierPublicKeyModeTest.java

+  public void testVerifyShouldReturnFalseWhenNoCertificateAvailable() {
+    String invalidMchId = "invalid_mch_id";
+    String invalidApiV3Key = "invalid_api_v3_key_must_be_32_b";
+    String invalidCertSerialNo = "invalid_serial_no";
+    String payBaseUrl = "https://api.mch.weixin.qq.com";
+
+    WxPayCredentials credentials = new WxPayCredentials(
+      invalidMchId,
+      new PrivateKeySigner(invalidCertSerialNo, null)
+    );
+
+    AutoUpdateCertificatesVerifier verifier = new AutoUpdateCertificatesVerifier(
+      credentials,
+      invalidApiV3Key.getBytes(StandardCharsets.UTF_8),
+      60,
+      payBaseUrl,
+      null
+    );
+
+    // verify 方法应该返回 false，而不是抛出异常
+    boolean result = verifier.verify("test_serial", "test_message".getBytes(), "test_signature");
+    assertFalse(result, "当没有有效证书时，verify 应该返回 false");
+  }
+
+  /**
+   * 测试当没有有效证书时，getValidCertificate 方法应该抛出有意义的异常
+   */
+  @Test(expectedExceptions = me.chanjar.weixin.common.error.WxRuntimeException.class,
+    expectedExceptionsMessageRegExp = ".*No valid certificate available.*")
+  public void testGetValidCertificateShouldThrowMeaningfulException() {
+    String invalidMchId = "invalid_mch_id";
+    String invalidApiV3Key = "invalid_api_v3_key_must_be_32_b";
+    String invalidCertSerialNo = "invalid_serial_no";
+    String payBaseUrl = "https://api.mch.weixin.qq.com";
+
+    WxPayCredentials credentials = new WxPayCredentials(
+      invalidMchId,
+      new PrivateKeySigner(invalidCertSerialNo, null)
+    );
+
+    AutoUpdateCertificatesVerifier verifier = new AutoUpdateCertificatesVerifier(
+      credentials,
+      invalidApiV3Key.getBytes(StandardCharsets.UTF_8),
+      60,
+      payBaseUrl,
+      null
+    );


三个测试方法中都重复了相同的测试数据初始化代码（invalidMchId、invalidApiV3Key、invalidCertSerialNo、payBaseUrl、credentials）。建议考虑将这些通用的测试数据提取到 @BeforeMethod 或私有辅助方法中，以减少代码重复并提高可维护性。

Copilot · 2026-01-19T03:50:00Z

...ava/com/github/binarywang/wxpay/v3/auth/AutoUpdateCertificatesVerifierPublicKeyModeTest.java

@@ -0,0 +1,107 @@
+package com.github.binarywang.wxpay.v3.auth;
+
+import com.github.binarywang.wxpay.config.WxPayHttpProxy;


测试代码中导入了未使用的类 WxPayHttpProxy。虽然这不会影响测试的功能，但会增加不必要的依赖引用。建议移除这个未使用的导入。

Suggested change

import com.github.binarywang.wxpay.config.WxPayHttpProxy;

Copilot · 2026-01-19T03:50:00Z

...ava/com/github/binarywang/wxpay/v3/auth/AutoUpdateCertificatesVerifierPublicKeyModeTest.java

+    WxPayCredentials credentials = new WxPayCredentials(
+      invalidMchId,
+      new PrivateKeySigner(invalidCertSerialNo, null)


测试中直接将 null 传递给 PrivateKeySigner 构造函数的第二个参数（应该是 PrivateKey 类型）。这可能在实际调用时导致空指针异常，虽然这个测试的目的是验证构造函数不抛异常，但从测试设计角度来说，应该提供一个虽然无效但至少类型正确的对象，而不是 null。建议考虑生成一个模拟的 PrivateKey 对象或者在注释中说明为何这里使用 null 是安全的。

augmentcode · 2026-01-19T03:56:10Z

🤖 Augment PR Summary

Summary: 本 PR 修复微信支付 V3 在公钥模式下因平台证书自动更新失败而导致 SDK 初始化/请求阶段报“无可用的平台证书”的问题。

Changes:

AutoUpdateCertificatesVerifier：构造期证书更新失败时仅告警并允许后续重试；无证书时 verify() 返回 false，getValidCertificate() 抛出更明确的提示异常
WxPayServiceApacheHttpImpl/WxPayServiceHttpComponentsImpl：获取 Wechatpay-Serial 失败时不再中断请求，改为告警并返回空串
新增 AutoUpdateCertificatesVerifierPublicKeyModeTest 覆盖公钥模式下的健壮性

_{🤖 Was this summary useful? React with 👍 or 👎}

augmentcode

Review completed. 2 suggestions posted.

Comment augment review to trigger a new review at any time.

augmentcode · 2026-01-19T03:56:11Z

...va-pay/src/main/java/com/github/binarywang/wxpay/v3/auth/AutoUpdateCertificatesVerifier.java

      instant = Instant.now();
    } catch (IOException | GeneralSecurityException e) {
-      throw new WxRuntimeException(e);
+      log.warn("Auto update cert failed during initialization, will retry later, exception = {}", e.getMessage());


这里的容错只捕获了 IOException | GeneralSecurityException，但 autoUpdateCert() 在 HTTP 非 200（以及签名等运行时失败）时会直接抛 WxRuntimeException，仍可能导致初始化阶段异常退出（与 PR 目标的 404 场景不一致）。建议确认是否需要把该运行时异常也纳入容错范围。

_{🤖 Was this useful? React with 👍 or 👎}

augmentcode · 2026-01-19T03:56:11Z

...ava/com/github/binarywang/wxpay/v3/auth/AutoUpdateCertificatesVerifierPublicKeyModeTest.java

+    String invalidMchId = "invalid_mch_id";
+    String invalidApiV3Key = "invalid_api_v3_key_must_be_32_b";
+    String invalidCertSerialNo = "invalid_serial_no";
+    String payBaseUrl = "https://api.mch.weixin.qq.com";


该测试使用真实的 https://api.mch.weixin.qq.com 作为 payBaseUrl，会引入外部网络依赖（CI 环境下可能被禁网/超时/不稳定），从而导致单测偶发失败或变慢。建议确认项目单测规范是否允许这种外部依赖。

_{🤖 Was this useful? React with 👍 or 👎}

Initial plan

0b3ec25

Copilot AI assigned Copilot and binarywang Jan 19, 2026

Copilot started work on behalf of binarywang January 19, 2026 03:29 View session

修复公钥模式下自动更新证书报错问题

0b3c45a

Co-authored-by: binarywang <1343140+binarywang@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Fix platform certificate issue for version 4.7.7.B~~ 修复公钥模式下平台证书自动更新导致的初始化失败问题 Jan 19, 2026

Copilot AI requested a review from binarywang January 19, 2026 03:40

Copilot finished work on behalf of binarywang January 19, 2026 03:40

binarywang marked this pull request as ready for review January 19, 2026 03:44

Copilot AI review requested due to automatic review settings January 19, 2026 03:44

Copilot started reviewing on behalf of binarywang January 19, 2026 03:44 View session

Copilot AI reviewed Jan 19, 2026

View reviewed changes

augmentcode bot reviewed Jan 19, 2026

View reviewed changes

binarywang added this to the 4.8.2 milestone Jan 25, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

修复公钥模式下平台证书自动更新导致的初始化失败问题#3854

修复公钥模式下平台证书自动更新导致的初始化失败问题#3854
Copilot wants to merge 2 commits intodevelopfrom
copilot/fix-platform-certificate-issue

Copilot AI commented Jan 19, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jan 19, 2026

Uh oh!

Copilot AI Jan 19, 2026

Uh oh!

Copilot AI Jan 19, 2026

Uh oh!

Copilot AI Jan 19, 2026

Uh oh!

Copilot AI Jan 19, 2026

Uh oh!

augmentcode bot commented Jan 19, 2026

Uh oh!

augmentcode bot left a comment

Uh oh!

augmentcode bot Jan 19, 2026

Uh oh!

augmentcode bot Jan 19, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	// 设置instant为null，在第一次使用时会触发重新下载
	// 设置 instant 为 null，后续每次使用时都会尝试下载证书直到成功

		@@ -0,0 +1,107 @@
		package com.github.binarywang.wxpay.v3.auth;

		import com.github.binarywang.wxpay.config.WxPayHttpProxy;

Uh oh!

Conversation

Copilot AI commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

问题描述

主要修改

1. AutoUpdateCertificatesVerifier 容错增强

2. WxPayService 实现类异常处理

3. 单元测试

影响范围

Comments on the Issue (you are @copilot in this section)

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot commented Jan 19, 2026

Uh oh!

augmentcode bot left a comment

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

augmentcode bot Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Jan 19, 2026 •

edited

Loading