]> git.arvados.org - arvados.git/blob - services/api/app/models/collection.rb
Merge branch '22466-output-glob-fix' refs #22466
[arvados.git] / services / api / app / models / collection.rb
1 # Copyright (C) The Arvados Authors. All rights reserved.
2 #
3 # SPDX-License-Identifier: AGPL-3.0
4
5 require 'arvados/keep'
6 require 'trashable'
7
8 class Collection < ArvadosModel
9   extend CurrentApiClient
10   extend DbCurrentTime
11   include HasUuid
12   include KindAndEtag
13   include CommonApiTemplate
14   include Trashable
15
16   # Posgresql JSONB columns should NOT be declared as serialized, Rails 5
17   # already know how to properly treat them.
18   attribute :properties, :jsonbHash, default: {}
19   attribute :storage_classes_desired, :jsonbArray, default: lambda { Rails.configuration.DefaultStorageClasses }
20   attribute :storage_classes_confirmed, :jsonbArray, default: []
21
22   before_validation :default_empty_manifest
23   before_validation :default_storage_classes, on: :create
24   before_validation :managed_properties, on: :create
25   before_validation :check_encoding
26   before_validation :check_manifest_validity
27   before_validation :check_signatures
28   before_validation :strip_signatures_and_update_replication_confirmed
29   before_validation :name_null_if_empty
30   validate :ensure_filesystem_compatible_name
31   validate :ensure_pdh_matches_manifest_text
32   validate :ensure_storage_classes_desired_is_not_empty
33   validate :ensure_storage_classes_contain_non_empty_strings
34   validate :versioning_metadata_updates, on: :update
35   validate :past_versions_cannot_be_updated, on: :update
36   validate :protected_managed_properties_updates, on: :update
37   after_validation :set_file_count_and_total_size
38   before_save :set_file_names
39   around_update :manage_versioning, unless: :is_past_version?
40
41   api_accessible :user, extend: :common do |t|
42     t.add lambda { |x| x.name || "" }, as: :name
43     t.add :description
44     t.add :properties
45     t.add :portable_data_hash
46     t.add :manifest_text, as: :unsigned_manifest_text
47     t.add :manifest_text, as: :manifest_text
48     t.add :replication_desired
49     t.add :replication_confirmed
50     t.add :replication_confirmed_at
51     t.add :storage_classes_desired
52     t.add :storage_classes_confirmed
53     t.add :storage_classes_confirmed_at
54     t.add :delete_at
55     t.add :trash_at
56     t.add :is_trashed
57     t.add :version
58     t.add :current_version_uuid
59     t.add :preserve_version
60     t.add :file_count
61     t.add :file_size_total
62   end
63
64   UNLOGGED_CHANGES = ['preserve_version', 'updated_at']
65
66   after_initialize do
67     @signatures_checked = false
68     @computed_pdh_for_manifest_text = false
69   end
70
71   def self.attributes_required_columns
72     super.merge(
73                 # If we don't list unsigned_manifest_text explicitly,
74                 # the params[:select] code gets confused by the way we
75                 # expose manifest_text as unsigned_manifest_text in
76                 # the API response, and never let clients select the
77                 # unsigned_manifest_text column.
78                 'unsigned_manifest_text' => ['manifest_text'],
79                 'name' => ['name'],
80                 )
81   end
82
83   def self.ignored_select_attributes
84     super + ["updated_at", "file_names"]
85   end
86
87   FILE_TOKEN = /^[[:digit:]]+:[[:digit:]]+:/
88   def check_signatures
89     throw(:abort) if self.manifest_text.nil?
90
91     return true if current_user.andand.is_admin
92
93     # Provided the manifest_text hasn't changed materially since an
94     # earlier validation, it's safe to pass this validation on
95     # subsequent passes without checking any signatures. This is
96     # important because the signatures have probably been stripped off
97     # by the time we get to a second validation pass!
98     if @signatures_checked && @signatures_checked == computed_pdh
99       return true
100     end
101
102     if self.manifest_text_changed?
103       # Check permissions on the collection manifest.
104       # If any signature cannot be verified, raise PermissionDeniedError
105       # which will return 403 Permission denied to the client.
106       api_token = Thread.current[:token]
107       signing_opts = {
108         api_token: api_token,
109         now: @validation_timestamp.to_i,
110       }
111       self.manifest_text.each_line do |entry|
112         entry.split.each do |tok|
113           if tok == '.' or tok.starts_with? './'
114             # Stream name token.
115           elsif tok =~ FILE_TOKEN
116             # This is a filename token, not a blob locator. Note that we
117             # keep checking tokens after this, even though manifest
118             # format dictates that all subsequent tokens will also be
119             # filenames. Safety first!
120           elsif Blob.verify_signature tok, signing_opts
121             # OK.
122           elsif Keep::Locator.parse(tok).andand.signature
123             # Signature provided, but verify_signature did not like it.
124             logger.warn "Invalid signature on locator #{tok}"
125             raise ArvadosModel::PermissionDeniedError
126           elsif !Rails.configuration.Collections.BlobSigning
127             # No signature provided, but we are running in insecure mode.
128             logger.debug "Missing signature on locator #{tok} ignored"
129           elsif Blob.new(tok).empty?
130             # No signature provided -- but no data to protect, either.
131           else
132             logger.warn "Missing signature on locator #{tok}"
133             raise ArvadosModel::PermissionDeniedError
134           end
135         end
136       end
137     end
138     @signatures_checked = computed_pdh
139   end
140
141   def strip_signatures_and_update_replication_confirmed
142     if self.manifest_text_changed?
143       in_old_manifest = {}
144       # manifest_text_was could be nil when dealing with a freshly created snapshot,
145       # so we skip this case because there was no real manifest change. (Bug #18005)
146       if (not self.replication_confirmed.nil?) and (not self.manifest_text_was.nil?)
147         self.class.each_manifest_locator(manifest_text_was) do |match|
148           in_old_manifest[match[1]] = true
149         end
150       end
151
152       stripped_manifest = self.class.munge_manifest_locators(manifest_text) do |match|
153         if not self.replication_confirmed.nil? and not in_old_manifest[match[1]]
154           # If the new manifest_text contains locators whose hashes
155           # weren't in the old manifest_text, storage replication is no
156           # longer confirmed.
157           self.replication_confirmed_at = nil
158           self.replication_confirmed = nil
159         end
160
161         # Return the locator with all permission signatures removed,
162         # but otherwise intact.
163         match[0].gsub(/\+A[^+]*/, '')
164       end
165
166       if @computed_pdh_for_manifest_text == manifest_text
167         # If the cached PDH was valid before stripping, it is still
168         # valid after stripping.
169         @computed_pdh_for_manifest_text = stripped_manifest.dup
170       end
171
172       self[:manifest_text] = stripped_manifest
173     end
174     true
175   end
176
177   def ensure_pdh_matches_manifest_text
178     if not manifest_text_changed? and not portable_data_hash_changed?
179       true
180     elsif portable_data_hash.nil? or not portable_data_hash_changed?
181       self.portable_data_hash = computed_pdh
182     elsif portable_data_hash !~ Keep::Locator::LOCATOR_REGEXP
183       errors.add(:portable_data_hash, "is not a valid locator")
184       false
185     elsif portable_data_hash[0..31] != computed_pdh[0..31]
186       errors.add(:portable_data_hash,
187                  "'#{portable_data_hash}' does not match computed hash '#{computed_pdh}'")
188       false
189     else
190       # Ignore the client-provided size part: always store
191       # computed_pdh in the database.
192       self.portable_data_hash = computed_pdh
193     end
194   end
195
196   def name_null_if_empty
197     if name == ""
198       self.name = nil
199     end
200   end
201
202   def set_file_names
203     if self.manifest_text_changed?
204       self.file_names = manifest_files
205     end
206     true
207   end
208
209   def set_file_count_and_total_size
210     # Only update the file stats if the manifest changed
211     if self.manifest_text_changed?
212       m = Keep::Manifest.new(self.manifest_text)
213       self.file_size_total = m.files_size
214       self.file_count = m.files_count
215     # If the manifest didn't change but the attributes did, ignore the changes
216     elsif self.file_count_changed? || self.file_size_total_changed?
217       self.file_count = self.file_count_was
218       self.file_size_total = self.file_size_total_was
219     end
220     true
221   end
222
223   def manifest_files
224     return '' if !self.manifest_text
225
226     done = {}
227     names = ''
228     self.manifest_text.scan(/ \d+:\d+:(\S+)/) do |name|
229       next if done[name]
230       done[name] = true
231       names << name.first.gsub('\040',' ') + "\n"
232     end
233     self.manifest_text.scan(/^\.\/(\S+)/m) do |stream_name|
234       next if done[stream_name]
235       done[stream_name] = true
236       names << stream_name.first.gsub('\040',' ') + "\n"
237     end
238     names
239   end
240
241   def default_empty_manifest
242     self.manifest_text ||= ''
243   end
244
245   def skip_uuid_existence_check
246     # Avoid checking the existence of current_version_uuid, as it's
247     # assigned on creation of a new 'current version' collection, so
248     # the collection's UUID only lives on memory when the validation check
249     # is performed.
250     ['current_version_uuid']
251   end
252
253   def manage_versioning
254     should_preserve_version = should_preserve_version? # Time sensitive, cache value
255     return(yield) unless (should_preserve_version || syncable_updates.any?)
256
257     # Put aside the changes because with_lock does a record reload
258     changes = self.changes
259     snapshot = nil
260     restore_attributes
261     with_lock do
262       # Copy the original state to save it as old version
263       if should_preserve_version
264         snapshot = self.dup
265         snapshot.uuid = nil # Reset UUID so it's created as a new record
266         snapshot.created_at = self.created_at
267         snapshot.modified_at = self.modified_at_was
268       end
269
270       # Restore requested changes on the current version
271       changes.keys.each do |attr|
272         if attr == 'preserve_version' && changes[attr].last == false && !should_preserve_version
273           next # Ignore false assignment, once true it'll be true until next version
274         end
275         self.attributes = {attr => changes[attr].last}
276         if attr == 'uuid'
277           # Also update the current version reference
278           self.attributes = {'current_version_uuid' => changes[attr].last}
279         end
280       end
281
282       if should_preserve_version
283         self.version += 1
284       end
285
286       yield
287
288       sync_past_versions if syncable_updates.any?
289       if snapshot
290         snapshot.attributes = self.syncable_updates
291         leave_modified_by_user_alone do
292           leave_modified_at_alone do
293             act_as_system_user do
294               snapshot.save
295             end
296           end
297         end
298       end
299     end
300   end
301
302   def maybe_update_modified_by_fields
303     if !(self.changes.keys - ['updated_at', 'preserve_version']).empty?
304       super
305     end
306   end
307
308   def syncable_updates
309     updates = {}
310     if self.changes.any?
311       changes = self.changes
312     else
313       # If called after save...
314       changes = self.saved_changes
315     end
316     (syncable_attrs & changes.keys).each do |attr|
317       if attr == 'uuid'
318         # Point old versions to current version's new UUID
319         updates['current_version_uuid'] = changes[attr].last
320       else
321         updates[attr] = changes[attr].last
322       end
323     end
324     return updates
325   end
326
327   def sync_past_versions
328     Collection.where('current_version_uuid = ? AND uuid != ?', self.uuid_before_last_save, self.uuid_before_last_save).update_all self.syncable_updates
329   end
330
331   def versionable_updates?(attrs)
332     (['manifest_text', 'description', 'properties', 'name'] & attrs).any?
333   end
334
335   def syncable_attrs
336     ['uuid', 'owner_uuid', 'delete_at', 'trash_at', 'is_trashed', 'replication_desired', 'storage_classes_desired']
337   end
338
339   def is_past_version?
340     # Check for the '_was' values just in case the update operation
341     # includes a change on current_version_uuid or uuid.
342     !(new_record? || self.current_version_uuid_was == self.uuid_was)
343   end
344
345   def should_preserve_version?
346     return false unless (Rails.configuration.Collections.CollectionVersioning && versionable_updates?(self.changes.keys))
347
348     return false if self.is_trashed
349
350     idle_threshold = Rails.configuration.Collections.PreserveVersionIfIdle
351     if !self.preserve_version_was &&
352       !self.preserve_version &&
353       (idle_threshold < 0 ||
354         (idle_threshold > 0 && self.modified_at_was > db_current_time-idle_threshold.seconds))
355       return false
356     end
357     return true
358   end
359
360   def check_encoding
361     if !(manifest_text.encoding.name == 'UTF-8' and manifest_text.valid_encoding?)
362       begin
363         # If Ruby thinks the encoding is something else, like 7-bit
364         # ASCII, but its stored bytes are equal to the (valid) UTF-8
365         # encoding of the same string, we declare it to be a UTF-8
366         # string.
367         utf8 = manifest_text
368         utf8.force_encoding Encoding::UTF_8
369         if utf8.valid_encoding? and utf8 == manifest_text.encode(Encoding::UTF_8)
370           self.manifest_text = utf8
371           return true
372         end
373       rescue
374       end
375       errors.add :manifest_text, "must use UTF-8 encoding"
376       throw(:abort)
377     end
378   end
379
380   def check_manifest_validity
381     begin
382       Keep::Manifest.validate! manifest_text
383       true
384     rescue ArgumentError => e
385       errors.add :manifest_text, e.message
386       throw(:abort)
387     end
388   end
389
390   def signed_manifest_text_only_for_tests
391     if !has_attribute? :manifest_text
392       return nil
393     elsif is_trashed
394       return manifest_text
395     else
396       token = Thread.current[:token]
397       exp = [db_current_time.to_i + Rails.configuration.Collections.BlobSigningTTL.to_i,
398              trash_at].compact.map(&:to_i).min
399       self.class.sign_manifest_only_for_tests manifest_text, token, exp
400     end
401   end
402
403   def self.sign_manifest_only_for_tests manifest, token, exp=nil
404     if exp.nil?
405       exp = db_current_time.to_i + Rails.configuration.Collections.BlobSigningTTL.to_i
406     end
407     signing_opts = {
408       api_token: token,
409       expire: exp,
410     }
411     m = munge_manifest_locators(manifest) do |match|
412       Blob.sign_locator(match[0], signing_opts)
413     end
414     return m
415   end
416
417   def self.munge_manifest_locators manifest
418     # Given a manifest text and a block, yield the regexp MatchData
419     # for each locator. Return a new manifest in which each locator
420     # has been replaced by the block's return value.
421     return nil if !manifest
422     return '' if manifest == ''
423
424     new_lines = []
425     manifest.each_line do |line|
426       line.rstrip!
427       new_words = []
428       line.split(' ').each do |word|
429         if new_words.empty?
430           new_words << word
431         elsif match = Keep::Locator::LOCATOR_REGEXP.match(word)
432           new_words << yield(match)
433         else
434           new_words << word
435         end
436       end
437       new_lines << new_words.join(' ')
438     end
439     new_lines.join("\n") + "\n"
440   end
441
442   def self.each_manifest_locator manifest
443     # Given a manifest text and a block, yield the regexp match object
444     # for each locator.
445     manifest.each_line do |line|
446       # line will have a trailing newline, but the last token is never
447       # a locator, so it's harmless here.
448       line.split(' ').each do |word|
449         if match = Keep::Locator::LOCATOR_REGEXP.match(word)
450           yield(match)
451         end
452       end
453     end
454   end
455
456   def self.normalize_uuid uuid
457     hash_part = nil
458     size_part = nil
459     uuid.split('+').each do |token|
460       if token.match(/^[0-9a-f]{32,}$/)
461         raise "uuid #{uuid} has multiple hash parts" if hash_part
462         hash_part = token
463       elsif token.match(/^\d+$/)
464         raise "uuid #{uuid} has multiple size parts" if size_part
465         size_part = token
466       end
467     end
468     raise "uuid #{uuid} has no hash part" if !hash_part
469     [hash_part, size_part].compact.join '+'
470   end
471
472   def self.get_compatible_images(readers, pattern, collections)
473     if collections.empty?
474       return []
475     end
476
477     migrations = Hash[
478       Link.where('tail_uuid in (?) AND link_class=? AND links.owner_uuid=?',
479                  collections.map(&:portable_data_hash),
480                  'docker_image_migration',
481                  system_user_uuid).
482       order('links.created_at asc').
483       map { |l|
484         [l.tail_uuid, l.head_uuid]
485       }]
486
487     migrated_collections = Hash[
488       Collection.readable_by(*readers).
489       where('portable_data_hash in (?)', migrations.values).
490       map { |c|
491         [c.portable_data_hash, c]
492       }]
493
494     collections.map { |c|
495       # Check if the listed image is compatible first, if not, then try the
496       # migration link.
497       manifest = Keep::Manifest.new(c.manifest_text)
498       if manifest.exact_file_count?(1) and manifest.files[0][1] =~ pattern
499         c
500       elsif m = migrated_collections[migrations[c.portable_data_hash]]
501         manifest = Keep::Manifest.new(m.manifest_text)
502         if manifest.exact_file_count?(1) and manifest.files[0][1] =~ pattern
503           m
504         end
505       end
506     }.compact
507   end
508
509   # Resolve a Docker repo+tag, hash, or collection PDH to an array of
510   # Collection objects, sorted by timestamp starting with the most recent
511   # match.
512   #
513   # If filter_compatible_format is true (the default), only return image
514   # collections which are support by the installation as indicated by
515   # Rails.configuration.Containers.SupportedDockerImageFormats.  Will follow
516   # 'docker_image_migration' links if search_term resolves to an incompatible
517   # image, but an equivalent compatible image is available.
518   def self.find_all_for_docker_image(search_term, search_tag=nil, readers=nil, filter_compatible_format: true)
519     readers ||= [Thread.current[:user]]
520     base_search = Link.
521       readable_by(*readers).
522       readable_by(*readers, table_name: "collections").
523       joins("JOIN collections ON links.head_uuid = collections.uuid").
524       order("links.created_at DESC")
525
526     docker_image_formats = Rails.configuration.Containers.SupportedDockerImageFormats.keys.map(&:to_s)
527
528     if (docker_image_formats.include? 'v1' and
529         docker_image_formats.include? 'v2') or filter_compatible_format == false
530       pattern = /^(sha256:)?[0-9A-Fa-f]{64}\.tar$/
531     elsif docker_image_formats.include? 'v2'
532       pattern = /^(sha256:)[0-9A-Fa-f]{64}\.tar$/
533     elsif docker_image_formats.include? 'v1'
534       pattern = /^[0-9A-Fa-f]{64}\.tar$/
535     else
536       raise "Unrecognized configuration for docker_image_formats #{docker_image_formats}"
537     end
538
539     # If the search term is a Collection locator that contains one file
540     # that looks like a Docker image, return it.
541     if loc = Keep::Locator.parse(search_term)
542       loc.strip_hints!
543       coll_match = readable_by(*readers).where(portable_data_hash: loc.to_s).limit(1)
544       rc = Rails.configuration.RemoteClusters.select{ |k|
545         k != :"*" && k != Rails.configuration.ClusterID}
546       if coll_match.any? or rc.length == 0
547         return get_compatible_images(readers, pattern, coll_match)
548       else
549         # Allow bare pdh that doesn't exist in the local database so
550         # that federated container requests which refer to remotely
551         # stored containers will validate.
552         return [Collection.new(portable_data_hash: loc.to_s)]
553       end
554     end
555
556     if search_tag.nil? and (n = search_term.index(":"))
557       search_tag = search_term[n+1..-1]
558       search_term = search_term[0..n-1]
559     end
560
561     # Find Collections with matching Docker image repository+tag pairs.
562     matches = base_search.
563       where(link_class: "docker_image_repo+tag",
564             name: "#{search_term}:#{search_tag || 'latest'}")
565
566     # If that didn't work, find Collections with matching Docker image hashes.
567     if matches.empty?
568       matches = base_search.
569         where("link_class = ? and links.name LIKE ?",
570               "docker_image_hash", "#{search_term}%")
571     end
572
573     # Generate an order key for each result.  We want to order the results
574     # so that anything with an image timestamp is considered more recent than
575     # anything without; then we use the link's created_at as a tiebreaker.
576     uuid_timestamps = {}
577     matches.each do |link|
578       uuid_timestamps[link.head_uuid] = [(-link.properties["image_timestamp"].to_datetime.to_i rescue 0),
579        -link.created_at.to_i]
580      end
581
582     sorted = Collection.where('uuid in (?)', uuid_timestamps.keys).sort_by { |c|
583       uuid_timestamps[c.uuid]
584     }
585     compatible = get_compatible_images(readers, pattern, sorted)
586     if sorted.length > 0 and compatible.empty?
587       raise ArvadosModel::UnresolvableContainerError.new "Matching Docker image is incompatible with 'docker_image_formats' configuration."
588     end
589     compatible
590   end
591
592   def self.for_latest_docker_image(search_term, search_tag=nil, readers=nil)
593     find_all_for_docker_image(search_term, search_tag, readers).first
594   end
595
596   def self.searchable_columns operator
597     super - ["manifest_text"]
598   end
599
600   def self.full_text_searchable_columns
601     super - ["manifest_text", "storage_classes_desired", "storage_classes_confirmed", "current_version_uuid"]
602   end
603
604   protected
605
606   # Although the defaults for these columns is already set up on the schema,
607   # collection creation from an API client seems to ignore them, making the
608   # validation on empty desired storage classes return an error.
609   def default_storage_classes
610     if self.storage_classes_desired.nil? || self.storage_classes_desired.empty?
611       self.storage_classes_desired = Rails.configuration.DefaultStorageClasses
612     end
613     self.storage_classes_confirmed ||= []
614   end
615
616   # Sets managed properties at creation time
617   def managed_properties
618     managed_props = Rails.configuration.Collections.ManagedProperties.with_indifferent_access
619     if managed_props.empty?
620       return
621     end
622     (managed_props.keys - self.properties.keys).each do |key|
623       if managed_props[key]['Function'] == 'original_owner'
624         self.properties[key] = self.user_owner_uuid
625       elsif managed_props[key]['Value']
626         self.properties[key] = managed_props[key]['Value']
627       else
628         logger.warn "Unidentified default property definition '#{key}': #{managed_props[key].inspect}"
629       end
630     end
631   end
632
633   def portable_manifest_text
634     self.class.munge_manifest_locators(manifest_text) do |match|
635       if match[2] # size
636         match[1] + match[2]
637       else
638         match[1]
639       end
640     end
641   end
642
643   def compute_pdh
644     portable_manifest = portable_manifest_text
645     (Digest::MD5.hexdigest(portable_manifest) +
646      '+' +
647      portable_manifest.bytesize.to_s)
648   end
649
650   def computed_pdh
651     if @computed_pdh_for_manifest_text == manifest_text
652       return @computed_pdh
653     end
654     @computed_pdh = compute_pdh
655     @computed_pdh_for_manifest_text = manifest_text.dup
656     @computed_pdh
657   end
658
659   def ensure_permission_to_save
660     if (not current_user.andand.is_admin)
661       if (replication_confirmed_at_changed? or replication_confirmed_changed?) and
662         not (replication_confirmed_at.nil? and replication_confirmed.nil?)
663         raise ArvadosModel::PermissionDeniedError.new("replication_confirmed and replication_confirmed_at attributes cannot be changed, except by setting both to nil")
664       end
665       if (storage_classes_confirmed_changed? or storage_classes_confirmed_at_changed?) and
666         not (storage_classes_confirmed == [] and storage_classes_confirmed_at.nil?)
667         raise ArvadosModel::PermissionDeniedError.new("storage_classes_confirmed and storage_classes_confirmed_at attributes cannot be changed, except by setting them to [] and nil respectively")
668       end
669     end
670     super
671   end
672
673   def ensure_storage_classes_desired_is_not_empty
674     if self.storage_classes_desired.empty?
675       raise ArvadosModel::InvalidStateTransitionError.new("storage_classes_desired shouldn't be empty")
676     end
677   end
678
679   def ensure_storage_classes_contain_non_empty_strings
680     (self.storage_classes_desired + self.storage_classes_confirmed).each do |c|
681       if !c.is_a?(String) || c == ''
682         raise ArvadosModel::InvalidStateTransitionError.new("storage classes should only be non-empty strings")
683       end
684     end
685   end
686
687   def past_versions_cannot_be_updated
688     if is_past_version?
689       errors.add(:base, "past versions cannot be updated")
690       false
691     end
692   end
693
694   def protected_managed_properties_updates
695     managed_properties = Rails.configuration.Collections.ManagedProperties.with_indifferent_access
696     if managed_properties.empty? || !properties_changed? || current_user.is_admin
697       return true
698     end
699     protected_props = managed_properties.keys.select do |p|
700       Rails.configuration.Collections.ManagedProperties[p]['Protected']
701     end
702     # Pre-existent protected properties can't be updated
703     invalid_updates = properties_was.keys.select{|p| properties_was[p] != properties[p]} & protected_props
704     if !invalid_updates.empty?
705       invalid_updates.each do |p|
706         errors.add("protected property cannot be updated:", p)
707       end
708       raise PermissionDeniedError.new
709     end
710     true
711   end
712
713   def versioning_metadata_updates
714     valid = true
715     if !is_past_version? && current_version_uuid_changed?
716       errors.add(:current_version_uuid, "cannot be updated")
717       valid = false
718     end
719     if version_changed?
720       errors.add(:version, "cannot be updated")
721       valid = false
722     end
723     valid
724   end
725
726   def assign_uuid
727     super
728     self.current_version_uuid ||= self.uuid
729     true
730   end
731
732   def log_update
733     super unless (saved_changes.keys - UNLOGGED_CHANGES).empty?
734   end
735 end