修复未开启行政区划和分组推送的上级平台收到行政区划和目录的BUG

src/main/java/com/genersoft/iot/vmp/gb28181/bean/AlarmChannelMessage.java

@@ -16,12 +16,12 @@ public class AlarmChannelMessage {
     /**
      * 报警编号
      */
-    private int alarmSn;
+    private Integer alarmSn;
 
     /**
      * 告警类型
      */
-    private int alarmType;
+    private Integer alarmType;
 
     /**
      * 报警描述

src/main/java/com/genersoft/iot/vmp/gb28181/bean/CatalogData.java

@@ -19,8 +19,8 @@ public class CatalogData {
     private Instant time;
     private Device device;
     private String errorMsg;
+    private boolean complete;
     private Set<String> redisKeysForChannel = new HashSet<>();
-    private Set<String> errorChannel = new HashSet<>();
     private Set<String> redisKeysForRegion = new HashSet<>();
     private Set<String> redisKeysForGroup = new HashSet<>();
 

src/main/java/com/genersoft/iot/vmp/gb28181/bean/GbSteamIdentification.java

@@ -1,44 +0,0 @@
-package com.genersoft.iot.vmp.gb28181.bean;
-
-/**
- * 码流索引标识
- */
-public enum GbSteamIdentification {
-    /**
-     * 主码流 stream:0
-     * 子码流 stream:1s
-     */
-    streamMain("stream", new String[]{"0","1"}),
-    /**
-     * 国标28181-2022定义的方式
-     * 主码流 streamnumber:0
-     * 子码流 streamnumber:1
-     */
-    streamnumber("streamnumber", new String[]{"0","1"}),
-    /**
-     * 主码流 streamprofile:0
-     * 子码流 streamprofile:1
-     */
-    streamprofile("streamprofile", new String[]{"0","1"}),
-    /**
-     * 适用的品牌： TP-LINK
-     */
-    streamMode("streamMode", new String[]{"main","sub"}),
-    ;
-
-    GbSteamIdentification(String value, String[] indexArray) {
-        this.value = value;
-        this.indexArray = indexArray;
-    }
-
-    private String value;
-    private String[] indexArray;
-
-    public String getValue() {
-        return value;
-    }
-
-    public String[] getIndexArray() {
-        return indexArray;
-    }
-}

src/main/java/com/genersoft/iot/vmp/gb28181/bean/NotifyCatalogChannel.java

@@ -7,16 +7,19 @@ public class NotifyCatalogChannel {
 
     private DeviceChannel channel;
 
+    private String deviceId;
+
 
     public enum Type {
         ADD, DELETE, UPDATE, STATUS_CHANGED
     }
 
 
-    public static NotifyCatalogChannel getInstance(Type type, DeviceChannel channel) {
+    public static NotifyCatalogChannel getInstance(Type type, DeviceChannel channel, String deviceId) {
         NotifyCatalogChannel notifyCatalogChannel = new NotifyCatalogChannel();
         notifyCatalogChannel.setType(type);
         notifyCatalogChannel.setChannel(channel);
+        notifyCatalogChannel.setDeviceId(deviceId);
         return notifyCatalogChannel;
     }
 
@@ -35,4 +38,12 @@ public class NotifyCatalogChannel {
     public void setChannel(DeviceChannel channel) {
         this.channel = channel;
     }
+
+    public String getDeviceId() {
+        return deviceId;
+    }
+
+    public void setDeviceId(String deviceId) {
+        this.deviceId = deviceId;
+    }
 }

src/main/java/com/genersoft/iot/vmp/gb28181/bean/PlatformCatalog.java

@@ -1,116 +0,0 @@
-package com.genersoft.iot.vmp.gb28181.bean;
-
-import io.swagger.v3.oas.annotations.media.Schema;
-
-/**
- * 国标级联-目录
- * @author lin
- */
-@Schema(description = "目录信息")
-public class PlatformCatalog {
-    @Schema(description = "ID")
-    private String id;
-
-    @Schema(description = "名称")
-    private String name;
-
-    @Schema(description = "平台ID")
-    private String platformId;
-
-    @Schema(description = "父级目录ID")
-    private String parentId;
-
-    @Schema(description = "行政区划")
-    private String civilCode;
-
-    @Schema(description = "目录分组")
-    private String businessGroupId;
-
-    /**
-     * 子节点数
-     */
-    @Schema(description = "子节点数")
-    private int childrenCount;
-
-    /**
-     * 0 目录, 1 国标通道, 2 直播流
-     */
-    @Schema(description = "类型：0 目录, 1 国标通道, 2 直播流")
-    private int type;
-
-    public String getId() {
-        return id;
-    }
-
-    public void setId(String id) {
-        this.id = id;
-    }
-
-    public String getName() {
-        return name;
-    }
-
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    public String getPlatformId() {
-        return platformId;
-    }
-
-    public void setPlatformId(String platformId) {
-        this.platformId = platformId;
-    }
-
-    public String getParentId() {
-        return parentId;
-    }
-
-    public void setParentId(String parentId) {
-        this.parentId = parentId;
-    }
-
-    public int getChildrenCount() {
-        return childrenCount;
-    }
-
-    public void setChildrenCount(int childrenCount) {
-        this.childrenCount = childrenCount;
-    }
-
-    public int getType() {
-        return type;
-    }
-
-    public void setType(int type) {
-        this.type = type;
-    }
-
-    public void setTypeForCatalog() {
-        this.type = 0;
-    }
-
-    public void setTypeForGb() {
-        this.type = 1;
-    }
-
-    public void setTypeForStream() {
-        this.type = 2;
-    }
-
-    public String getCivilCode() {
-        return civilCode;
-    }
-
-    public void setCivilCode(String civilCode) {
-        this.civilCode = civilCode;
-    }
-
-    public String getBusinessGroupId() {
-        return businessGroupId;
-    }
-
-    public void setBusinessGroupId(String businessGroupId) {
-        this.businessGroupId = businessGroupId;
-    }
-}

src/main/java/com/genersoft/iot/vmp/gb28181/bean/PlatformCatch.java

@@ -1,24 +0,0 @@
-package com.genersoft.iot.vmp.gb28181.bean;
-
-import lombok.Data;
-
-@Data
-public class PlatformCatch {
-
-    private String id;
-
-    /**
-     * 心跳未回复次数
-     */
-    private int keepAliveReply;
-
-    // 注册未回复次数
-    private int registerAliveReply;
-
-    private String callId;
-
-    private Platform platform;
-
-    private SipTransactionInfo sipTransactionInfo;
-
-}

src/main/java/com/genersoft/iot/vmp/gb28181/service/impl/DeviceChannelServiceImpl.java

@@ -404,12 +404,7 @@ public class DeviceChannelServiceImpl implements IDeviceChannelService {
             allChannelMap.remove(deviceChannel.getDataDeviceId() + deviceChannel.getDeviceId());
             upsertChannels.add(deviceChannel);
             if (!ObjectUtils.isEmpty(deviceChannel.getParentId())) {
-                if (subContMap.get(deviceChannel.getParentId()) == null) {
+                subContMap.merge(deviceChannel.getParentId(), 1, Integer::sum);
-                    subContMap.put(deviceChannel.getParentId(), 1);
-                } else {
-                    Integer count = subContMap.get(deviceChannel.getParentId());
-                    subContMap.put(deviceChannel.getParentId(), count++);
-                }
             }
         }
         deleteChannels.addAll(allChannelMap.values());

src/main/java/com/genersoft/iot/vmp/gb28181/service/impl/DeviceServiceImpl.java

@@ -750,13 +750,16 @@ public class DeviceServiceImpl implements IDeviceService {
 
     @Override
     public void sync(Device device) {
-        if (catalogResponseMessageHandler.isSyncRunning(device.getDeviceId())) {
+        int sn;
-            SyncStatus syncStatus = catalogResponseMessageHandler.getChannelSyncProgress(device.getDeviceId());
+        synchronized (device.getDeviceId().intern()) {
-            log.info("[同步通道] 同步已存在, 设备: {}, 同步信息: {}", device.getDeviceId(), JSON.toJSON(syncStatus));
+            if (catalogResponseMessageHandler.isSyncRunning(device.getDeviceId())) {
-            return;
+                SyncStatus syncStatus = catalogResponseMessageHandler.getChannelSyncProgress(device.getDeviceId());
+                log.info("[同步通道] 同步已存在, 设备: {}, 同步信息: {}", device.getDeviceId(), JSON.toJSON(syncStatus));
+                return;
+            }
+            sn = (int)((Math.random()*9+1)*100000);
+            catalogResponseMessageHandler.setChannelSyncReady(device, sn);
         }
-        int sn = (int)((Math.random()*9+1)*100000);
-        catalogResponseMessageHandler.setChannelSyncReady(device, sn);
         try {
             sipCommander.catalogQuery(device, sn, event -> {
                 String errorMsg = String.format("同步通道失败，错误码： %s, %s", event.statusCode, event.msg);

src/main/java/com/genersoft/iot/vmp/gb28181/service/impl/PlatformChannelServiceImpl.java

@@ -393,25 +393,30 @@ public class PlatformChannelServiceImpl implements IPlatformChannelService {
         int result = platformChannelMapper.addChannels(platformId, channelList);
         if (result > 0) {
             // 查询通道相关的行政区划信息是否共享，如果没共享就添加
-            Set<Region> regionListNotShare =  getRegionNotShareByChannelList(channelList, platformId);
+            // 判断平台是否客气了推送行政区划
-            if (!regionListNotShare.isEmpty()) {
+            if (platform.getCatalogWithRegion() != 0) {
-                int addGroupResult = platformChannelMapper.addPlatformRegion(new ArrayList<>(regionListNotShare), platformId);
+                Set<Region> regionListNotShare =  getRegionNotShareByChannelList(channelList, platformId);
-                if (addGroupResult > 0) {
+                if (!regionListNotShare.isEmpty()) {
-                    for (Region region : regionListNotShare) {
+                    int addGroupResult = platformChannelMapper.addPlatformRegion(new ArrayList<>(regionListNotShare), platformId);
-                        // 分组信息排序时需要将顶层排在最后
+                    if (addGroupResult > 0) {
-                        channelList.add(0, CommonGBChannel.build(region));
+                        for (Region region : regionListNotShare) {
+                            // 分组信息排序时需要将顶层排在最后
+                            channelList.addFirst(CommonGBChannel.build(region));
+                        }
                     }
                 }
             }
 
-            // 查询通道相关的分组信息是否共享，如果没共享就添加
+            if (platform.getCatalogWithGroup() != 0) {
-            Set<Group> groupListNotShare =  getGroupNotShareByChannelList(channelList, platformId);
+                // 查询通道相关的分组信息是否共享，如果没共享就添加
-            if (!groupListNotShare.isEmpty()) {
+                Set<Group> groupListNotShare =  getGroupNotShareByChannelList(channelList, platformId);
-                int addGroupResult = platformChannelMapper.addPlatformGroup(new ArrayList<>(groupListNotShare), platformId);
+                if (!groupListNotShare.isEmpty()) {
-                if (addGroupResult > 0) {
+                    int addGroupResult = platformChannelMapper.addPlatformGroup(new ArrayList<>(groupListNotShare), platformId);
-                    for (Group group : groupListNotShare) {
+                    if (addGroupResult > 0) {
-                        // 分组信息排序时需要将顶层排在最后
+                        for (Group group : groupListNotShare) {
-                        channelList.add(0, CommonGBChannel.build(group));
+                            // 分组信息排序时需要将顶层排在最后
+                            channelList.addFirst(CommonGBChannel.build(group));
+                        }
                     }
                 }
             }

src/main/java/com/genersoft/iot/vmp/gb28181/session/CatalogDataManager.java

@@ -16,6 +16,7 @@ import java.time.Instant;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
+import java.util.Iterator;
 import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.TimeUnit;
@@ -56,29 +57,17 @@ public class CatalogDataManager{
     }
 
     public void addReady(Device device, int sn ) {
-        CatalogData catalogData = dataMap.get(buildMapKey(device.getDeviceId(),sn));
+        // 清除该设备的所有旧条目
-        if (catalogData != null) {
+        Iterator<Map.Entry<String, CatalogData>> it = dataMap.entrySet().iterator();
-            Set<String> redisKeysForChannel = catalogData.getRedisKeysForChannel();
+        while (it.hasNext()) {
-            if (redisKeysForChannel != null && !redisKeysForChannel.isEmpty()) {
+            Map.Entry<String, CatalogData> entry = it.next();
-                for (String deleteKey : redisKeysForChannel) {
+            CatalogData old = entry.getValue();
-                    redisTemplate.opsForHash().delete(key, deleteKey);
+            if (old != null && device.getDeviceId().equals(old.getDevice().getDeviceId())) {
-                }
+                deleteRedisKeys(old);
+                it.remove();
             }
-            Set<String> redisKeysForRegion = catalogData.getRedisKeysForRegion();
-            if (redisKeysForRegion != null && !redisKeysForRegion.isEmpty()) {
-                for (String deleteKey : redisKeysForRegion) {
-                    redisTemplate.opsForHash().delete(key, deleteKey);
-                }
-            }
-            Set<String> redisKeysForGroup = catalogData.getRedisKeysForGroup();
-            if (redisKeysForGroup != null && !redisKeysForGroup.isEmpty()) {
-                for (String deleteKey : redisKeysForGroup) {
-                    redisTemplate.opsForHash().delete(key, deleteKey);
-                }
-            }
-            dataMap.remove(buildMapKey(device.getDeviceId(),sn));
         }
-        catalogData = new CatalogData();
+        CatalogData catalogData = new CatalogData();
         catalogData.setDevice(device);
         catalogData.setSn(sn);
         catalogData.setStatus(CatalogData.CatalogDataStatus.ready);
@@ -86,6 +75,27 @@ public class CatalogDataManager{
         dataMap.put(buildMapKey(device.getDeviceId(),sn), catalogData);
     }
 
+    private void deleteRedisKeys(CatalogData catalogData) {
+        Set<String> redisKeysForChannel = catalogData.getRedisKeysForChannel();
+        if (redisKeysForChannel != null && !redisKeysForChannel.isEmpty()) {
+            for (String deleteKey : redisKeysForChannel) {
+                redisTemplate.opsForHash().delete(key, deleteKey);
+            }
+        }
+        Set<String> redisKeysForRegion = catalogData.getRedisKeysForRegion();
+        if (redisKeysForRegion != null && !redisKeysForRegion.isEmpty()) {
+            for (String deleteKey : redisKeysForRegion) {
+                redisTemplate.opsForHash().delete(key, deleteKey);
+            }
+        }
+        Set<String> redisKeysForGroup = catalogData.getRedisKeysForGroup();
+        if (redisKeysForGroup != null && !redisKeysForGroup.isEmpty()) {
+            for (String deleteKey : redisKeysForGroup) {
+                redisTemplate.opsForHash().delete(key, deleteKey);
+            }
+        }
+    }
+
     public void put(String deviceId, int sn, int total, Device device, List<DeviceChannel> deviceChannelList,
                     List<Region> regionList, List<Group> groupList) {
         CatalogData catalogData = dataMap.get(buildMapKey(device.getDeviceId(),sn));
@@ -188,10 +198,6 @@ public class CatalogDataManager{
                 }else {
                     syncStatus.setSyncIng(true);
                 }
-                if (catalogData.getErrorMsg() != null) {
-                    // 失败的同步信息,返回一次后直接移除
-                    dataMap.remove(key);
-                }
                 return syncStatus;
             }
         }
@@ -249,20 +255,19 @@ public class CatalogDataManager{
                     catalogData.setStatus(CatalogData.CatalogDataStatus.end);
                 }
             }else if (catalogData.getStatus().equals(CatalogData.CatalogDataStatus.runIng)) {
-                if ( catalogData.getTime().isBefore(instantBefore5S)) {
+                boolean complete = catalogData.isComplete();
+                boolean timeout = catalogData.getTime().isBefore(instantBefore5S);
+                if (complete || timeout) {
                     String deviceId = catalogData.getDevice().getDeviceId();
                     ReentrantLock lock = getDeviceWriteLock(deviceId);
                     if (!lock.tryLock()) {
-                        // saveData() 正在执行，跳过本次，等下一个5s周期
                         continue;
                     }
                     try {
                         int sn = catalogData.getSn();
                         List<DeviceChannel> deviceChannelList = getDeviceChannelList(deviceId, sn);
-                        if (catalogData.getTotal() == deviceChannelList.size()) {
+                        if (!deviceChannelList.isEmpty()) {
                             deviceChannelService.resetChannels(catalogData.getDevice().getId(), deviceChannelList);
-                        } else {
-                            deviceChannelService.updateChannels(catalogData.getDevice(), deviceChannelList);
                         }
                         List<Region> regionList = getRegionList(deviceId, sn);
                         if ( regionList!= null && !regionList.isEmpty()) {
@@ -272,10 +277,10 @@ public class CatalogDataManager{
                         if (groupList != null && !groupList.isEmpty()) {
                             groupService.batchAdd(groupList);
                         }
-                        String errorMsg = "更新成功，共" + catalogData.getTotal() + "条，已更新" + deviceChannelList.size() + "条";
+                        catalogData.setErrorMsg(null);
-                        catalogData.setErrorMsg(errorMsg);
                     } catch (Exception e) {
                         log.error("[国标通道同步] 入库失败： ", e);
+                        catalogData.setErrorMsg("入库失败: " + e.getMessage());
                     } finally {
                         lock.unlock();
                     }
@@ -289,30 +294,21 @@ public class CatalogDataManager{
                     if (deviceWriteLocks.containsKey(deviceId)) {
                         deviceWriteLocks.remove(deviceId);
                     }
-                    Set<String> redisKeysForChannel = catalogData.getRedisKeysForChannel();
+                    deleteRedisKeys(catalogData);
-                    if (redisKeysForChannel != null && !redisKeysForChannel.isEmpty()) {
-                        for (String deleteKey : redisKeysForChannel) {
-                            redisTemplate.opsForHash().delete(key, deleteKey);
-                        }
-                    }
-                    Set<String> redisKeysForRegion = catalogData.getRedisKeysForRegion();
-                    if (redisKeysForRegion != null && !redisKeysForRegion.isEmpty()) {
-                        for (String deleteKey : redisKeysForRegion) {
-                            redisTemplate.opsForHash().delete(key, deleteKey);
-                        }
-                    }
-                    Set<String> redisKeysForGroup = catalogData.getRedisKeysForGroup();
-                    if (redisKeysForGroup != null && !redisKeysForGroup.isEmpty()) {
-                        for (String deleteKey : redisKeysForGroup) {
-                            redisTemplate.opsForHash().delete(key, deleteKey);
-                        }
-                    }
                 }
             }
         }
     }
 
 
+    public void setComplete(String deviceId, int sn) {
+        CatalogData catalogData = dataMap.get(buildMapKey(deviceId,sn));
+        if (catalogData == null) {
+            return;
+        }
+        catalogData.setComplete(true);
+    }
+
     public void setChannelSyncEnd(String deviceId, int sn, String errorMsg) {
         CatalogData catalogData = dataMap.get(buildMapKey(deviceId,sn));
         if (catalogData == null) {
@@ -328,7 +324,7 @@ public class CatalogDataManager{
         if (catalogData == null) {
             return 0;
         }
-        return catalogData.getRedisKeysForChannel().size() + catalogData.getErrorChannel().size();
+        return catalogData.getRedisKeysForChannel().size();
     }
 
     public int sumNum(String deviceId, int sn) {

src/main/java/com/genersoft/iot/vmp/gb28181/transmit/event/request/impl/NotifyRequestForCatalogProcessor.java

@@ -7,6 +7,7 @@ import com.genersoft.iot.vmp.gb28181.event.channel.ChannelEvent;
 import com.genersoft.iot.vmp.gb28181.event.subscribe.catalog.CatalogEvent;
 import com.genersoft.iot.vmp.gb28181.service.IDeviceChannelService;
 import com.genersoft.iot.vmp.gb28181.service.IGbChannelService;
+import com.genersoft.iot.vmp.gb28181.session.CatalogDataManager;
 import com.genersoft.iot.vmp.gb28181.transmit.event.request.SIPRequestProcessorParent;
 import com.genersoft.iot.vmp.gb28181.utils.SipUtils;
 import com.genersoft.iot.vmp.storager.IRedisCatchStorage;
@@ -26,9 +27,12 @@ import javax.sip.header.FromHeader;
 import java.lang.reflect.InvocationTargetException;
 import java.nio.charset.Charset;
 import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Map;
 import java.util.concurrent.ConcurrentLinkedQueue;
+import java.util.concurrent.locks.ReentrantLock;
 
 /**
  * SIP命令类型： NOTIFY请求中的目录请求处理
@@ -56,6 +60,9 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 	@Autowired
 	private IGbChannelService channelService;
 
+	@Autowired
+	private CatalogDataManager catalogDataManager;
+
 
 	public void process(RequestEvent evt) {
 		if (taskQueue.size() >= userSetting.getMaxNotifyCountQueue()) {
@@ -149,7 +156,7 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 								// 上线
 								log.info("[收到通道上线通知] 来自设备: {}, 通道 {}", device.getDeviceId(), catalogChannelEvent.getChannel().getDeviceId());
 								channel.setStatus("ON");
-								channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel));
+								channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel, device.getDeviceId()));
 
 								if (userSetting.getDeviceStatusNotify()) {
 									// 发送redis消息
@@ -163,7 +170,7 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 									log.info("[收到通道离线通知] 但是平台已配置拒绝此消息，来自设备: {}, 通道 {}", device.getDeviceId(), catalogChannelEvent.getChannel().getDeviceId());
 								} else {
 									channel.setStatus("OFF");
-									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel));
+									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel, device.getDeviceId()));
 									if (userSetting.getDeviceStatusNotify()) {
 										// 发送redis消息
 										redisCatchStorage.sendDeviceOrChannelStatus(device.getDeviceId(), catalogChannelEvent.getChannel().getDeviceId(), false);
@@ -177,7 +184,7 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 									log.info("[收到通道视频丢失通知] 但是平台已配置拒绝此消息，来自设备: {}, 通道 {}", device.getDeviceId(), catalogChannelEvent.getChannel().getDeviceId());
 								} else {
 									channel.setStatus("OFF");
-									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel));
+									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel, device.getDeviceId()));
 
 									if (userSetting.getDeviceStatusNotify()) {
 										// 发送redis消息
@@ -192,7 +199,7 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 									log.info("[收到通道视频故障通知] 但是平台已配置拒绝此消息，来自设备: {}, 通道 {}", device.getDeviceId(), catalogChannelEvent.getChannel().getDeviceId());
 								} else {
 									channel.setStatus("OFF");
-									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel));
+									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.STATUS_CHANGED, channel, device.getDeviceId()));
 
 									if (userSetting.getDeviceStatusNotify()) {
 										// 发送redis消息
@@ -210,12 +217,12 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 									channel.setId(deviceChannel.getId());
 									channel.setHasAudio(deviceChannel.isHasAudio());
 									channel.setUpdateTime(DateUtil.getNow());
-									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.UPDATE, channel));
+									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.UPDATE, channel, device.getDeviceId()));
 
 								} else {
 									catalogChannelEvent.getChannel().setUpdateTime(DateUtil.getNow());
 									catalogChannelEvent.getChannel().setCreateTime(DateUtil.getNow());
-									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.ADD, channel));
+									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.ADD, channel, device.getDeviceId()));
 
 									if (userSetting.getDeviceStatusNotify()) {
 										// 发送redis消息
@@ -227,7 +234,7 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 							case CatalogEvent.DEL:
 								// 删除
 								log.info("[收到删除通道通知] 来自设备: {}, 通道 {}", device.getDeviceId(), catalogChannelEvent.getChannel().getDeviceId());
-								channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.DELETE, channel));
+								channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.DELETE, channel, device.getDeviceId()));
 
 								if (userSetting.getDeviceStatusNotify()) {
 									// 发送redis消息
@@ -244,12 +251,12 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 									channel.setHasAudio(deviceChannelForUpdate.isHasAudio());
 									channel.setUpdateTime(DateUtil.getNow());
 									channel.setUpdateTime(DateUtil.getNow());
-									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.UPDATE, channel));
+									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.UPDATE, channel, device.getDeviceId()));
 
 								} else {
 									catalogChannelEvent.getChannel().setCreateTime(DateUtil.getNow());
 									catalogChannelEvent.getChannel().setUpdateTime(DateUtil.getNow());
-									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.ADD, channel));
+									channelList.add(NotifyCatalogChannel.getInstance(NotifyCatalogChannel.Type.ADD, channel, device.getDeviceId()));
 
 									if (userSetting.getDeviceStatusNotify()) {
 										// 发送redis消息
@@ -281,38 +288,51 @@ public class NotifyRequestForCatalogProcessor extends SIPRequestProcessorParent
 			channelListForSave.add(channelList.poll());
 		}
 
-		for (NotifyCatalogChannel notifyCatalogChannel : channelListForSave) {
+		Map<String, List<NotifyCatalogChannel>> grouped = new HashMap<>();
+		for (NotifyCatalogChannel item : channelListForSave) {
+			grouped.computeIfAbsent(item.getDeviceId(), k -> new ArrayList<>()).add(item);
+		}
+
+		for (Map.Entry<String, List<NotifyCatalogChannel>> entry : grouped.entrySet()) {
+			ReentrantLock lock = catalogDataManager.getDeviceWriteLock(entry.getKey());
+			lock.lock();
 			try {
-				switch (notifyCatalogChannel.getType()) {
+				for (NotifyCatalogChannel notifyCatalogChannel : entry.getValue()) {
-					case STATUS_CHANGED:
+					try {
-						deviceChannelService.updateChannelStatusForNotify(notifyCatalogChannel.getChannel());
+						switch (notifyCatalogChannel.getType()) {
-						CommonGBChannel channelForStatus = channelService.queryCommonChannelByDeviceChannel(notifyCatalogChannel.getChannel());
+							case STATUS_CHANGED:
-						if ("ON".equals(notifyCatalogChannel.getChannel().getStatus()) ) {
+								deviceChannelService.updateChannelStatusForNotify(notifyCatalogChannel.getChannel());
-							eventPublisher.channelEventPublish(channelForStatus, ChannelEvent.ChannelEventMessageType.ON);
+								CommonGBChannel channelForStatus = channelService.queryCommonChannelByDeviceChannel(notifyCatalogChannel.getChannel());
-						}else {
+								if ("ON".equals(notifyCatalogChannel.getChannel().getStatus()) ) {
-							eventPublisher.channelEventPublish(channelForStatus, ChannelEvent.ChannelEventMessageType.OFF);
+									eventPublisher.channelEventPublish(channelForStatus, ChannelEvent.ChannelEventMessageType.ON);
+								}else {
+									eventPublisher.channelEventPublish(channelForStatus, ChannelEvent.ChannelEventMessageType.OFF);
+								}
+								break;
+							case ADD:
+								deviceChannelService.addChannel(notifyCatalogChannel.getChannel());
+								CommonGBChannel channelForAdd = channelService.getOne(notifyCatalogChannel.getChannel().getId());
+								eventPublisher.channelEventPublish(channelForAdd, ChannelEvent.ChannelEventMessageType.ADD);
+								break;
+							case UPDATE:
+								CommonGBChannel oldCommonChannel = channelService.getOne(notifyCatalogChannel.getChannel().getId());
+								deviceChannelService.updateChannelForNotify(notifyCatalogChannel.getChannel());
+								CommonGBChannel channel = channelService.getOne(oldCommonChannel.getGbId());
+								eventPublisher.channelEventPublishForUpdate(channel, oldCommonChannel);
+								break;
+							case DELETE:
+								CommonGBChannel oldCommonChannelForDelete = channelService.queryCommonChannelByDeviceChannel(notifyCatalogChannel.getChannel());
+								deviceChannelService.deleteForNotify(notifyCatalogChannel.getChannel());
+								eventPublisher.channelEventPublish(oldCommonChannelForDelete, ChannelEvent.ChannelEventMessageType.DEL);
+								break;
 						}
-						break;
+					}catch (Exception e) {
-					case ADD:
+						log.error("[存储收到的通道-异常]类型：{}，编号：{}", notifyCatalogChannel.getType(),
-						deviceChannelService.addChannel(notifyCatalogChannel.getChannel());
+								notifyCatalogChannel.getChannel().getDeviceId(), e);
-						CommonGBChannel channelForAdd = channelService.getOne(notifyCatalogChannel.getChannel().getId());
+					}
-						eventPublisher.channelEventPublish(channelForAdd, ChannelEvent.ChannelEventMessageType.ADD);
-						break;
-					case UPDATE:
-						CommonGBChannel oldCommonChannel = channelService.getOne(notifyCatalogChannel.getChannel().getId());
-						deviceChannelService.updateChannelForNotify(notifyCatalogChannel.getChannel());
-						CommonGBChannel channel = channelService.getOne(oldCommonChannel.getGbId());
-						eventPublisher.channelEventPublishForUpdate(channel, oldCommonChannel);
-						break;
-					case DELETE:
-						CommonGBChannel oldCommonChannelForDelete = channelService.queryCommonChannelByDeviceChannel(notifyCatalogChannel.getChannel());
-						deviceChannelService.deleteForNotify(notifyCatalogChannel.getChannel());
-						eventPublisher.channelEventPublish(oldCommonChannelForDelete, ChannelEvent.ChannelEventMessageType.DEL);
-						break;
 				}
-			}catch (Exception e) {
+			} finally {
-				log.error("[存储收到的通道-异常]类型：{}，编号：{}", notifyCatalogChannel.getType(),
+				lock.unlock();
-						notifyCatalogChannel.getChannel().getDeviceId(), e);
 			}
 		}
 	}

src/main/java/com/genersoft/iot/vmp/gb28181/transmit/event/request/impl/message/response/cmd/CatalogResponseMessageHandler.java

@@ -3,8 +3,6 @@ package com.genersoft.iot.vmp.gb28181.transmit.event.request.impl.message.respon
 import com.genersoft.iot.vmp.conf.SipConfig;
 import com.genersoft.iot.vmp.gb28181.bean.*;
 import com.genersoft.iot.vmp.gb28181.service.IDeviceChannelService;
-import com.genersoft.iot.vmp.gb28181.service.IGroupService;
-import com.genersoft.iot.vmp.gb28181.service.IRegionService;
 import com.genersoft.iot.vmp.gb28181.session.CatalogDataManager;
 import com.genersoft.iot.vmp.gb28181.transmit.event.request.SIPRequestProcessorParent;
 import com.genersoft.iot.vmp.gb28181.transmit.event.request.impl.message.IMessageHandler;
@@ -17,7 +15,6 @@ import org.dom4j.Element;
 import org.springframework.beans.factory.InitializingBean;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
-import org.springframework.transaction.annotation.Transactional;
 
 import javax.sip.InvalidArgumentException;
 import javax.sip.RequestEvent;
@@ -28,9 +25,6 @@ import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
-import java.util.concurrent.ConcurrentLinkedQueue;
-import java.util.concurrent.locks.ReentrantLock;
-import java.lang.Thread;
 
 /**
  * 目录查询的回复
@@ -44,17 +38,9 @@ public class CatalogResponseMessageHandler extends SIPRequestProcessorParent imp
     @Autowired
     private ResponseMessageHandler responseMessageHandler;
 
-    private final ConcurrentLinkedQueue<HandlerCatchData> taskQueue = new ConcurrentLinkedQueue<>();
-
     @Autowired
     private IDeviceChannelService deviceChannelService;
 
-    @Autowired
-    private IRegionService regionService;
-
-    @Autowired
-    private IGroupService groupService;
-
     @Autowired
     private CatalogDataManager catalogDataCatch;
 
@@ -155,58 +141,19 @@ public class CatalogResponseMessageHandler extends SIPRequestProcessorParent imp
                     catalogDataCatch.put(device.getDeviceId(), sn, sumNum, device,
                             channelList, regionList, groupList);
                     log.info("[收到通道]设备: {} -> {}个，{}/{}", device.getDeviceId(), channelList.size(), catalogDataCatch.size(device.getDeviceId(), sn), sumNum);
+
+                    if (catalogDataCatch.size(device.getDeviceId(), sn) > 0
+                            && catalogDataCatch.size(device.getDeviceId(), sn) == catalogDataCatch.sumNum(device.getDeviceId(), sn)) {
+                        catalogDataCatch.setComplete(device.getDeviceId(), sn);
+                    }
                 }
             }
         } catch (Exception e) {
             log.warn("[收到通道] 发现未处理的异常, \r\n{}", evt.getRequest());
             log.error("[收到通道] 异常内容： ", e);
-        } finally {
-            String deviceId = device.getDeviceId();
-            if (catalogDataCatch.size(deviceId, sn) > 0
-                    && catalogDataCatch.size(deviceId, sn) == catalogDataCatch.sumNum(deviceId, sn)) {
-                    // 数据已经完整接收， 此时可能存在某个设备离线变上线的情况，但是考虑到性能，此处不做处理，
-                // 目前支持设备通道上线通知时和设备上线时向上级通知
-                int finalSn = sn;
-                Thread.startVirtualThread(() -> {
-                    ReentrantLock lock = catalogDataCatch.getDeviceWriteLock(device.getDeviceId());
-                    lock.lock();
-                    try {
-                        boolean resetChannelsResult = saveData(device, finalSn);
-                        if (!resetChannelsResult) {
-                            String errorMsg = "接收成功，写入失败，共" + catalogDataCatch.sumNum(deviceId, finalSn) + "条，已接收" + catalogDataCatch.getDeviceChannelList(device.getDeviceId(), finalSn).size() + "条";
-                            catalogDataCatch.setChannelSyncEnd(deviceId, finalSn, errorMsg);
-                        } else {
-                            catalogDataCatch.setChannelSyncEnd(deviceId, finalSn, null);
-                        }
-                    } finally {
-                        lock.unlock();
-                    }
-                });
-            }
         }
     }
 
-    @Transactional
-    public boolean saveData(Device device, int sn) {
-
-        boolean result = true;
-        List<DeviceChannel> deviceChannelList = catalogDataCatch.getDeviceChannelList(device.getDeviceId(), sn);
-        if (deviceChannelList != null && !deviceChannelList.isEmpty()) {
-            result &= deviceChannelService.resetChannels(device.getId(), deviceChannelList);
-        }
-
-        List<Region> regionList = catalogDataCatch.getRegionList(device.getDeviceId(), sn);
-        if ( regionList!= null && !regionList.isEmpty()) {
-            result &= regionService.batchAdd(regionList);
-        }
-
-        List<Group> groupList = catalogDataCatch.getGroupList(device.getDeviceId(), sn);
-        if (groupList != null && !groupList.isEmpty()) {
-            result &= groupService.batchAdd(groupList);
-        }
-        return result;
-    }
-
     @Override
     public void handForPlatform(RequestEvent evt, Platform parentPlatform, Element rootElement) {