From 36ac75a5c9c118c10f70beb742c087eafa742048 Mon Sep 17 00:00:00 2001
From: Matthew Letter <mletter1@unm.edu>
Date: Fri, 26 Aug 2016 10:35:30 -0600
Subject: [PATCH] fixed bug cleanup sessions, resolves @alexsielicki login
 breaking on server restart #633

---
 packages/slycat/web/server/handlers.py | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/packages/slycat/web/server/handlers.py b/packages/slycat/web/server/handlers.py
index a5171b446..a37e88f73 100644
--- a/packages/slycat/web/server/handlers.py
+++ b/packages/slycat/web/server/handlers.py
@@ -1759,16 +1759,16 @@ def post_remotes():
     database = slycat.web.server.database.couchdb.connect()
     session = database.get("session", cherrypy.request.cookie["slycatauth"].value)
     hostname_not_found = True
-    for session in session["sessions"]:
-      if session["hostname"] == hostname:
-        session["sid"] = sid
-        session["username"] = username
+    for i in xrange(len(session["sessions"])):
+      if session["sessions"][i]["hostname"] == hostname:
+        session["sessions"][i]["sid"] = sid
+        session["sessions"][i]["username"] = username
         hostname_not_found = False
     if hostname_not_found:
       session["sessions"].append({"sid": sid,"hostname": hostname, "username": username})
     database.save(session)
   except Exception as e:
-    cherrypy.log.error("could not save session for remotes %s" % e)
+    cherrypy.log.error("login could not save session for remotes %s" % e)
   return {"sid":sid, "status":True, "msg":""}
 
 @cherrypy.tools.json_out(on = True)
@@ -1784,16 +1784,16 @@ def get_remotes(hostname):
   try:
     database = slycat.web.server.database.couchdb.connect()
     session = database.get("session", cherrypy.request.cookie["slycatauth"].value)
-    for session in session["sessions"]:
-      if session["hostname"] == hostname:
-        if slycat.web.server.remote.check_session(session["sid"]):
+    for h_session in session["sessions"]:
+      if h_session["hostname"] == hostname:
+        if slycat.web.server.remote.check_session(h_session["sid"]):
           status = True
           msg = "hostname session was found"
         else:
           session["sessions"][:] = [tup for tup in session["sessions"] if tup["hostname"] != hostname]
           database.save(session)
   except Exception as e:
-    cherrypy.log.error("could not save session for remotes %s" % e)
+    cherrypy.log.error("status could not save session for remotes %s" % e)
   return {"status":status, "msg":msg}
 
 def delete_remote(sid):