From 8774f87aeaf92855317fcc08ccf20ff887c86722 Mon Sep 17 00:00:00 2001 From: Vlad Wolanyk Date: Wed, 17 Dec 2025 21:59:37 -0500 Subject: [PATCH] Add tags to security_policy_management endpoints Changelog: fixed EE: true --- .rubocop_todo/api/description_tags.yml | 2 -- ee/lib/api/managed_licenses.rb | 5 +++++ ee/lib/api/status_checks.rb | 7 +++++++ 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/.rubocop_todo/api/description_tags.yml b/.rubocop_todo/api/description_tags.yml index 771020dd0a3181..24f965c3129a85 100644 --- a/.rubocop_todo/api/description_tags.yml +++ b/.rubocop_todo/api/description_tags.yml @@ -28,7 +28,6 @@ API/DescriptionTags: - 'ee/lib/api/internal/search/zoekt.rb' - 'ee/lib/api/iterations.rb' - 'ee/lib/api/manage/groups.rb' - - 'ee/lib/api/managed_licenses.rb' - 'ee/lib/api/project_google_cloud_integration.rb' - 'ee/lib/api/project_mirror.rb' - 'ee/lib/api/provider_identity.rb' @@ -39,7 +38,6 @@ API/DescriptionTags: - 'ee/lib/api/security/vulnerability_scanning/sbom_scans.rb' - 'ee/lib/api/security_scans.rb' - 'ee/lib/api/service_accounts.rb' - - 'ee/lib/api/status_checks.rb' - 'ee/lib/api/vulnerabilities.rb' - 'ee/lib/api/vulnerability_archive_exports.rb' - 'ee/lib/api/vulnerability_exports.rb' diff --git a/ee/lib/api/managed_licenses.rb b/ee/lib/api/managed_licenses.rb index 46d2d2b38fdbbd..cd870902a55e60 100644 --- a/ee/lib/api/managed_licenses.rb +++ b/ee/lib/api/managed_licenses.rb @@ -36,6 +36,7 @@ def deprecation_message resource :projects, requirements: API::NAMESPACE_OR_PROJECT_REQUIREMENTS do desc 'Get project software license policies' do success code: 200, model: ::API::Entities::ManagedLicense + tags ['managed_licenses'] failure [ { code: 401, message: 'Unauthorized' }, { code: 403, message: 'Forbidden' }, @@ -55,6 +56,7 @@ def deprecation_message desc 'Get a specific software license policy from a project' do success code: 200, model: ::API::Entities::ManagedLicense + tags ['managed_licenses'] failure [ { code: 401, message: 'Unauthorized' }, { code: 403, message: 'Forbidden' }, @@ -69,6 +71,7 @@ def deprecation_message desc 'Create a new software license policy in a project' do success code: 201, model: ::API::Entities::ManagedLicense + tags ['managed_licenses'] failure [ { code: 400, message: 'Bad Request' }, { code: 401, message: 'Unauthorized' }, @@ -92,6 +95,7 @@ def deprecation_message desc 'Update an existing software license policy from a project' do success code: 200, model: ::API::Entities::ManagedLicense + tags ['managed_licenses'] failure [ { code: 400, message: 'Bad Request' }, { code: 401, message: 'Unauthorized' }, @@ -115,6 +119,7 @@ def deprecation_message desc 'Delete an existing software license policy from a project' do success code: 204 + tags ['managed_licenses'] failure [ { code: 401, message: 'Unauthorized' }, { code: 403, message: 'Forbidden' }, diff --git a/ee/lib/api/status_checks.rb b/ee/lib/api/status_checks.rb index 268f9e9a64b0cf..a1492b4d3f7a83 100644 --- a/ee/lib/api/status_checks.rb +++ b/ee/lib/api/status_checks.rb @@ -21,6 +21,7 @@ def check_feature_enabled! segment ':id/external_status_checks' do desc 'Create external status check' do success code: 201, model: ::API::Entities::ExternalStatusCheck + tags ['external_status_checks'] end params do requires :name, type: String, desc: 'Display name of external status check', documentation: { example: 'QA' } @@ -49,6 +50,7 @@ def check_feature_enabled! end desc 'Get project external status checks' do success ::API::Entities::ExternalStatusCheck + tags ['external_status_checks'] is_array true end params do @@ -63,6 +65,7 @@ def check_feature_enabled! segment ':check_id' do desc 'Update external status check' do success ::API::Entities::ExternalStatusCheck + tags ['external_status_checks'] end params do requires :check_id, @@ -96,6 +99,7 @@ def check_feature_enabled! desc 'Delete external status check' do success code: 204 + tags ['external_status_checks'] end params do requires :check_id, type: Integer, desc: 'ID of an external status check' @@ -116,6 +120,7 @@ def check_feature_enabled! segment ':id/merge_requests/:merge_request_iid' do desc 'Set status of an external status check' do success Entities::MergeRequests::StatusCheckResponse + tags ['external_status_checks'] end params do requires :id, type: String, desc: 'ID of a project', documentation: { example: '1' } @@ -164,6 +169,7 @@ def check_feature_enabled! segment 'status_checks' do desc 'List status checks for a merge request' do success Entities::MergeRequests::StatusCheck + tags ['external_status_checks'] is_array true end get '/', urgency: :low do @@ -175,6 +181,7 @@ def check_feature_enabled! desc 'Retry failed external status check' do success code: 202 + tags ['external_status_checks'] end params do requires :id, type: String, desc: 'ID of a project', documentation: { example: '1' } -- GitLab