projects
/
arvados.git
/ commitdiff
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(from parent 1:
e79e5c2
)
3687: Add comment about redundant lookup.
author
Tom Clegg <tom@curoverse.com>
Fri, 3 Oct 2014 23:59:00 +0000
(19:59 -0400)
committer
Tom Clegg <tom@curoverse.com>
Fri, 3 Oct 2014 23:59:00 +0000
(19:59 -0400)
services/api/app/models/user.rb
patch
|
blob
|
history
diff --git
a/services/api/app/models/user.rb
b/services/api/app/models/user.rb
index f283ea3ef8f2e8d1911a46534c65bd50b41f837a..6e7facd5d550ee45bd948a255e28e5b18ddec6cb 100644
(file)
--- a/
services/api/app/models/user.rb
+++ b/
services/api/app/models/user.rb
@@
-83,7
+83,10
@@
class User < ArvadosModel
end
if sufficient_perms
# Check permission links with head_uuid pointing directly at
- # the target object
+ # the target object. If target is a Group, this is redundant
+ # and will fail except [a] if permission caching is broken or
+ # [b] during a race condition, where a permission link has
+ # *just* been added.
if Link.where(link_class: 'permission',
name: sufficient_perms,
tail_uuid: groups_i_can(action) + [self.uuid],