Resolve merge conflicts

evennia/server/portal/telnet.py

@@ -83,7 +83,11 @@ class TelnetProtocol(Telnet, StatefulTelnetProtocol, _BASE_SESSION_CLASS):
         of incoming data.
         """
         # print(f"telnet dataReceived: {data}")
-        super().dataReceived(data)
+        try:
+            super().dataReceived(data)
+        except ValueError as err:
+            from evennia.utils import logger
+            logger.log_err(f"Malformed telnet input: {err}")
 
     def connectionMade(self):
         """

evennia/server/portal/webclient.py

@@ -22,6 +22,7 @@ from evennia.utils.utils import mod_import, class_from_module
 from evennia.utils.ansi import parse_ansi
 from evennia.utils.text2html import parse_html
 from autobahn.twisted.websocket import WebSocketServerProtocol
+from autobahn.exception import Disconnected
 
 _RE_SCREENREADER_REGEX = re.compile(
     r"%s" % settings.SCREENREADER_REGEX_STRIP, re.DOTALL + re.MULTILINE
@@ -150,7 +151,7 @@ class WebSocketClient(WebSocketServerProtocol, _BASE_SESSION_CLASS):
         # in case anyone wants to expose this functionality later.
         #
         # sendClose() under autobahn/websocket/interfaces.py
-        self.sendClose(CLOSE_NORMAL, reason)
+        ret = self.sendClose(CLOSE_NORMAL, reason)
 
     def onClose(self, wasClean, code=None, reason=None):
         """
@@ -191,7 +192,12 @@ class WebSocketClient(WebSocketServerProtocol, _BASE_SESSION_CLASS):
             line (str): Text to send.
 
         """
-        return self.sendMessage(line.encode())
+        try:
+            return self.sendMessage(line.encode())
+        except Disconnected:
+            # this can happen on an unclean close of certain browsers.
+            # it means this link is actually already closed.
+            self.disconnect(reason="Browser already closed.")
 
     def at_login(self):
         csession = self.get_client_session()
@@ -249,8 +255,6 @@ class WebSocketClient(WebSocketServerProtocol, _BASE_SESSION_CLASS):
                 return
         else:
             return
-        # just to be sure
-        text = to_str(text)
 
         flags = self.protocol_flags
 

evennia/server/webserver.py

@@ -102,7 +102,8 @@ class EvenniaReverseProxyResource(ReverseProxyResource):
 
         """
         request.notifyFinish().addErrback(
-            lambda f: logger.log_trace("%s\nCaught errback in webserver.py:75." % f)
+            lambda f: 0
+            # lambda f: logger.log_trace("%s\nCaught errback in webserver.py" % f)
         )
         return EvenniaReverseProxyResource(
             self.host, self.port, self.path + "/" + urlquote(path, safe=""), self.reactor
@@ -139,9 +140,9 @@ class EvenniaReverseProxyResource(ReverseProxyResource):
         clientFactory.noisy = False
         self.reactor.connectTCP(self.host, self.port, clientFactory)
         # don't trigger traceback if connection is lost before request finish.
-        request.notifyFinish().addErrback(
-            lambda f: logger.log_trace("%s\nCaught errback in webserver.py:75." % f)
-        )
+        request.notifyFinish().addErrback(lambda f: 0)
+        # request.notifyFinish().addErrback(
+        #   lambda f:logger.log_trace("Caught errback in webserver.py: %s" % f)
         return NOT_DONE_YET
 
 
@@ -207,6 +208,11 @@ class DjangoWebRoot(resource.Resource):
         path0 = request.prepath.pop(0)
         request.postpath.insert(0, path0)
 
+        request.notifyFinish().addErrback(
+            lambda f: 0
+            # lambda f: logger.log_trace("%s\nCaught errback in webserver.py:" % f)
+        )
+
         deferred = request.notifyFinish()
         self._pending_requests[deferred] = deferred
         deferred.addBoth(self._decrement_requests, deferred=deferred)