Merge pull request #315 from caztg/dev

issues#314 fix DingNotifier
dromara · Jul 14, 2023 · 3060b91 · 3060b91
2 parents dbacde9 + e73cdb5
commit 3060b91
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/core/src/main/java/org/dromara/dynamictp/core/notifier/base/DingNotifier.java b/core/src/main/java/org/dromara/dynamictp/core/notifier/base/DingNotifier.java
@@ -43,6 +43,8 @@
 @Slf4j
 public class DingNotifier extends AbstractHttpNotifier {
 
+    private final static String ALL = "all";
+
     @Override
     public String platform() {
         return NotifyPlatformEnum.DING.name().toLowerCase();
@@ -58,7 +60,7 @@ protected String buildMsgBody(NotifyPlatform platform, String content) {
 
         List<String> mobiles = Lists.newArrayList(getNotifyReceivers(platform));
         at.setAtMobiles(mobiles);
-        if (CollectionUtils.isEmpty(mobiles)) {
+        if (mobiles.contains(ALL) || CollectionUtils.isEmpty(mobiles)) {
             at.setAtAll(true);
         }
 
@@ -76,7 +78,8 @@ protected String buildUrl(NotifyPlatform platform) {
 
     /**
      * Build target url.
-     * @param secret secret
+     *
+     * @param secret      secret
      * @param accessToken accessToken
      * @return url
      */