LogalDeveloper/owncast
LogalDeveloper/owncast
0
Fork 0
Code

fix(chat): do not send chat join message

Browse Source 
If a user is already connected to chat from elsewhere, do not send an
additional chat join message. Closes #3487
This commit is contained in:
Gabe Kangas 2024-02-18 15:49:50 -08:00
parent 19f257151e
commit ebf64dd17d
No known key found for this signature in database
GPG Key ID: 4345B2060657F330
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
core/chat/server.go
View File

@ -96,6 +96,13 @@ func (s *Server) Addclient(conn *websocket.Conn, user *user.User, accessToken st
shouldSendJoinedMessages := data.GetChatJoinPartMessagesEnabled()
// If there are existing clients connected for this user do not send
// a user joined message. Do not put this under a mutex, as
// GetClientsForUser already has a lock.
if existingConnectedClients, _ := GetClientsForUser(user.ID); len(existingConnectedClients) > 0 {
shouldSendJoinedMessages = false
}
s.mu.Lock()
{
// If there is a pending disconnect timer then clear it.