Skip to content

Commit e199d51

Browse files
authored
Update messageStore.js to fix Issue:It it duplicating the self messages after refreshing the window in examples/private messages socketio#5340
1 parent df2a3aa commit e199d51

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

examples/private-messaging/server/messageStore.js

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,16 @@ class RedisMessageStore extends MessageStore {
3030

3131
saveMessage(message) {
3232
const value = JSON.stringify(message);
33-
this.redisClient
34-
.multi()
35-
.rpush(`messages:${message.from}`, value)
36-
.rpush(`messages:${message.to}`, value)
33+
const multi = this.redisClient.multi();
34+
35+
multi.rpush(`messages:${message.from}`, value);
36+
37+
// Only push to recipient if different from sender
38+
if (message.from !== message.to) {
39+
multi.rpush(`messages:${message.to}`, value);
40+
}
41+
42+
multi
3743
.expire(`messages:${message.from}`, CONVERSATION_TTL)
3844
.expire(`messages:${message.to}`, CONVERSATION_TTL)
3945
.exec();

0 commit comments

Comments
 (0)