Merge branch 'master' into 61278-next61278-next

* master: (64 commits)
  Allow usage of quick actions for internal users
  Added Omniauth OpenId Connect startegy
  Changelog update for MR #27025, Issue #57528
  Docs: Realigning scattered EE docs into CE
  Add troubleshooting step for older docker clients
  Edit regionendpoint text
  Docs: Change geo redirects to relative to match EE
  Docs: Second Attempt to realign administration/geo docs
  Document Prometheus app can be uninstalled
  Docs: Merge EE doc/administration/high_availability to CE
  Docs: Merge EE doc/gitlab-geo to CE
  Docs: Merge EE doc/user/application_security to CE
  Docs: Merge EE doc/​administration/​geo to CE
  Docs: Merge Various EE /doc dirs to CE
  Docs: Merge EE doc/user/project/*.md to CE
  Docs: Merge 4 EE doc/user/project dirs to CE
  Docs: Merge misc EE doc/user/project/i* dirs to CE
  Docs: Merge Misc EE doc/administration files and dirs to CE
  Docs: Merge EE doc/ci to CE
  Docs: Merge EE doc/user/project/issues to CE
  ...

1 files changed, 15 insertions, 4 deletions

diff --git a/lib/api/entities.rb b/lib/api/entities.rb
index ee8480122c4..a228614f684 100644
--- a/lib/api/entities.rb
+++ b/lib/api/entities.rb
@@ -1156,22 +1156,33 @@ module API
       end
     end
 
-    class Release < TagRelease
+    class Release < Grape::Entity
       expose :name
+      expose :tag, as: :tag_name, if: lambda { |_, _| can_download_code? }
+      expose :description
       expose :description_html do |entity|
         MarkupHelper.markdown_field(entity, :description)
       end
       expose :created_at
       expose :author, using: Entities::UserBasic, if: -> (release, _) { release.author.present? }
-      expose :commit, using: Entities::Commit
+      expose :commit, using: Entities::Commit, if: lambda { |_, _| can_download_code? }
 
       expose :assets do
-        expose :assets_count, as: :count
-        expose :sources, using: Entities::Releases::Source
+        expose :assets_count, as: :count do |release, _|
+          assets_to_exclude = can_download_code? ? [] : [:sources]
+          release.assets_count(except: assets_to_exclude)
+        end
+        expose :sources, using: Entities::Releases::Source, if: lambda { |_, _| can_download_code? }
         expose :links, using: Entities::Releases::Link do |release, options|
           release.links.sorted
         end
       end
+
+      private
+
+      def can_download_code?
+        Ability.allowed?(options[:current_user], :download_code, object.project)
+      end
     end
 
     class Tag < Grape::Entity