diff options
| author |   Hugo Seabrook <hugo.seabrook@gmail.com> | 2013-02-16 23:23:14 +0100 |
|---|---|---|
| committer |   Frédéric Buclin <LpSolit@gmail.com> | 2013-02-16 23:23:14 +0100 |
| commit | 81c26ce64c05a0c4a7b800f5212412e96fb2e4cc (patch) | |
| tree | 471a9151d25791fc3930f8eabcff5b204f824fbc /Bugzilla/BugUrl/GitHub.pm | |
| parent | Bug 840824: It is possible to create a new bug with a non active target miles... (diff) | |
| download | bugzilla-81c26ce64c05a0c4a7b800f5212412e96fb2e4cc.tar.gz bugzilla-81c26ce64c05a0c4a7b800f5212412e96fb2e4cc.tar.bz2 bugzilla-81c26ce64c05a0c4a7b800f5212412e96fb2e4cc.zip | |
Bug 825758: Most Bugzilla::BugUrl::* modules incorrectly validate the domain name
r/a=LpSolit
Diffstat (limited to 'Bugzilla/BugUrl/GitHub.pm')
| -rw-r--r-- | Bugzilla/BugUrl/GitHub.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Bugzilla/BugUrl/GitHub.pm b/Bugzilla/BugUrl/GitHub.pm index 63be65bed..eeffa2ed6 100644 --- a/Bugzilla/BugUrl/GitHub.pm +++ b/Bugzilla/BugUrl/GitHub.pm @@ -18,7 +18,7 @@ sub should_handle { # GitHub issue URLs have only one form: # https://github.com/USER_OR_TEAM_OR_ORGANIZATION_NAME/REPOSITORY_NAME/issues/111 - return ($uri->authority =~ /^github.com$/i + return (lc($uri->authority) eq 'github.com' and $uri->path =~ m|^/[^/]+/[^/]+/issues/\d+$|) ? 1 : 0; } |