sessions malfunction at current weboid [beaker] revision

Bug #148214 reported by The Dod
2
Affects Status Importance Assigned to Milestone
web.py
Fix Committed
Undecided
The Dod

Bug Description

Revisions 5,6 of weboid use beaker for sessions (instead of flup).
Session don't function as expected.
The session doesn't "forget" stuff when you log out.
More than that - it even "remembers" everything on *other* machines :)

Evidently, I don't understand something essential in beaker, but I think the best course of action would be to use Karol's sessions at web.py 3.0, as this seems to be the upcoming standard

Revision history for this message
The Dod (the-dod) wrote :

If you want a working version of weboid, please pull revision 4

Revision history for this message
Eleanor Berger (intellectronica) wrote :

I'm working on converting the sessions stuff to a new v3-style processor.

Revision history for this message
Anand Chitipothu (anandology) wrote : Re: [Bug 148214] Re: sessions malfunction at current weboid [beaker] revision

On 03-Oct-07, at 12:13 AM, Tom Berger wrote:

> I'm working on converting the sessions stuff to a new v3-style
> processor.

Thats great.

I think the right way to do is to take application and session
parameters as argument to Session.__init__ and add a processor to the
application instead of loadhook.
Session can be extended from ThreadedDict instead of Storage to store
session data for every request instead of creating a new session
object for every request.

Revision history for this message
Eleanor Berger (intellectronica) wrote :

> I think the right way to do is to take application and session
> parameters as argument to Session.__init__ and add a processor to the
> application instead of loadhook.

That's indeed what I've done so far.

In addition, I am moving most of the functionality out of the session object itself and into the processor (which is a class implementing __call__).

> Session can be extended from ThreadedDict instead of Storage to store
> session data for every request instead of creating a new session
> object for every request.

That's an excellent idea!

Aaron Swartz (aaronsw)
Changed in webpy:
status: New → Invalid
Revision history for this message
The Dod (the-dod) wrote :

Latest version no longer uses beaker, and seems to work with web.py 0.32
Please check

Changed in webpy:
status: Invalid → Fix Committed
assignee: nobody → zzzen (the-dod)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.