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 #7747 (closed)

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

Bug: enableKeepAlive hangs python

Reported by: jamoore Owned by: jamoore
Priority: minor Milestone: OMERO-4.4
Component: General Version: n.a.
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2012-01-17 (6)

Description

The script from #2973 hangs after createSession throws (with a bad login) since enableKeepAlive created a thread. A thread should only be activated when a session is present. Otherwise, i.e. after closeSession and on a failed createSession, there should be no actual thread, only the keep alive setting should be modified.

Change History (4)

comment:1 Changed 12 years ago by jmoore

  • Status changed from new to accepted

comment:2 Changed 12 years ago by jmoore

  • Remaining Time changed from 0.1 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed

Refactored startKeepAlive and stopKeepAlive out of enableKeepAlive. createSession and closeSession now use them more appropriately. Added mock tests to show the possibilities.

comment:3 Changed 12 years ago by jmoore

Forgot to mention: pushed to sprint6-server-backlog

comment:4 Changed 12 years ago by jmoore <josh@…>

(In [c301e9f9460e461d2abda80655d4c3a1d217eea6/ome.git] on branch develop) Refactor enableKeepAlive to not hang (Fix #7747)

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.69190 sec.)

We're Hiring!