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"

User Story #2541 (closed)

Opened 10 years ago

Closed 8 years ago

Searches should take permissions into account

Reported by: jamoore Owned by:
Priority: major Milestone: Future
Component: Search Keywords: n.a.
Cc: jburel, cneves, wmoore, cxallan Story Points: n.a.
Sprint: n.a. Importance: n.a.
Total Remaining Time: n.a. Estimated Remaining Time: n.a.

Description

Currently, searching for unloaded objects returns items which are possibly not readable by the user.

Reported by John Hufnagle.

Change History (8)

comment:1 Changed 9 years ago by jmoore

  • Cc jburel added

Jean-Marie: you think it's ok to have this a larger story? Or is this something we should address for sprint6?

comment:2 Changed 9 years ago by jburel

This is a larger problem. Search approach needs to be reviewed, so a story makes sense.

comment:3 Changed 9 years ago by jmoore

  • Summary changed from Bug: Searches should take permissions into account to Searches should take permissions into account
  • Type changed from Task to User Story

The simplest case (of a private group) is no longer happening in 4.3+. However, this still holds true in general, that the permissions system should be tightly integrated into the index (perhaps one per group for better throughput?). Changing to an RFE story.

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

(In [89193d7d73e93a06f836ed702784f89257c37549/ome.git] on branch develop) Passing perm search test (See #2541)

comment:5 Changed 9 years ago by jmoore

  • Cc cneves-x wmoore cxallan added
  • Milestone changed from Unscheduled to OMERO-Beta4.4
  • Type changed from User Story to Task

Note: with the modifications for #3529 (omero.group) it should now be possible to run a search over all groups safely.

comment:6 Changed 9 years ago by jmoore

  • Priority changed from minor to major
  • Type changed from Task to User Story

Turning into a story, but still see #3527 (call context)

comment:7 Changed 8 years ago by jmoore

  • Milestone changed from OMERO-Beta4.4 to Future

Pushing to Future since this should probably coincide with other search testing/reworking.

comment:8 Changed 8 years ago by cxallan

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

Will be covered by #2094.

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

We're Hiring!