#8616: File-based sessions dropped with SESSION_SAVE_EVERY_REQUEST = True
----------------------------------------------+-----------------------------
          Reporter:  [EMAIL PROTECTED]    |         Owner:  jacob              
            Status:  reopened                 |     Milestone:  1.0             
   
         Component:  django.contrib.sessions  |       Version:  SVN             
   
        Resolution:                           |      Keywords:  session, file, 
race
             Stage:  Accepted                 |     Has_patch:  1               
   
        Needs_docs:  0                        |   Needs_tests:  0               
   
Needs_better_patch:  0                        |  
----------------------------------------------+-----------------------------
Changes (by [EMAIL PROTECTED]):

  * status:  closed => reopened
  * resolution:  fixed =>

Comment:

 I recognize the value of the slight modifications made to my patch.  An
 exception should not be generated if all goes well.

 However, there is a slight problem with the changes in #8750.  Because the
 "renamed" variable is initialized after the try:finally around the
 encoding and writing of the file, if there is any problem there, we will
 get "NameError: name 'renamed' is not defined" in the finally: block of
 the enclosing try:.

 I am submitting a patch that fixes this.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/8616#comment:28>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to