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 #9592 (new)

Opened 12 years ago

Last modified 8 years ago

BUG: SharedResources.openTable with group -1 fails

Reported by: cneves Owned by: spli
Priority: major Milestone: Permissions
Component: General Version: 4.4.8
Keywords: n.a. Cc: jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

In a groupread group, user in group (not owner nor admin) trying to openTable will fail.
Same user but with the proper group in ctx will work correctly

Change History (6)

comment:2 Changed 11 years ago by jmoore

Carlos: what group would you propose creating the OriginalFile in?

comment:3 Changed 11 years ago by spli

  • Cc spli added
  • Priority changed from minor to major
  • Version set to 4.4.8

@jamoore It's opening an existing table, not creating one.

comment:4 Changed 10 years ago by jamoore

  • Cc jamoore added; spli removed
  • Owner changed from jamoore to spli

Is this covered by your recent fixes?

comment:5 Changed 10 years ago by spli

No.

https://github.com/manics/openmicroscopy/commit/8ecb7503056bcf7aa3327d56bb05f3819158be26

            # Same thing, but with group -1 in ctx
            gatewaywrapper.gateway.SERVICE_OPTS.setOmeroGroup(-1)
            table = sr.openTable(file_annotation._obj.file, gatewaywrapper.gateway.SERVICE_OPTS)
>           assert table != None
E           assert None != None

test/gatewaytest/test_services.py:165: AssertionError

comment:6 Changed 8 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.81289 sec.)

We're Hiring!