Enhance logging and error handling in sendMessageToConnection method
- Introduce detailed logging for the message sending process, including checks for user data validity and message queue status. - Implement additional null checks for user data before and after locking the mutex to ensure thread safety. - Ensure proper message copying to maintain validity during queuing, improving overall stability and error visibility.
This commit is contained in:
committed by
Torsten (PC)
parent
2595cb8565
commit
1af4b6c2e4
@@ -845,31 +845,59 @@ void WebSocketServer::sendMessageToConnection(struct lws *wsi, WebSocketUserData
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
std::cout << "[sendMessageToConnection] Start, wsi=" << wsi << ", ud=" << ud << ", message.length()=" << message.length() << std::endl;
|
||||||
|
|
||||||
try {
|
try {
|
||||||
// Prüfe ob wsi noch gültig ist (wenn verfügbar)
|
// Prüfe ob ud noch gültig ist, bevor wir den Mutex locken
|
||||||
// lws_wsi_is_valid ist nicht in allen Versionen verfügbar, daher try-catch
|
std::cout << "[sendMessageToConnection] Prüfe ud-Gültigkeit..." << std::endl;
|
||||||
|
volatile bool test = ud->pongReceived;
|
||||||
|
(void)test;
|
||||||
|
std::cout << "[sendMessageToConnection] ud ist gültig" << std::endl;
|
||||||
|
|
||||||
|
// Kopiere die Nachricht, um sicherzustellen, dass sie gültig bleibt
|
||||||
|
std::string messageCopy = message;
|
||||||
|
std::cout << "[sendMessageToConnection] Nachricht kopiert: " << messageCopy.length() << " Bytes" << std::endl;
|
||||||
|
|
||||||
bool wasEmpty = false;
|
bool wasEmpty = false;
|
||||||
{
|
{
|
||||||
|
std::cout << "[sendMessageToConnection] Versuche Mutex zu locken..." << std::endl;
|
||||||
std::lock_guard<std::mutex> lock(ud->messageQueueMutex);
|
std::lock_guard<std::mutex> lock(ud->messageQueueMutex);
|
||||||
|
std::cout << "[sendMessageToConnection] Mutex gelockt" << std::endl;
|
||||||
|
|
||||||
|
// Prüfe ob ud noch gültig ist, nachdem wir den Mutex gelockt haben
|
||||||
|
if (!ud) {
|
||||||
|
std::cerr << "[sendMessageToConnection] ud wurde während des Lockens ungültig" << std::endl;
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
std::cout << "[sendMessageToConnection] Prüfe Queue-Größe..." << std::endl;
|
||||||
wasEmpty = ud->messageQueue.empty();
|
wasEmpty = ud->messageQueue.empty();
|
||||||
ud->messageQueue.push(message);
|
std::cout << "[sendMessageToConnection] Queue war leer: " << wasEmpty << std::endl;
|
||||||
|
|
||||||
|
std::cout << "[sendMessageToConnection] Füge Nachricht zur Queue hinzu..." << std::endl;
|
||||||
|
ud->messageQueue.push(messageCopy);
|
||||||
|
std::cout << "[sendMessageToConnection] Nachricht zur Queue hinzugefügt, neue Größe: " << ud->messageQueue.size() << std::endl;
|
||||||
}
|
}
|
||||||
|
|
||||||
// Nur wenn die Queue leer war, den Callback aufrufen
|
// Nur wenn die Queue leer war, den Callback aufrufen
|
||||||
// (sonst wird er bereits durch den WRITEABLE-Handler aufgerufen)
|
// (sonst wird er bereits durch den WRITEABLE-Handler aufgerufen)
|
||||||
if (wasEmpty) {
|
if (wasEmpty) {
|
||||||
|
std::cout << "[sendMessageToConnection] Queue war leer, rufe lws_cancel_service auf..." << std::endl;
|
||||||
// Verwende lws_cancel_service, um den Service zu benachrichtigen
|
// Verwende lws_cancel_service, um den Service zu benachrichtigen
|
||||||
// Das ist sicherer, wenn wir uns in einem Callback befinden
|
// Das ist sicherer, wenn wir uns in einem Callback befinden
|
||||||
// lws_cancel_service triggert einen Service-Loop, der dann LWS_CALLBACK_SERVER_WRITEABLE aufruft
|
// lws_cancel_service triggert einen Service-Loop, der dann LWS_CALLBACK_SERVER_WRITEABLE aufruft
|
||||||
if (context) {
|
if (context) {
|
||||||
lws_cancel_service(context);
|
lws_cancel_service(context);
|
||||||
|
std::cout << "[sendMessageToConnection] lws_cancel_service aufgerufen" << std::endl;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
std::cout << "[sendMessageToConnection] Erfolgreich abgeschlossen" << std::endl;
|
||||||
} catch (const std::exception &e) {
|
} catch (const std::exception &e) {
|
||||||
std::cerr << "[sendMessageToConnection] Fehler: " << e.what() << std::endl;
|
std::cerr << "[sendMessageToConnection] Fehler: " << e.what() << std::endl;
|
||||||
} catch (...) {
|
} catch (...) {
|
||||||
std::cerr << "[sendMessageToConnection] Unbekannter Fehler" << std::endl;
|
std::cerr << "[sendMessageToConnection] Unbekannter Fehler" << std::endl;
|
||||||
|
} catch (...) {
|
||||||
|
std::cerr << "[sendMessageToConnection] Unbekannter Fehler" << std::endl;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user