Commit 862e1e6f authored by Dan Tudor's avatar Dan Tudor

Unescape branch param to delete

Branch names that contain `/` return a 405 error when being deleted because the slash is escaped to `%2F`
This patch will unescape the param prior to  executing the delete action.
parent c3c97034
require 'mime/types' require 'mime/types'
require 'uri'
module API module API
# Projects API # Projects API
...@@ -103,7 +104,7 @@ module API ...@@ -103,7 +104,7 @@ module API
delete ":id/repository/branches/:branch" do delete ":id/repository/branches/:branch" do
authorize_push_project authorize_push_project
result = DeleteBranchService.new(user_project, current_user). result = DeleteBranchService.new(user_project, current_user).
execute(params[:branch]) execute(URI.unescape(params[:branch]))
if result[:status] == :success if result[:status] == :success
{ {
......
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