summaryrefslogtreecommitdiffstats
path: root/Bugzilla/Error.pm
diff options
context:
space:
mode:
authorByron Jones <bjones@mozilla.com>2013-04-22 18:54:36 +0200
committerByron Jones <bjones@mozilla.com>2013-04-22 18:54:36 +0200
commitd07b2dda47346c3c5b94c3019d0fdfa5165247eb (patch)
treeb075f34b742726a3dd0ef94559646e09db255d71 /Bugzilla/Error.pm
parentbaf36f72baa2095b92b298fd45d9cbb522cdca87 (diff)
downloadbugzilla-d07b2dda47346c3c5b94c3019d0fdfa5165247eb.tar.gz
bugzilla-d07b2dda47346c3c5b94c3019d0fdfa5165247eb.tar.xz
Bug 750170: switch from arecibo to sentry for error reporting
Diffstat (limited to 'Bugzilla/Error.pm')
-rw-r--r--Bugzilla/Error.pm14
1 files changed, 5 insertions, 9 deletions
diff --git a/Bugzilla/Error.pm b/Bugzilla/Error.pm
index e49f466d6..08978fa93 100644
--- a/Bugzilla/Error.pm
+++ b/Bugzilla/Error.pm
@@ -28,7 +28,7 @@ use base qw(Exporter);
@Bugzilla::Error::EXPORT = qw(ThrowCodeError ThrowTemplateError ThrowUserError ThrowErrorPage);
-use Bugzilla::Arecibo;
+use Bugzilla::Sentry;
use Bugzilla::Constants;
use Bugzilla::WebService::Constants;
use Bugzilla::Util;
@@ -110,9 +110,8 @@ sub _throw_error {
message => \$message });
if (Bugzilla->error_mode == ERROR_MODE_WEBPAGE) {
- if (arecibo_should_notify($vars->{error})) {
+ if (sentry_should_notify($vars->{error})) {
$vars->{maintainers_notified} = 1;
- $vars->{uid} = arecibo_generate_id();
$vars->{processed} = {};
} else {
$vars->{maintainers_notified} = 0;
@@ -123,8 +122,7 @@ sub _throw_error {
|| ThrowTemplateError($template->error());
if ($vars->{maintainers_notified}) {
- arecibo_handle_error(
- $vars->{error}, $vars->{processed}->{error_message}, $vars->{uid});
+ sentry_handle_error($vars->{error}, $vars->{processed}->{error_message});
}
}
elsif (Bugzilla->error_mode == ERROR_MODE_TEST) {
@@ -206,8 +204,7 @@ sub ThrowTemplateError {
$vars->{'template_error_msg'} = $template_err;
$vars->{'error'} = "template_error";
- $vars->{'uid'} = arecibo_generate_id();
- arecibo_handle_error('error', $template_err, $vars->{'uid'});
+ sentry_handle_error('error', $template_err);
$vars->{'template_error_msg'} =~ s/ at \S+ line \d+\.\s*$//;
my $template = Bugzilla->template;
@@ -218,7 +215,6 @@ sub ThrowTemplateError {
my $maintainer = html_quote(Bugzilla->params->{'maintainer'});
my $error = html_quote($vars->{'template_error_msg'});
my $error2 = html_quote($template->error());
- my $uid = html_quote($vars->{'uid'});
print <<END;
<tt>
<p>
@@ -232,7 +228,7 @@ sub ThrowTemplateError {
-->
<p>
The <a href="mailto:$maintainer">Bugzilla maintainers</a> have
- been notified of this error [#$uid].
+ been notified of this error.
</p>
</tt>
END