From c8ea8185b453428fdc90ed9bf3e83b50c6cc8dd8 Mon Sep 17 00:00:00 2001
From: Sander Speetjens <sander.speetjens@gmail.com>
Date: Mon, 20 Oct 2025 20:16:08 +0200
Subject: [PATCH] server websocket

Removes unnecessary debug and trace logging statements from the websocket server implementation to reduce verbosity. Also removes unused binary message processing.
---
 server/websocket.cpp | 15 ---------------
 server/websocket.hpp |  1 -
 2 files changed, 16 deletions(-)

diff --git a/server/websocket.cpp b/server/websocket.cpp
index c467665..5342148 100644
--- a/server/websocket.cpp
+++ b/server/websocket.cpp
@@ -12,7 +12,6 @@ Websocket::Websocket(uint16_t port, QObject *parent)
 {
     if (m_pWebSocketServer->listen(QHostAddress::Any, port))
     {
-        SPDLOG_DEBUG("Server listening on port {}", port);
         connect(m_pWebSocketServer, &QWebSocketServer::newConnection, this, &Websocket::onNewConnection);
         connect(m_pWebSocketServer, &QWebSocketServer::closed, this, &Websocket::closed);
     }
@@ -37,7 +36,6 @@ Websocket::Websocket(uint16_t port, QString key_file, QString cert_file, QObject
     m_pWebSocketServer->setSslConfiguration(sslConfiguration);
     if (m_pWebSocketServer->listen(QHostAddress::Any, port))
     {
-        SPDLOG_DEBUG("Server listening on port {}", port);
         connect(m_pWebSocketServer, &QWebSocketServer::newConnection, this, &Websocket::onNewConnection);
         connect(m_pWebSocketServer, &QWebSocketServer::closed, this, &Websocket::closed);
     }
@@ -60,7 +58,6 @@ void Websocket::broadcast(QString message)
 void Websocket::send(QWebSocket *conn, QString message)
 {
     conn->sendTextMessage(message);
-    SPDLOG_TRACE("sending message: {}", message.toStdString());
 }
 
 void Websocket::onNewConnection()
@@ -68,32 +65,20 @@ void Websocket::onNewConnection()
     QWebSocket *pSocket = m_pWebSocketServer->nextPendingConnection();
 
     connect(pSocket, &QWebSocket::textMessageReceived, this, &Websocket::processTextMessage);
-    connect(pSocket, &QWebSocket::binaryMessageReceived, this, &Websocket::processBinaryMessage);
     connect(pSocket, &QWebSocket::disconnected, this, &Websocket::socketDisconnected);
 
     m_clients << pSocket;
-    SPDLOG_DEBUG("socket connected");
 }
 
 void Websocket::processTextMessage(QString message)
 {
     QWebSocket *pClient = qobject_cast<QWebSocket *>(sender());
-    SPDLOG_TRACE("Message received: {}", message.toStdString());
     emit on_message(pClient, message);
 }
 
-void Websocket::processBinaryMessage(QByteArray message)
-{
-    QWebSocket *pClient = qobject_cast<QWebSocket *>(sender());
-    SPDLOG_TRACE("Binary Message received: {}", message.data());
-    Q_UNUSED(message);
-    Q_UNUSED(pClient);
-}
-
 void Websocket::socketDisconnected()
 {
     QWebSocket *pClient = qobject_cast<QWebSocket *>(sender());
-    SPDLOG_DEBUG("socket Disconnected");
     if (pClient)
     {
         m_clients.removeAll(pClient);
diff --git a/server/websocket.hpp b/server/websocket.hpp
index bd9558d..fd4e393 100644
--- a/server/websocket.hpp
+++ b/server/websocket.hpp
@@ -24,7 +24,6 @@ class Websocket : public QObject
   private slots:
     void onNewConnection();
     void processTextMessage(QString message);
-    void processBinaryMessage(QByteArray message);
     void socketDisconnected();
 
   private: