[Libreoffice-commits] online.git: 2 commits - loolwsd/LOOLSession.cpp loolwsd/LOOLSession.hpp
Tor Lillqvist
tml at collabora.com
Fri Sep 16 15:29:34 UTC 2016
loolwsd/LOOLSession.cpp | 2 +-
loolwsd/LOOLSession.hpp | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
New commits:
commit 20d6acae29fd9685e58952e2597834a27e9cb9cc
Author: Tor Lillqvist <tml at collabora.com>
Date: Fri Sep 16 15:57:36 2016 +0300
Return true in LOOLSession::forwardToPeer() when peer is closing
Fixes the testEditAnnotationWriter unit test, and doesn't break any
other ones. Many other tests do fail, but they fail also without this
change.
diff --git a/loolwsd/LOOLSession.hpp b/loolwsd/LOOLSession.hpp
index 4c7c485..79b4e1a 100644
--- a/loolwsd/LOOLSession.hpp
+++ b/loolwsd/LOOLSession.hpp
@@ -108,13 +108,13 @@ protected:
else if (peer->isCloseFrame())
{
Log::trace(getName() + ": peer began the closing handshake. Dropping forward message [" + message + "].");
- return false;
+ return true;
}
else if (peer->isHeadless())
{
// Fail silently and return as there is no actual websocket
// connection in this case.
- Log::info(getName() + ": Ignoring forward message due to peer being headless");
+ Log::info(getName() + ": Headless peer, not forwarding message [" + message + "].");
return true;
}
commit fbd738c32adf12274e2562dfffbfb41a6a6b0b4e
Author: Tor Lillqvist <tml at collabora.com>
Date: Fri Sep 16 13:16:49 2016 +0300
One more
diff --git a/loolwsd/LOOLSession.cpp b/loolwsd/LOOLSession.cpp
index f9510f1..6b37618 100644
--- a/loolwsd/LOOLSession.cpp
+++ b/loolwsd/LOOLSession.cpp
@@ -218,7 +218,7 @@ bool LOOLSession::handleInput(const char *buffer, int length)
const auto summary = getAbbreviatedMessage(buffer, length);
try
{
- Log::trace(getName() + " Recv: " + summary);
+ Log::trace(getName() + ": Recv: " + summary);
if (TerminationFlag)
{
Log::warn("Input while terminating: [" + summary + "].");
More information about the Libreoffice-commits
mailing list