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 #9853 (accepted)

Opened 7 years ago

Last modified 4 years ago

Bug: Share metadata loading

Reported by: wmoore Owned by: jamoore
Priority: major Milestone: Permissions
Component: Security Version: n.a.
Keywords: n.a. Cc: kgillen, jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

4.4.5 release testing:
For images with a lot of metadata, we get exceptions when 'share' user views metadata panel.

E.g. for aryeh 1h.oib:

...
org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:202)
	at $Proxy84.loadChannelAcquisitionData_async(Unknown Source)
	at omero.api._IMetadataTie.loadChannelAcquisitionData_async(_IMetadataTie.java:106)
	at omero.api._IMetadataDisp.___loadChannelAcquisitionData(_IMetadataDisp.java:161)
	at omero.api._IMetadataDisp.__dispatch(_IMetadataDisp.java:486)
	at IceInternal.Incoming.invoke(Incoming.java:159)
	at Ice.ConnectionI.invokeAll(ConnectionI.java:2037)
	at Ice.ConnectionI.message(ConnectionI.java:972)
	at IceInternal.ThreadPool.run(ThreadPool.java:577)
	at IceInternal.ThreadPool.access$100(ThreadPool.java:12)
	at IceInternal.ThreadPool$EventHandlerThread.run(ThreadPool.java:971)

    serverExceptionClass = ome.conditions.SecurityViolation
    message = ome.model.acquisition.LightSettings:Id_9 not contained in share
}

OR for Beta Catenin.liff (test_images_good).

...
    at omero.api._IQueryTie.get_async(_IQueryTie.java:127)
    at omero.api._IQueryDisp.___get(_IQueryDisp.java:170)
    at omero.api._IQueryDisp.__dispatch(_IQueryDisp.java:516)
    at IceInternal.Incoming.invoke(Incoming.java:159)
    at Ice.ConnectionI.invokeAll(ConnectionI.java:2037)
    at Ice.ConnectionI.message(ConnectionI.java:972)
    at IceInternal.ThreadPool.run(ThreadPool.java:577)
    at IceInternal.ThreadPool.access$100(ThreadPool.java:12)
    at IceInternal.ThreadPool$EventHandlerThread.run(ThreadPool.java:971)

    serverExceptionClass = ome.conditions.SecurityViolation
    message = ome.model.acquisition.ObjectiveSettings:Id_292 not contained in share
}

Attachments (2)

Screen shot 2012-11-09 at 10.15.24.png (317.2 KB) - added by wmoore 7 years ago.
Full metadata
Screen shot 2012-11-09 at 10.14.28.png (172.6 KB) - added by wmoore 7 years ago.
Reduced metadata in share

Download all attachments as: .zip

Change History (15)

comment:2 Changed 7 years ago by wmoore

  • Status changed from new to accepted

comment:3 Changed 7 years ago by wmoore

  • Cc jmoore added

Since I'm not too familiar with how Share security settings work, I'm going to assume that this is handled on the server for other metadata (where supported).

I don't see any share-specific code for E.g. loading Channels.

As a temporary measure in web, I can disable the metadata that is not supported in shares.

comment:4 Changed 7 years ago by wmoore

  • Component changed from Web to Security
  • Owner changed from wmoore to jmoore

Metadata disabled for shared images in https://github.com/will-moore/openmicroscopy/commit/37c2428d09a0b0f92344759ed61edc3e34da1ad0 (https://github.com/openmicroscopy/openmicroscopy/pull/464).

Passing to Josh in the hope that the underlying issue can be addressed server-side...

Changed 7 years ago by wmoore

Full metadata

Changed 7 years ago by wmoore

Reduced metadata in share

comment:5 Changed 7 years ago by Will Moore <will@…>

(In [5e5f348a08006d4b13e487e89296e4233b74e8d0/ome.git] on branch develop) Shared images display limited metadata. See #9853

comment:6 Changed 7 years ago by jburel

  • Sprint changed from 2012-11-20 (2) to 2012-12-18 (3)

Moved from sprint 2012-11-20 (2)

comment:7 Changed 7 years ago by jburel

  • Sprint changed from 2012-12-18 (3) to 2013-01-15 (4)

Moved from sprint 2012-12-18 (3)

comment:8 Changed 7 years ago by jburel

  • Sprint changed from 2013-01-15 (4) to 2013-02-12 (5)

Moved from sprint 2013-01-15 (4)

comment:9 Changed 7 years ago by jburel

  • Sprint changed from 2013-02-12 (5) to 2013-03-12 (6))

Moved from sprint 2013-02-12 (5)

comment:10 Changed 7 years ago by jburel

  • Sprint changed from 2013-03-12 (6)) to 2013-04-09 (7))

Moved from sprint 2013-03-12 (6))

comment:11 Changed 7 years ago by jburel

  • Sprint changed from 2013-04-09 (7)) to 2013-05-07 (8)

Moved from sprint 2013-04-09 (7))

comment:12 Changed 6 years ago by jburel

  • Milestone changed from OMERO-4.4.x to OMERO-4.4.9
  • Sprint changed from 2013-05-07 (8) to Blocker 4.4.9 (1)

Moved from sprint 2013-05-07 (8)

comment:13 Changed 4 years ago by jamoore

  • Milestone changed from 5.x to Permissions
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.87859 sec.)

We're Hiring!