fix : refactor go-lint code

pet-sitter · Jan 19, 2025 · be42d04 · be42d04
1 parent d95152e
commit be42d04
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/internal/service/chat_service.go b/internal/service/chat_service.go
@@ -414,7 +414,9 @@ func (s *ChatService) SaveChatMessage(
 		return nil, databaseGenError
 	}
 
-	tx.Commit()
+	if err := tx.Commit(); err != nil {
+		return nil, err
+	}
 
 	return chat.ToChatRoomMessage(row), nil
 }
diff --git a/internal/wschat/server.go b/internal/wschat/server.go
@@ -125,14 +125,27 @@ func (s *WSServer) LoopOverClientMessages() {
 			var msg MessageResponse
 			switch msgReq.MessageType {
 			case "plain":
-				s.chatService.SaveChatMessage(
+				saveChatMessage, err := s.chatService.SaveChatMessage(
 					ctx,
 					msgReq.Sender.ID,
 					msgReq.Room.ID,
 					msgReq.MessageType,
 					msgReq.Message,
 				)
+				if err != nil {
+					log.Error().Err(err).Msg("Failed to save chat message")
+					msg = NewErrorMessageResponse(
+						msgReq.MessageID,
+						msgReq.Sender,
+						msgReq.Room,
+						"Failed to save chat message",
+						time.Now(),
+					)
+
+					break
+				}
 
+				log.Info().Msg("Save chat message: " + saveChatMessage.ID.String())
 				msg = NewPlainMessageResponse(
 					msgReq.MessageID,
 					msgReq.Sender,