Commit 4f687aaa8e7f62da40cb03f007cc6ad30047c18f
Exists in
master
and in
4 other branches
Merge pull request #1827 from mgrobelin/master
grack auth bugfix: allow dots in reponame
Showing
2 changed files
with
3 additions
and
3 deletions
Show diff stats
config/routes.rb
@@ -18,7 +18,7 @@ Gitlab::Application.routes.draw do | @@ -18,7 +18,7 @@ Gitlab::Application.routes.draw do | ||
18 | project_root: Gitlab.config.git_base_path, | 18 | project_root: Gitlab.config.git_base_path, |
19 | upload_pack: Gitlab.config.git_upload_pack, | 19 | upload_pack: Gitlab.config.git_upload_pack, |
20 | receive_pack: Gitlab.config.git_receive_pack | 20 | receive_pack: Gitlab.config.git_receive_pack |
21 | - }), at: '/:path', constraints: { path: /[\w-]+\.git/ } | 21 | + }), at: '/:path', constraints: { path: /[\w\.-]+\.git/ } |
22 | 22 | ||
23 | # | 23 | # |
24 | # Help | 24 | # Help |
lib/gitlab/backend/grack_auth.rb
@@ -18,7 +18,7 @@ module Grack | @@ -18,7 +18,7 @@ module Grack | ||
18 | @env['SCRIPT_NAME'] = "" | 18 | @env['SCRIPT_NAME'] = "" |
19 | 19 | ||
20 | # Find project by PATH_INFO from env | 20 | # Find project by PATH_INFO from env |
21 | - if m = /^\/([\w-]+).git/.match(@request.path_info).to_a | 21 | + if m = /^\/([\w\.-]+)\.git/.match(@request.path_info).to_a |
22 | self.project = Project.find_by_path(m.last) | 22 | self.project = Project.find_by_path(m.last) |
23 | return false unless project | 23 | return false unless project |
24 | end | 24 | end |
@@ -65,7 +65,7 @@ module Grack | @@ -65,7 +65,7 @@ module Grack | ||
65 | end | 65 | end |
66 | # Need to reset seek point | 66 | # Need to reset seek point |
67 | @request.body.rewind | 67 | @request.body.rewind |
68 | - /refs\/heads\/([\w-]+)/.match(input).to_a.first | 68 | + /refs\/heads\/([\w\.-]+)/.match(input).to_a.first |
69 | end | 69 | end |
70 | 70 | ||
71 | protected | 71 | protected |