Ticket #329 (defect)
Opened 3 years ago
Last modified 3 years ago
session lock stays
Status: closed (fixed)
| Reported by: | mdt@emdete.de | Assigned to: | rdelon |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | CherryPy code | Keywords: | |
| Cc: |
under special conditions the session lock is not cleared. i encountered this when doing loadtests with siege.
now i found a condition where this reproduceable occurs: if _cpOnError() throws an exception, the lock is not removed and further calls get an deadlock exception.
Attachments
Change History
10/10/05 22:51:31: Modified by dowski
10/11/05 09:20:35: Modified by mdt@emdete.de
- attachment test.py added.
10/13/05 11:38:01: Modified by mdt@emdete.de
- attachment test.2.py added.
another example on session problems... see my comment please
10/13/05 11:42:59: Modified by mdt@emdete.de
while my other example did not fail on all platform i have made another one today that failed on debian/linux and solaris.
background: images come from database in my app and fail since several days (svn update, i use HEAD revision). i encounter cherrypy/lib/filter/sessionfilter.py", line 175, in saveData sess.sessionStorage.save(sess.sessionID, sess.sessionData, AttributeError?: EmptyClass? instance has no attribute 'sessionStorage'.
i think somehow syncronisation is broken in the session code which also leads to the problem this ticket originaly was created for.
i can't paste the whole app here because it's too fat so i try to reproduce the problem with small snippets that lead to different behaviors... sorry
10/13/05 11:52:14: Modified by mdt@emdete.de
oh, there is a small bug in there, change the mimetype to image/png sorry
10/13/05 17:03:23: Modified by mdt
again... :D
i now traced down the initial bug of this ticket a bit more... it seems to be connected with clients using keep-alive under load. switching this off in the client (siege in my case) works fine while switched on cp locks up.
i will investigate that further, but perhaps we have to split this ticket...
10/14/05 05:00:39: Modified by rdelon
- status changed from new to closed.
- resolution set to fixed.
10/20/05 17:12:28: Modified by gar1a@yahoo.com
This patch fixed my issue where I was rendering a large RTF report and returning it to the user. Thanks! -George
I tried to reproduce this error to no avail. I contacted emdete about it to see if he could provide the code he used to reproduce it.