Ticket #220 (defect)

It might also be nice to include a sample sqlobject definition of how to use the sqlobject session storage type, eg:

import cherrypy
from sqlobject import *

class Session(SQLObject):
    session_key = StringCol(length=40, notNone=True)
    timeout = IntCol(notNone=True, default=60*60)
    last_access = FloatCol(default=time.time())
    timestamp = FloatCol(default=time.time())

    # Add other session-associated fields here

    sessionKeyIndex = DatabaseIndex('session_key')

cherrypy.config.update({'global':
                        {'sessionFilter.storageType': 'sqlobject',
                         'sessionFilter.default.tableObject': Session}})

My original sqlobjectadaptor.py patch had some problems. Here's an updated one:

--- cherrypy/lib/filter/sessionfilter/sqlobjectadaptor.py       (revision 443)
+++ cherrypy/lib/filter/sessionfilter/sqlobjectadaptor.py       (working copy)
@@ -60,6 +60,8 @@
     def __setattr__(self, attr, value):
         if attr == 'key' or attr == 'timestamp':
             raise SessionImmutableError
+        elif attr == 'timeout' or attr == 'lastAccess':
+            setattr(self.__sqlObject, self.__attrSub(attr), value)
         else:
             object.__setattr__(self, self.__attrSub(attr), value)

@@ -117,12 +119,13 @@

     def delSession(self, sessionKey):
         # figure out what to catch when this doesn't work
-        Session.delete(Session.q.session_key==sessionKey)
+        for session in self.Session.select(self.Session.q.session_key == sessionKey):
+            session.destroySelf()

         #raise SessionNotFoundError

     def cleanUpOldSessions(self):
         # print cleaning up sql sessions
         now = time.time()
-        for session in Session.select( ((now - Session.q.last_access) < Session.q.timeout) ):
-            Session.delete(session.id)
+        for session in self.Session.select((now - self.Session.q.last_access) > self.Session.q.timeout):
+            session.destroySelf()

The sqlobject adaptor has been very low priority, now that the sessionFilter api is frozen it should start to get more attention. It also appears that the sqlobject is not 100% thread safe, using the the adaptor will result in occasional errors.

It might be a while before the adaptor is ready for production applications, patches are welcome. For know it will raise "a warning that it is buggy"

In the meantime, If anyone knows what type of exception sqlobject raises if a delete fails please let me know.

the expired() function is fixed

changed the topic and deleted everything that is no longer relavent.

An sqlobject adaptor requires uneccessary complexity do to convertion to and from python dictionarys. The sqlobject adaptor has been dropped. A driver based on PyDO2 will take its place. (I am not against including an sqlobject adaptor if someone else would like to contribute it, but I suggesting waiting till after I release the PyDO adaptor, so you can see how column names are handled.)