summaryrefslogtreecommitdiff
path: root/app/policies/note_policy.rb
diff options
context:
space:
mode:
authorAlexandru Croitor <acroitor@gitlab.com>2019-09-18 11:26:20 +0300
committerYorick Peterse <yorick@yorickpeterse.com>2019-09-30 14:22:05 +0200
commitc99402c05f18c6ca8dd7b64c59527abe1e6e80d4 (patch)
tree486679c27fb41fdf9d729b7277b98237841590ab /app/policies/note_policy.rb
parente5fdd91318c590753cf0f920a2cf9b1a530e96f8 (diff)
downloadgitlab-ce-c99402c05f18c6ca8dd7b64c59527abe1e6e80d4.tar.gz
Add policy check if cross reference system notes are accessible
Diffstat (limited to 'app/policies/note_policy.rb')
-rw-r--r--app/policies/note_policy.rb9
1 files changed, 9 insertions, 0 deletions
diff --git a/app/policies/note_policy.rb b/app/policies/note_policy.rb
index 8d23e3abed3..b2af6c874c7 100644
--- a/app/policies/note_policy.rb
+++ b/app/policies/note_policy.rb
@@ -11,6 +11,8 @@ class NotePolicy < BasePolicy
condition(:can_read_noteable) { can?(:"read_#{@subject.to_ability_name}") }
+ condition(:is_visible) { @subject.visible_for?(@user) }
+
rule { ~editable }.prevent :admin_note
# If user can't read the issue/MR/etc then they should not be allowed to do anything to their own notes
@@ -27,6 +29,13 @@ class NotePolicy < BasePolicy
enable :resolve_note
end
+ rule { ~is_visible }.policy do
+ prevent :read_note
+ prevent :admin_note
+ prevent :resolve_note
+ prevent :award_emoji
+ end
+
rule { is_noteable_author }.policy do
enable :resolve_note
end