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"

Bug #428 (closed)

Opened 18 years ago

Closed 17 years ago

RE.load() stumbles on a SecuritySystem error.

Reported by: jamoore Owned by: jamoore
Priority: blocker Cc: cxallan
Sprint: n.a.
Total Remaining Time: n.a.

Description

Currently the SecuritySystem is being reset at some point (between sessions?) so that a flush later is causing :

Exception in thread "main" ome.conditions.ApiUsageException: The security system is not ready.
Cannot execute: managedDetails
	at ome.security.basic.BasicSecuritySystem.checkReady(BasicSecuritySystem.java:1285)
	at ome.security.basic.BasicSecuritySystem.checkManagedDetails(BasicSecuritySystem.java:470)
	at ome.security.basic.OmeroInterceptor.resetDetails(OmeroInterceptor.java:280)
	at ome.security.basic.OmeroInterceptor.onFlushDirty(OmeroInterceptor.java:165)

Appears that the fix for #427 is now flushing against our wishes :

... (later in stack trace)
at org.hibernate.impl.SessionImpl.flush(SessionImpl.java:1000)
	at org.springframework.orm.hibernate3.SpringSessionSynchronization.beforeCommit(SpringSessionSynchronization.java:130)
	at org.springframework.transaction.support.AbstractPlatformTransactionManager.triggerBeforeCommit(AbstractPlatformTransactionManager.java:709)

effectively undermining the fix for #258.

Should be configurable.

Change History (6)

comment:1 Changed 18 years ago by jmoore

Also related to Hibernate's changing FlushMode's:
http://opensource.atlassian.com/projects/spring/browse/SPR-2623

comment:2 Changed 18 years ago by jmoore

  • Cc callan added

r1039 fixes this. The fix was relatively complicated:

  • Upgrade to Spring 2.0 (#238) to handle FlushMode.MANUAL in addition to NEVER
  • Always set stateful service sessions to MANUAL
  • Subclass HibernateTemplate to allow write operations when FlushMode is MANUAL.


comment:3 Changed 18 years ago by jmoore

  • Resolution set to fixed
  • Status changed from new to closed

Chris seems to be happy. Closing.

comment:4 Changed 17 years ago by jmoore

r1079 adds necessary config. (Added in branch. Will need to be merged)

comment:5 Changed 17 years ago by jmoore

  • Keywords iteration1 deleted
  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:6 Changed 17 years ago by jmoore

  • Resolution set to fixed
  • Status changed from reopened to closed

r1112 merges.

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

We're Hiring!