• Views
  • Iteration Report
  • My Iteration Report
  •  
OMERO.server
  • Login
  • Help/Guide
  • About Trac
  • Preferences
  • Wiki
  • Timeline
  • Roadmap
  • Browse Source
  • View Tickets
  • Search

Context Navigation

  • ← Previous Ticket
  • Next Ticket →

Ticket #428 (closed defect: fixed)

Opened 23 months ago

Last modified 21 months ago

RE.load() stumbles on a SecuritySystem error.

Reported by: jmoore Owned by: jmoore
Priority: blocker Milestone: 3.0-Beta1
Component: Services Version: 3.0-M3
Keywords: Cc: callan

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

Changed 23 months ago by jmoore

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

Changed 23 months 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.

Changed 23 months ago by jmoore

  • status changed from new to closed
  • resolution set to fixed

Chris seems to be happy. Closing.

Changed 22 months ago by jmoore

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

Changed 21 months ago by jmoore

  • keywords iteration1 removed
  • status changed from closed to reopened
  • resolution deleted

Changed 21 months ago by jmoore

  • status changed from reopened to closed
  • resolution set to fixed

r1112 merges.

Note: See TracTickets for help on using tickets.

Download in other formats:

  • Comma-delimited Text
  • Tab-delimited Text
  • RSS Feed

Trac Powered

Powered by Trac 0.11
By Edgewall Software.

Visit the Trac open source project at
http://trac.edgewall.org/