opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(wecom): guard dedup map cleanup under mutex

Browse source
This commit is contained in:
Jared Mahotiere 2026-02-23 13:21:39 -05:00
parent efb68501b9
commit a37b3ba3ff
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
pkg/channels/wecom.go
View file

@ -339,14 +339,15 @@ func (c *WeComBotChannel) processMessage(ctx context.Context, msg WeComBotMessag
return return
} }
c.processedMsgs[msgID] = true c.processedMsgs[msgID] = true
c.msgMu.Unlock()
// Clean up old messages periodically (keep last 1000) // Clean up old messages periodically (keep last 1000).
// Keep the current message id in the new map to preserve dedup behavior.
if len(c.processedMsgs) > 1000 { if len(c.processedMsgs) > 1000 {
c.msgMu.Lock() c.processedMsgs = map[string]bool{
c.processedMsgs = make(map[string]bool) msgID: true,
c.msgMu.Unlock() }
} }
c.msgMu.Unlock()
senderID := msg.From.UserID senderID := msg.From.UserID