实习踩坑之路：集群场景下，我的redis锁为什么不生效啊？别的机器上的线程给解锁了？

最新推荐文章于 2023-08-12 11:00:24 发布

会写代码的花城

最新推荐文章于 2023-08-12 11:00:24 发布

阅读量960

点赞数 1

分类专栏：实习踩坑之路 reids 文章标签： redis java 集群分布式锁分布式锁锁被别的线程给释放了

本文链接：https://blog.csdn.net/qq_22155255/article/details/121772322

版权

在一次代码评审中，作者发现其在集群环境中使用Redis实现的分布式锁存在安全隐患。问题包括加锁解锁操作未妥善处理异常，可能导致锁在未执行完毕时被其他机器线程释放。解决方案包括确保只有加锁成功的线程能解锁，以及在解锁时校验唯一标识以防止误解锁。建议采用如线程ID或UUID作为解锁凭证。

摘要由CSDN通过智能技术生成

今天在小组codereview的时候，被大佬们揪出来我的程序写的拉的一点，分享出来，部分代码涉及内部机密，我用注释尽量描述清楚

场景

我是一个对话场景，需要对群会话加锁，然后去自动回复的，也就是说，我需要在消息进来的时候，不能让同一个群的所有人触发我这个自动回复的规则，只有在拿到锁的执行完自动回复解锁之后，在进行加锁/解锁

代码

  			//判断需要加锁
            boolean lockFlag = judgeLock(chatMsgDO.getChatType(),autoReplyRuleDO);
            if (lockFlag){
   
                //群聊消息处理
                //会话id + 规则主键ID
                String key = RedisKeyConstant.genAutoReplyKey(chatMsgDO.getBindId(),autoReplyRuleDO.getId());
                try{
   
                    redisService.lockWithExpireTimeAndRetryTimes(key, 60, 5);
                }catch (Exception e){
   
                    log.info("[关键词自动回复(群聊)]加锁失败，{}微信不处理",chatMsgDO.getReceiverId());
                }
                //直接保证只有一个托管微信进行同一个消息内容的自动回复
                //判断群聊回复时是否需要@
                 boolean mentionSwitch = judgeNeedMention(autoReplyRuleDO);
                 if (mentionSwitch ==