From aaa3e7e9fc57273dbacd10632a28138a864f8951 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 11 Apr 2026 09:21:49 +0000 Subject: [PATCH] =?UTF-8?q?=E4=BF=AE=E5=A4=8D=E8=AF=84=E5=AE=A1=E6=84=8F?= =?UTF-8?q?=E8=A7=81=EF=BC=9A=E4=BF=AE=E5=A4=8D=20agentId=20=E5=88=86?= =?UTF-8?q?=E7=BB=84=20null=20=E5=86=B2=E7=AA=81=20bug=EF=BC=8C=E4=BF=AE?= =?UTF-8?q?=E6=AD=A3=20Javadoc=20HTML=20=E7=BB=93=E6=9E=84=EF=BC=8C?= =?UTF-8?q?=E5=AE=8C=E5=96=84=20README=20=E9=85=8D=E7=BD=AE=E9=99=90?= =?UTF-8?q?=E5=88=B6=E8=AF=B4=E6=98=8E=E5=92=8C=E7=A4=BA=E4=BE=8B=E4=BB=A3?= =?UTF-8?q?=E7=A0=81?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Agent-Logs-Url: https://github.com/binarywang/WxJava/sessions/dcb81708-f62f-4f63-837b-0e62c1da2883 Co-authored-by: binarywang <1343140+binarywang@users.noreply.github.com> --- .../wx-java-cp-multi-solon-plugin/README.md | 9 +++++++-- .../services/AbstractWxCpConfiguration.java | 10 +++++----- .../wx-java-cp-multi-spring-boot-starter/README.md | 14 +++++++++++++- .../services/AbstractWxCpConfiguration.java | 5 +++-- 4 files changed, 28 insertions(+), 10 deletions(-) diff --git a/solon-plugins/wx-java-cp-multi-solon-plugin/README.md b/solon-plugins/wx-java-cp-multi-solon-plugin/README.md index 9904da302..8eb467f98 100644 --- a/solon-plugins/wx-java-cp-multi-solon-plugin/README.md +++ b/solon-plugins/wx-java-cp-multi-solon-plugin/README.md @@ -23,7 +23,7 @@ 如需同时使用多种权限范围,可在 `wx.cp.corps` 下配置多个条目,每个条目使用对应权限的 Secret,通过不同的 `tenantId` 区分后使用。 > **注意**: -> 当前 starter/插件实现会校验同一 `corp-id` 下的 `agent-id` **必须唯一**,并且 **只能有一个条目不填写 `agent-id`**。 +> 当前插件实现会校验同一 `corp-id` 下的 `agent-id` **必须唯一**,并且 **只能有一个条目不填写 `agent-id`**。 > 如果在同一 `corp-id` 下同时配置多个未填写 `agent-id` 的条目,会因 token/ticket 缓存 key 冲突而在启动时直接抛异常。 ## 快速开始 @@ -77,6 +77,7 @@ import com.binarywang.solon.wxjava.cp_multi.service.WxCpMultiServices; import me.chanjar.weixin.cp.api.WxCpDepartmentService; import me.chanjar.weixin.cp.api.WxCpService; import me.chanjar.weixin.cp.api.WxCpUserService; +import me.chanjar.weixin.cp.bean.WxCpDepart; import org.noear.solon.annotation.Component; import org.noear.solon.annotation.Inject; @@ -96,7 +97,11 @@ public class DemoService { // 通讯录同步 Secret 具有部门/成员增删改查等权限 WxCpService contactService = wxCpMultiServices.getWxCpService("contact"); WxCpDepartmentService departmentService = contactService.getDepartmentService(); - departmentService.update(department); + // 更新部门示例(WxCpDepart 包含 id、name、parentId 等字段) + WxCpDepart depart = new WxCpDepart(); + depart.setId(100L); + depart.setName("新部门名称"); + departmentService.update(depart); // todo ... } } diff --git a/solon-plugins/wx-java-cp-multi-solon-plugin/src/main/java/com/binarywang/solon/wxjava/cp_multi/configuration/services/AbstractWxCpConfiguration.java b/solon-plugins/wx-java-cp-multi-solon-plugin/src/main/java/com/binarywang/solon/wxjava/cp_multi/configuration/services/AbstractWxCpConfiguration.java index 3a5885037..25b4ab374 100644 --- a/solon-plugins/wx-java-cp-multi-solon-plugin/src/main/java/com/binarywang/solon/wxjava/cp_multi/configuration/services/AbstractWxCpConfiguration.java +++ b/solon-plugins/wx-java-cp-multi-solon-plugin/src/main/java/com/binarywang/solon/wxjava/cp_multi/configuration/services/AbstractWxCpConfiguration.java @@ -15,6 +15,7 @@ import org.apache.commons.lang3.StringUtils; import java.util.Collection; import java.util.List; import java.util.Map; +import java.util.Objects; import java.util.Set; import java.util.stream.Collectors; @@ -37,13 +38,12 @@ public abstract class AbstractWxCpConfiguration { /** * 校验同一个企业下,agentId 是否唯一,避免使用 redis 缓存 token、ticket 时错乱。 * - *
同一企业(corpId 相同)下可配置多个条目以使用不同的权限 Secret,例如: + *
同一企业(corpId 相同)下可配置多个条目以使用不同的权限 Secret,例如:
*但同一 corpId 下不允许出现重复的 agentId(包括多个 null)。
* * 查看 {@link me.chanjar.weixin.cp.config.impl.AbstractWxCpInRedisConfigImpl#setAgentId(Integer)} */ @@ -57,8 +57,8 @@ public abstract class AbstractWxCpConfiguration { String corpId = entry.getKey(); // 校验每个企业下,agentId 是否唯一 boolean multi = entry.getValue().stream() - // 通讯录没有 agentId,如果不判断是否为空,这里会报 NPE 异常 - .collect(Collectors.groupingBy(c -> c.getAgentId() == null ? 0 : c.getAgentId(), Collectors.counting())) + // 通讯录没有 agentId,使用字符串转换避免 null 与 agentId=0 冲突 + .collect(Collectors.groupingBy(c -> Objects.toString(c.getAgentId(), "null"), Collectors.counting())) .entrySet().stream().anyMatch(e -> e.getValue() > 1); if (multi) { throw new RuntimeException("请确保企业微信配置唯一性[" + corpId + "]"); diff --git a/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/README.md b/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/README.md index db54ad96c..0f0b74695 100644 --- a/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/README.md +++ b/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/README.md @@ -22,6 +22,13 @@ 如需同时使用多种权限范围,可在 `wx.cp.corps` 下配置多个条目,每个条目使用对应权限的 Secret,通过不同的 `tenantId` 区分后使用。 +> **配置限制说明**: +> - 当前 starter 实现会校验:同一 `corp-id` 下,`agent-id` **必须唯一** +> - 同一 `corp-id` 下,**只能有一个条目不填 `agent-id`** +> - 否则会因为 token/ticket 缓存 key 冲突而在启动时直接抛异常 +> +> 因此,像"通讯录同步 Secret""客户联系 Secret"这类通常不填写 `agent-id` 的配置,**不能**在同一个 `corp-id` 下同时配置多个 `agent-id` 均为空的条目;如确有多个条目,请确保其中最多只有一个未填写 `agent-id`。 + ## 快速开始 1. 引入依赖 @@ -74,6 +81,7 @@ import com.binarywang.spring.starter.wxjava.cp.service.WxCpMultiServices; import me.chanjar.weixin.cp.api.WxCpDepartmentService; import me.chanjar.weixin.cp.api.WxCpService; import me.chanjar.weixin.cp.api.WxCpUserService; +import me.chanjar.weixin.cp.bean.WxCpDepart; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.stereotype.Service; @@ -93,7 +101,11 @@ public class DemoService { // 通讯录同步 Secret 具有部门/成员增删改查等权限 WxCpService contactService = wxCpMultiServices.getWxCpService("contact"); WxCpDepartmentService departmentService = contactService.getDepartmentService(); - departmentService.update(department); + // 更新部门示例(WxCpDepart 包含 id、name、parentId 等字段) + WxCpDepart depart = new WxCpDepart(); + depart.setId(100L); + depart.setName("新部门名称"); + departmentService.update(depart); // todo ... } } diff --git a/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/src/main/java/com/binarywang/spring/starter/wxjava/cp/configuration/services/AbstractWxCpConfiguration.java b/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/src/main/java/com/binarywang/spring/starter/wxjava/cp/configuration/services/AbstractWxCpConfiguration.java index d94342315..a10bdf9be 100644 --- a/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/src/main/java/com/binarywang/spring/starter/wxjava/cp/configuration/services/AbstractWxCpConfiguration.java +++ b/spring-boot-starters/wx-java-cp-multi-spring-boot-starter/src/main/java/com/binarywang/spring/starter/wxjava/cp/configuration/services/AbstractWxCpConfiguration.java @@ -18,6 +18,7 @@ import org.apache.commons.lang3.StringUtils; import java.util.Collection; import java.util.List; import java.util.Map; +import java.util.Objects; import java.util.Set; import java.util.stream.Collectors; @@ -59,8 +60,8 @@ public abstract class AbstractWxCpConfiguration { String corpId = entry.getKey(); // 校验每个企业下,agentId 是否唯一 boolean multi = entry.getValue().stream() - // 通讯录没有 agentId,如果不判断是否为空,这里会报 NPE 异常 - .collect(Collectors.groupingBy(c -> c.getAgentId() == null ? 0 : c.getAgentId(), Collectors.counting())) + // 通讯录没有 agentId,使用字符串转换避免 null 与 agentId=0 冲突 + .collect(Collectors.groupingBy(c -> Objects.toString(c.getAgentId(), "null"), Collectors.counting())) .entrySet().stream().anyMatch(e -> e.getValue() > 1); if (multi) { throw new RuntimeException("请确保企业微信配置唯一性[" + corpId + "]");