User Story #2541 (closed)
Opened 14 years ago
Closed 12 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 12 years ago by jmoore
- Cc jburel added
comment:2 Changed 12 years ago by jburel
This is a larger problem. Search approach needs to be reviewed, so a story makes sense.
comment:3 Changed 12 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 12 years ago by jmoore <josh@…>
(In [89193d7d73e93a06f836ed702784f89257c37549/ome.git] on branch develop) Passing perm search test (See #2541)
comment:5 Changed 12 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 12 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 12 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 12 years ago by cxallan
- Resolution set to duplicate
- Status changed from new to closed
Will be covered by #2094.
Jean-Marie: you think it's ok to have this a larger story? Or is this something we should address for sprint6?