Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #12435 (closed)

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

clearsessions not remore session files from tmp

Reported by: atarkowska Owned by: atarkowska
Priority: major Milestone: 5.0.3
Component: Web Version: 5.0.2
Keywords: n.a. Cc: python-team@…, John.Webber@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: n.a.

Description (last modified by atarkowska)

OMERO.web uses file based session engine. Currently there is and issues with cleaning session files from tmp directory. Basically running clearsessions doesn't remove them at all.

I have been playing a bit with that to try various configurations and debugging Django code. Basically, get_expiry_agehttps://github.com/django/django/blob/stable/1.6.x/django/contrib/sessions/backends/file.py#L90 returns cookie age rather then negative value.

That seems to be due to session_data.get('_session_expiry') being None.

Noticed with the changes in https://github.com/openmicroscopy/openmicroscopy/pull/2686. Reported to https://code.djangoproject.com/ticket/22938

Change History (4)

comment:1 Changed 10 years ago by atarkowska

  • Description modified (diff)

comment:2 Changed 10 years ago by atarkowska

  • Cc python-team@… John.Webber@… added

comment:3 Changed 10 years ago by atarkowska

  • Resolution set to fixed
  • Status changed from new to closed
Last edited 10 years ago by atarkowska (previous) (diff)

comment:4 Changed 10 years ago by Aleksandra Tarkowska <A.Tarkowska@…>

  • Remaining Time set to 0

(In [e0ca683b5e13208b0f51f08a478755e3405abe8d/ome.git] on branch develop) fixing issues with cleaning sessions, close #12435

This commit add customized filesessionstore.py to replace
django.contrib.sessions.backend.file.py,
see https://code.djangoproject.com/ticket/22938
Once this is fixed filesesionstore.py should be removed in favor of the new
Django version

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.66966 sec.)

We're Hiring!