BigW Consortium Gitlab

Commit b3cc7385 by Grzegorz Bizon Committed by Winnie Hellmann

Merge branch 'sh-fix-broken-docker-delete' into 'master'

Fix inability to delete container registry tags Closes #39260 See merge request gitlab-org/gitlab-ce!14947 (cherry picked from commit 83f5955e) b7e35033 Fix inability to delete container registry tags de57492d Fix broken function invocation
parent 415483fd
...@@ -57,7 +57,7 @@ ...@@ -57,7 +57,7 @@
}, },
showError(message) { showError(message) {
Flash((errorMessages[message])); Flash(errorMessages[message]);
}, },
}, },
}; };
......
...@@ -57,7 +57,7 @@ ...@@ -57,7 +57,7 @@
}, },
showError(message) { showError(message) {
Flash((errorMessages[message])); Flash(errorMessages[message]);
}, },
}, },
}; };
......
...@@ -4,7 +4,7 @@ class ContainerTagEntity < Grape::Entity ...@@ -4,7 +4,7 @@ class ContainerTagEntity < Grape::Entity
expose :name, :location, :revision, :short_revision, :total_size, :created_at expose :name, :location, :revision, :short_revision, :total_size, :created_at
expose :destroy_path, if: -> (*) { can_destroy? } do |tag| expose :destroy_path, if: -> (*) { can_destroy? } do |tag|
project_registry_repository_tag_path(project, tag.repository, tag.name, format: :json) project_registry_repository_tag_path(project, tag.repository, tag.name)
end end
private private
......
...@@ -281,8 +281,13 @@ constraints(ProjectUrlConstrainer.new) do ...@@ -281,8 +281,13 @@ constraints(ProjectUrlConstrainer.new) do
namespace :registry do namespace :registry do
resources :repository, only: [] do resources :repository, only: [] do
resources :tags, only: [:index, :destroy], # We default to JSON format in the controller to avoid ambiguity.
constraints: { id: Gitlab::Regex.container_registry_tag_regex } # `latest.json` could either be a request for a tag named `latest`
# in JSON format, or a request for tag named `latest.json`.
scope format: false do
resources :tags, only: [:index, :destroy],
constraints: { id: Gitlab::Regex.container_registry_tag_regex }
end
end end
end end
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment