diff options
| author | David Lawrence <dkl@mozilla.com> | 2016-08-09 19:14:18 +0200 |
|---|---|---|
| committer | David Lawrence <dkl@mozilla.com> | 2016-08-09 19:14:18 +0200 |
| commit | c023fdabcb345293b7559102be534432977d29f6 (patch) | |
| tree | 5cfc540c7634246c82b547f0ea663ec79cedcae7 /extensions | |
| parent | dfb5957e473fae7619d529de6b5fdd3540588d21 (diff) | |
| download | bugzilla-c023fdabcb345293b7559102be534432977d29f6.tar.gz bugzilla-c023fdabcb345293b7559102be534432977d29f6.tar.xz | |
Bug 1287895 - fetching Github pull requests should have better logging to help with debugging errors
Diffstat (limited to 'extensions')
| -rw-r--r-- | extensions/BMO/Extension.pm | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm index ca4701748..5b87d2e8b 100644 --- a/extensions/BMO/Extension.pm +++ b/extensions/BMO/Extension.pm @@ -1176,8 +1176,12 @@ sub _attachment_fetch_github_pr_diff { $ua->proxy('https', Bugzilla->params->{proxy_url}); } - my $response = $ua->get($self->data . ".diff"); - return "Error retrieving Github pull request diff" if $response->is_error; + my $pr_diff = $self->data . ".diff"; + my $response = $ua->get($pr_diff); + if ($response->is_error) { + warn "Github fetch error: $pr_diff, " . $response->status_line; + return "Error retrieving Github pull request diff for " . $self->data; + } return $response->content; } |