diff options
author | Dan Tudor <dtudor01@gmail.com> | 2015-03-24 16:04:22 +0300 |
---|---|---|
committer | Dan Tudor <dtudor01@gmail.com> | 2015-03-24 16:04:22 +0300 |
commit | 862e1e6f178d11473319225de51c25b72174f45b (patch) | |
tree | 2eb0d64d51c7f08abf02c6f35218765393d375a1 /lib/api/branches.rb | |
parent | c3c9703437a552700c2a6ca010b05cc56aa6d73a (diff) |
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.
Diffstat (limited to 'lib/api/branches.rb')
-rw-r--r-- | lib/api/branches.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/api/branches.rb b/lib/api/branches.rb index b52d786e020..edfdf842f85 100644 --- a/lib/api/branches.rb +++ b/lib/api/branches.rb @@ -1,4 +1,5 @@ require 'mime/types' +require 'uri' module API # Projects API @@ -103,7 +104,7 @@ module API delete ":id/repository/branches/:branch" do authorize_push_project result = DeleteBranchService.new(user_project, current_user). - execute(params[:branch]) + execute(URI.unescape(params[:branch])) if result[:status] == :success { |