Skip to content

Commit fc59699

Browse files
committed
session bug fix for remote auth
1 parent 2bbdc5a commit fc59699

File tree

4 files changed

+405
-17
lines changed

4 files changed

+405
-17
lines changed

packages/slycat/web/server/remote.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -705,8 +705,11 @@ def browse(self, path, file_reject, file_allow, directory_reject, directory_allo
705705
return response
706706
except Exception as e:
707707
cherrypy.log.error(
708-
"Exception reading remote file %s: %s %s" % (path, type(e), str(e))
708+
"Exception reading remote file %s: %s, %s" % (path, type(e), str(e))
709709
)
710+
if str(e) == "Socket is closed" or str(e) == "Server connection dropped: ":
711+
delete_session(self._sid)
712+
raise cherrypy.HTTPError("400 Socket closed")
710713

711714
if str(e) == "Garbage packet received":
712715
cherrypy.response.headers["x-slycat-message"] = (
@@ -939,7 +942,9 @@ def get_file(self, path, **kwargs):
939942
cherrypy.log.error(
940943
"Exception reading remote file %s: %s %s" % (path, type(e), str(e))
941944
)
942-
945+
if str(e) == "Socket is closed":
946+
delete_session(self._sid)
947+
raise cherrypy.HTTPError("400 Socket closed")
943948
if "Garbage packet received" in str(e):
944949
cherrypy.response.headers["x-slycat-message"] = (
945950
"Remote access failed: %s" % str(e)

0 commit comments

Comments
 (0)