17119: bugfix: when "count: none" is specified, the group/contents
[arvados.git] / services / api / app / controllers / arvados / v1 / groups_controller.rb
1 # Copyright (C) The Arvados Authors. All rights reserved.
2 #
3 # SPDX-License-Identifier: AGPL-3.0
4
5 require "trashable"
6
7 class Arvados::V1::GroupsController < ApplicationController
8   include TrashableController
9
10   skip_before_action :find_object_by_uuid, only: :shared
11   skip_before_action :render_404_if_no_object, only: :shared
12
13   def self._index_requires_parameters
14     (super rescue {}).
15       merge({
16         include_trash: {
17           type: 'boolean', required: false, default: false, description: "Include items whose is_trashed attribute is true.",
18         },
19       })
20   end
21
22   def self._show_requires_parameters
23     (super rescue {}).
24       merge({
25         include_trash: {
26           type: 'boolean', required: false, default: false, description: "Show group/project even if its is_trashed attribute is true.",
27         },
28       })
29   end
30
31   def self._contents_requires_parameters
32     params = _index_requires_parameters.
33       merge({
34               uuid: {
35                 type: 'string', required: false, default: nil,
36               },
37               recursive: {
38                 type: 'boolean', required: false, default: false, description: 'Include contents from child groups recursively.',
39               },
40               include: {
41                 type: 'string', required: false, description: 'Include objects referred to by listed field in "included" (only owner_uuid).',
42               },
43               include_old_versions: {
44                 type: 'boolean', required: false, default: false, description: 'Include past collection versions.',
45               }
46             })
47     params.delete(:select)
48     params
49   end
50
51   def self._create_requires_parameters
52     super.merge(
53       {
54         async: {
55           required: false,
56           type: 'boolean',
57           location: 'query',
58           default: false,
59           description: 'defer permissions update',
60         }
61       }
62     )
63   end
64
65   def self._update_requires_parameters
66     super.merge(
67       {
68         async: {
69           required: false,
70           type: 'boolean',
71           location: 'query',
72           default: false,
73           description: 'defer permissions update',
74         }
75       }
76     )
77   end
78
79   def create
80     if params[:async]
81       @object = model_class.new(resource_attrs.merge({async_permissions_update: true}))
82       @object.save!
83       render_accepted
84     else
85       super
86     end
87   end
88
89   def update
90     if params[:async]
91       attrs_to_update = resource_attrs.reject { |k, v|
92         [:kind, :etag, :href].index k
93       }.merge({async_permissions_update: true})
94       @object.update_attributes!(attrs_to_update)
95       @object.save!
96       render_accepted
97     else
98       super
99     end
100   end
101
102   def render_404_if_no_object
103     if params[:action] == 'contents'
104       if !params[:uuid]
105         # OK!
106         @object = nil
107         true
108       elsif @object
109         # Project group
110         true
111       elsif (@object = User.where(uuid: params[:uuid]).first)
112         # "Home" pseudo-project
113         true
114       else
115         super
116       end
117     else
118       super
119     end
120   end
121
122   def contents
123     load_searchable_objects
124     list = {
125       :kind => "arvados#objectList",
126       :etag => "",
127       :self_link => "",
128       :offset => @offset,
129       :limit => @limit,
130       :items => @objects.as_api_response(nil)
131     }
132     if params[:count] != 'none'
133       list[:items_available] = @items_available
134     end
135     if @extra_included
136       list[:included] = @extra_included.as_api_response(nil, {select: @select})
137     end
138     send_json(list)
139   end
140
141   def shared
142     # The purpose of this endpoint is to return the toplevel set of
143     # groups which are *not* reachable through a direct ownership
144     # chain of projects starting from the current user account.  In
145     # other words, groups which to which access was granted via a
146     # permission link or chain of links.
147     #
148     # This also returns (in the "included" field) the objects that own
149     # those projects (users or non-project groups).
150     #
151     #
152     # The intended use of this endpoint is to support clients which
153     # wish to browse those projects which are visible to the user but
154     # are not part of the "home" project.
155
156     load_limit_offset_order_params
157     load_filters_param
158
159     @objects = exclude_home Group.readable_by(*@read_users), Group
160
161     apply_where_limit_order_params
162
163     if params["include"] == "owner_uuid"
164       owners = @objects.map(&:owner_uuid).to_set
165       @extra_included = []
166       [Group, User].each do |klass|
167         @extra_included += klass.readable_by(*@read_users).where(uuid: owners.to_a).to_a
168       end
169     end
170
171     index
172   end
173
174   def self._shared_requires_parameters
175     rp = self._index_requires_parameters
176     rp[:include] = { type: 'string', required: false }
177     rp
178   end
179
180   protected
181
182   def load_searchable_objects
183     all_objects = []
184     @items_available = 0
185
186     # Reload the orders param, this time without prefixing unqualified
187     # columns ("name" => "groups.name"). Here, unqualified orders
188     # apply to each table being searched, not "groups".
189     load_limit_offset_order_params(fill_table_names: false)
190
191     # Trick apply_where_limit_order_params into applying suitable
192     # per-table values. *_all are the real ones we'll apply to the
193     # aggregate set.
194     limit_all = @limit
195     offset_all = @offset
196     # save the orders from the current request as determined by load_param,
197     # but otherwise discard them because we're going to be getting objects
198     # from many models
199     request_orders = @orders.clone
200     @orders = []
201
202     request_filters = @filters
203
204     klasses = [Group,
205      Job, PipelineInstance, PipelineTemplate, ContainerRequest, Workflow,
206      Collection,
207      Human, Specimen, Trait]
208
209     table_names = Hash[klasses.collect { |k| [k, k.table_name] }]
210
211     disabled_methods = Rails.configuration.API.DisabledAPIs
212     avail_klasses = table_names.select{|k, t| !disabled_methods[t+'.index']}
213     klasses = avail_klasses.keys
214
215     request_filters.each do |col, op, val|
216       if col.index('.') && !table_names.values.include?(col.split('.', 2)[0])
217         raise ArgumentError.new("Invalid attribute '#{col}' in filter")
218       end
219     end
220
221     wanted_klasses = []
222     request_filters.each do |col,op,val|
223       if op == 'is_a'
224         (val.is_a?(Array) ? val : [val]).each do |type|
225           type = type.split('#')[-1]
226           type[0] = type[0].capitalize
227           wanted_klasses << type
228         end
229       end
230     end
231
232     filter_by_owner = {}
233     if @object
234       if params['recursive']
235         filter_by_owner[:owner_uuid] = [@object.uuid] + @object.descendant_project_uuids
236       else
237         filter_by_owner[:owner_uuid] = @object.uuid
238       end
239
240       if params['exclude_home_project']
241         raise ArgumentError.new "Cannot use 'exclude_home_project' with a parent object"
242       end
243     end
244
245     included_by_uuid = {}
246
247     seen_last_class = false
248     klasses.each do |klass|
249       @offset = 0 if seen_last_class  # reset offset for the new next type being processed
250
251       # if current klass is same as params['last_object_class'], mark that fact
252       seen_last_class = true if((params['count'].andand.==('none')) and
253                                 (params['last_object_class'].nil? or
254                                  params['last_object_class'].empty? or
255                                  params['last_object_class'] == klass.to_s))
256
257       # if klasses are specified, skip all other klass types
258       next if wanted_klasses.any? and !wanted_klasses.include?(klass.to_s)
259
260       # don't reprocess klass types that were already seen
261       next if params['count'] == 'none' and !seen_last_class
262
263       # don't process rest of object types if we already have needed number of objects
264       break if params['count'] == 'none' and all_objects.size >= limit_all
265
266       # If the currently requested orders specifically match the
267       # table_name for the current klass, apply that order.
268       # Otherwise, order by recency.
269       request_order =
270         request_orders.andand.find { |r| r =~ /^#{klass.table_name}\./i || r !~ /\./ } ||
271         klass.default_orders.join(", ")
272
273       @select = nil
274       where_conds = filter_by_owner
275       if klass == Collection
276         @select = klass.selectable_attributes - ["manifest_text", "unsigned_manifest_text"]
277       elsif klass == Group
278         where_conds = where_conds.merge(group_class: ["project","filter"])
279       end
280
281       @filters = request_filters.map do |col, op, val|
282         if !col.index('.')
283           [col, op, val]
284         elsif (col = col.split('.', 2))[0] == klass.table_name
285           [col[1], op, val]
286         else
287           nil
288         end
289       end.compact
290
291       @objects = klass.readable_by(*@read_users, {
292           :include_trash => params[:include_trash],
293           :include_old_versions => params[:include_old_versions]
294         }).order(request_order).where(where_conds)
295
296       if params['exclude_home_project']
297         @objects = exclude_home @objects, klass
298       end
299
300       klass_limit = limit_all - all_objects.count
301       @limit = klass_limit
302       apply_where_limit_order_params klass
303       klass_object_list = object_list(model_class: klass)
304       klass_items_available = klass_object_list[:items_available] || 0
305       @items_available += klass_items_available
306       @offset = [@offset - klass_items_available, 0].max
307       all_objects += klass_object_list[:items]
308
309       if klass_object_list[:limit] < klass_limit
310         # object_list() had to reduce @limit to comply with
311         # max_index_database_read. From now on, we'll do all queries
312         # with limit=0 and just accumulate items_available.
313         limit_all = all_objects.count
314       end
315
316       if params["include"] == "owner_uuid"
317         owners = klass_object_list[:items].map {|i| i[:owner_uuid]}.to_set
318         [Group, User].each do |ownerklass|
319           ownerklass.readable_by(*@read_users).where(uuid: owners.to_a).each do |ow|
320             included_by_uuid[ow.uuid] = ow
321           end
322         end
323       end
324     end
325
326     if params["include"]
327       @extra_included = included_by_uuid.values
328     end
329
330     @objects = all_objects
331     @limit = limit_all
332     @offset = offset_all
333   end
334
335   protected
336
337   def exclude_home objectlist, klass
338     # select records that are readable by current user AND
339     #   the owner_uuid is a user (but not the current user) OR
340     #   the owner_uuid is not readable by the current user
341     #   the owner_uuid is a group but group_class is not a project
342
343     read_parent_check = if current_user.is_admin
344                           ""
345                         else
346                           "NOT EXISTS(SELECT 1 FROM #{PERMISSION_VIEW} WHERE "+
347                             "user_uuid=(:user_uuid) AND target_uuid=#{klass.table_name}.owner_uuid AND perm_level >= 1) OR "
348                         end
349
350     objectlist.where("#{klass.table_name}.owner_uuid IN (SELECT users.uuid FROM users WHERE users.uuid != (:user_uuid)) OR "+
351                      read_parent_check+
352                      "EXISTS(SELECT 1 FROM groups as gp where gp.uuid=#{klass.table_name}.owner_uuid and gp.group_class != 'project')",
353                      user_uuid: current_user.uuid)
354   end
355
356 end