diff options
author | Hugo Seabrook <hugo.seabrook@gmail.com> | 2013-02-16 23:21:49 +0100 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2013-02-16 23:21:49 +0100 |
commit | 91a5a96b0b9890808fe888b60ba552cb49d874a8 (patch) | |
tree | bd33b67b550f6113920ad7a338bb00dd4cf9929d /Bugzilla/BugUrl/GitHub.pm | |
parent | f45ef9337770947f5bb36829c0427e14775ffe54 (diff) | |
download | bugzilla-91a5a96b0b9890808fe888b60ba552cb49d874a8.tar.gz bugzilla-91a5a96b0b9890808fe888b60ba552cb49d874a8.tar.xz |
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 dbcaf655f..2c35bfb08 100644 --- a/Bugzilla/BugUrl/GitHub.pm +++ b/Bugzilla/BugUrl/GitHub.pm @@ -21,7 +21,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; } |