X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/5620a47e20e004188e0f1d0245ae9e916575e806..417ff385d9e6bf111ebb1d5889c2081977dfe955:/services/api/app/models/arvados_model.rb diff --git a/services/api/app/models/arvados_model.rb b/services/api/app/models/arvados_model.rb index 09b61413b4..ed19636fcb 100644 --- a/services/api/app/models/arvados_model.rb +++ b/services/api/app/models/arvados_model.rb @@ -1,3 +1,4 @@ +require 'assign_uuid' class ArvadosModel < ActiveRecord::Base self.abstract_class = true @@ -9,19 +10,47 @@ class ArvadosModel < ActiveRecord::Base attr_protected :modified_at before_create :ensure_permission_to_create before_update :ensure_permission_to_update - before_create :update_modified_by_fields - before_update :maybe_update_modified_by_fields + before_destroy :ensure_permission_to_destroy + + before_validation :maybe_update_modified_by_fields validate :ensure_serialized_attribute_type + validate :normalize_collection_uuids + validate :ensure_valid_uuids has_many :permissions, :foreign_key => :head_uuid, :class_name => 'Link', :primary_key => :uuid, :conditions => "link_class = 'permission'" class PermissionDeniedError < StandardError + def http_status + 403 + end + end + + class UnauthorizedError < StandardError + def http_status + 401 + end end def self.kind_class(kind) kind.match(/^arvados\#(.+?)(_list|List)?$/)[1].pluralize.classify.constantize rescue nil end + def href + "#{current_api_base}/#{self.class.to_s.pluralize.underscore}/#{self.uuid}" + end + + def self.searchable_columns + self.columns.collect do |col| + if [:string, :text, :datetime, :integer].index(col.type) && col.name != 'owner_uuid' + col.name + end + end.compact + end + + def self.attribute_column attr + self.columns.select { |col| col.name == attr.to_s }.first + end + def eager_load_associations self.class.columns.each do |col| re = col.name.match /^(.*)_kind$/ @@ -35,6 +64,21 @@ class ArvadosModel < ActiveRecord::Base end end + def self.readable_by user + uuid_list = [user.uuid, *user.groups_i_can(:read)] + sanitized_uuid_list = uuid_list. + collect { |uuid| sanitize(uuid) }.join(', ') + or_references_me = '' + if self == Link and user + or_references_me = "OR (#{table_name}.link_class in (#{sanitize 'permission'}, #{sanitize 'resources'}) AND #{sanitize user.uuid} IN (#{table_name}.head_uuid, #{table_name}.tail_uuid))" + end + joins("LEFT JOIN links permissions ON permissions.head_uuid in (#{table_name}.owner_uuid, #{table_name}.uuid) AND permissions.tail_uuid in (#{sanitized_uuid_list}) AND permissions.link_class='permission'"). + where("?=? OR #{table_name}.owner_uuid in (?) OR #{table_name}.uuid=? OR permissions.head_uuid IS NOT NULL #{or_references_me}", + true, user.is_admin, + uuid_list, + user.uuid) + end + protected def ensure_permission_to_create @@ -58,11 +102,11 @@ class ArvadosModel < ActiveRecord::Base logger.warn "Inactive user #{current_user.uuid} tried to update #{self.class.to_s} #{self.uuid_was}" return false end + return true if current_user.is_admin if self.uuid_changed? logger.warn "User #{current_user.uuid} tried to change uuid of #{self.class.to_s} #{self.uuid_was} to #{self.uuid}" return false end - return true if current_user.is_admin if self.owner_uuid_changed? if current_user.uuid == self.owner_uuid or current_user.can? write: self.owner_uuid @@ -83,13 +127,21 @@ class ArvadosModel < ActiveRecord::Base end end + def ensure_permission_to_destroy + raise PermissionDeniedError unless permission_to_destroy + end + + def permission_to_destroy + permission_to_update + end + def maybe_update_modified_by_fields - update_modified_by_fields if self.changed? + update_modified_by_fields if self.changed? or self.new_record? end def update_modified_by_fields self.created_at ||= Time.now - self.owner_uuid ||= current_default_owner + self.owner_uuid ||= current_default_owner if self.respond_to? :owner_uuid= self.modified_at = Time.now self.modified_by_user_uuid = current_user ? current_user.uuid : nil self.modified_by_client_uuid = current_api_client ? current_api_client.uuid : nil @@ -110,4 +162,67 @@ class ArvadosModel < ActiveRecord::Base end end end + + def foreign_key_attributes + attributes.keys.select { |a| a.match /_uuid$/ } + end + + def normalize_collection_uuids + foreign_key_attributes.each do |attr| + attr_value = send attr + if attr_value.is_a? String and + attr_value.match /^[0-9a-f]{32,}(\+[@\w]+)*$/ + begin + send "#{attr}=", Collection.normalize_uuid(attr_value) + rescue + # TODO: abort instead of silently accepting unnormalizable value? + end + end + end + end + + @@UUID_REGEX = /^[0-9a-z]{5}-([0-9a-z]{5})-[0-9a-z]{15}$/ + + def ensure_valid_uuids + specials = [system_user_uuid, 'd41d8cd98f00b204e9800998ecf8427e+0'] + + foreign_key_attributes.each do |attr| + next if attr == "modified_by_client_uuid" + begin + attr_value = send attr + r = ArvadosModel::resource_class_for_uuid attr_value if attr_value + if r and r.readable_by(current_user).where(uuid: attr_value).count == 0 and not specials.include? attr_value + errors.add(attr, "'#{attr_value}' not found") + end + rescue Exception => e + errors.add(attr, "'#{attr_value}' error #{e}") + end + end + end + + def self.resource_class_for_uuid(uuid) + if uuid.is_a? ArvadosModel + return uuid.class + end + unless uuid.is_a? String + return nil + end + if uuid.match /^[0-9a-f]{32}(\+[^,]+)*(,[0-9a-f]{32}(\+[^,]+)*)*$/ + return Collection + end + resource_class = nil + + Rails.application.eager_load! + uuid.match @@UUID_REGEX do |re| + ActiveRecord::Base.descendants.reject(&:abstract_class?).each do |k| + if k.respond_to?(:uuid_prefix) + if k.uuid_prefix == re[1] + return k + end + end + end + end + nil + end + end