[ARVADOS] updated: 1.2.0-257-ge92125d70

Git user git at public.curoverse.com
Wed Oct 24 15:32:20 EDT 2018


Summary of changes:
 services/api/app/models/container.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  e92125d70c4b9b5fcaaee887942d415d4b197753 (commit)
      from  028d247fdfa44a607c5e302ab60a7f62a469b6b8 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit e92125d70c4b9b5fcaaee887942d415d4b197753
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Wed Oct 24 15:30:34 2018 -0400

    14262: Fix logic for when to allow unknown PDH for containers
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/services/api/app/models/container.rb b/services/api/app/models/container.rb
index 5d46ac29f..e469a49be 100644
--- a/services/api/app/models/container.rb
+++ b/services/api/app/models/container.rb
@@ -250,7 +250,7 @@ class Container < ArvadosModel
     if !coll
       if loc = Keep::Locator.parse(container_image)
         loc.strip_hints!
-        if !Collection.where(portable_data_hash: loc.to_s).any?
+        if !Collection.readable_by(current_user).where(portable_data_hash: loc.to_s).any?
           # Allow bare pdh that doesn't exist in the local database so
           # that federated container requests which refer to remotely
           # stored containers will validate.

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list