Feature #14201: [API] Accept "exclude_home_project" flag in groups#contents - Arvados

Actions

Copy link

Feature #14201

closed

[API] Accept "exclude_home_project" flag in groups#contents

Added by Tom Clegg over 5 years ago. Updated over 5 years ago.

Status:

Resolved

Priority:

Normal

Assigned To:

Peter Amstutz

Category:

API

Target version:

2018-10-03 Sprint

Story points:

1.0

Release:

Arvados 1.3

Release relationship:

Auto

Description

If a truthy exclude_home_project query parameter is given,

include only items that are not otherwise reachable by the current user via project tree -- i.e., items whose parents are not visible to the current user, or are users other than the current user, or are non-project groups (see #13146)
return a "bad request" error if parent/owner uuid is also given

Subtasks 1 (0 open — 1 closed)

Related issues

Actions

Copy link

Updated by Tom Clegg over 5 years ago

Description updated (diff)

Actions

Copy link

Updated by Tom Clegg over 5 years ago

Related to Idea #13146: [API] Endpoint to get projects shared with me added

Actions

Copy link

Updated by Tom Morris over 5 years ago

Target version set to To Be Groomed
Story points set to 1.0

Actions

Copy link

Updated by Tom Clegg over 5 years ago

Description updated (diff)

Actions

Copy link

Updated by Tom Clegg over 5 years ago

Description updated (diff)

Actions

Copy link

Updated by Tom Morris over 5 years ago

Target version changed from To Be Groomed to 2018-10-03 Sprint

Actions

Copy link

Updated by Lucas Di Pentima over 5 years ago

Assigned To set to Lucas Di Pentima

Actions

Copy link

Updated by Tom Clegg over 5 years ago

Subject changed from [API] Accept "shared with me" flag in groups#contents to [API] Accept "exclude_home_project" flag in groups#contents
Description updated (diff)

Actions

Copy link

Updated by Peter Amstutz over 5 years ago

Status changed from New to In Progress
Assigned To changed from Lucas Di Pentima to Peter Amstutz

Actions

Copy link

#10

Updated by Peter Amstutz over 5 years ago

14201-exclude-home-project @ 15f312dfec5cf19c2c71508b2ed7a9d65758eeb0

Added "exclude_home_project" to "contents" endpoint
Updated documentation
Added tests

https://ci.curoverse.com/view/Developer/job/developer-run-tests/909/

Actions

Copy link

#11

Updated by Lucas Di Pentima over 5 years ago

A couple of comments:

On file services/api/app/controllers/arvados/v1/groups_controller.rb, lines 115, 266: In case owners var contain lots of repeated uuids, would it be convenient to get the de-duplicated version before passing it to que query? (I’m not sure if Active Record already does this)
Should exclude_home() be a protected method?
Fuse tests seem to have halted on test_safer_realpath, don't know if it's related to this update.

Actions

Copy link

#12

Updated by Peter Amstutz over 5 years ago

Lucas Di Pentima wrote:

A couple of comments:

On file services/api/app/controllers/arvados/v1/groups_controller.rb, lines 115, 266: In case owners var contain lots of repeated uuids, would it be convenient to get the de-duplicated version before passing it to que query? (I’m not sure if Active Record already does this)

Now converts to Set() (which will deduplicate entries) and then back to Array.