summaryrefslogtreecommitdiffstats
path: root/Bugzilla
diff options
context:
space:
mode:
authorDylan William Hardison <dylan@mozilla.com>2015-05-29 18:22:20 +0200
committerDylan William Hardison <dylan@hardison.net>2015-05-29 18:22:20 +0200
commitd0acf505c5a451b8f90c84628398b42d65dabc42 (patch)
tree71c29cd5801d791cee84ffb28d0cfe5ee52bb33d /Bugzilla
parent09f53409bf172a3d6452d992e3afce7cd7879f12 (diff)
downloadbugzilla-d0acf505c5a451b8f90c84628398b42d65dabc42.tar.gz
bugzilla-d0acf505c5a451b8f90c84628398b42d65dabc42.tar.xz
Bug 1165866: "See Also" allows github issues but not github pull requests
Diffstat (limited to 'Bugzilla')
-rw-r--r--Bugzilla/BugUrl/GitHub.pm6
1 files changed, 4 insertions, 2 deletions
diff --git a/Bugzilla/BugUrl/GitHub.pm b/Bugzilla/BugUrl/GitHub.pm
index 63be65bed..29f3c2ea4 100644
--- a/Bugzilla/BugUrl/GitHub.pm
+++ b/Bugzilla/BugUrl/GitHub.pm
@@ -18,8 +18,10 @@ 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
- and $uri->path =~ m|^/[^/]+/[^/]+/issues/\d+$|) ? 1 : 0;
+ # GitHub pull request URLs have only one form:
+ # https://github.com/USER_OR_TEAM_OR_ORGANIZATION_NAME/REPOSITORY_NAME/pull/111
+ return (lc($uri->authority) eq 'github.com'
+ and $uri->path =~ m!^/[^/]+/[^/]+/(?:issues|pull)/\d+$!) ? 1 : 0;
}
sub _check_value {