1196093 - support http auth for delivering reports to sentry

Status: RESOLVED WONTFIX

(bugzilla.mozilla.org :: General, defect)

Description

[:glob ✱]

sentry has just been put behind http auth, so we'll need to add parameters to store ldap credentials and update sentry.pl to perform basic http auth when submitting reports.

Comment 1

[:glob ✱]

Attachment: 1196093_1.patch

Comment 2

[David Lawrence [:dkl]]

Comment on attachment 8649660 [details] [diff] [review]
1196093_1.patch

Review of attachment 8649660 [details] [diff] [review]:
-----------------------------------------------------------------

Looks fine and works as expected. r=dkl

::: sentry.pl
@@ +88,5 @@
> +    package RequestAgent;
> +    use parent 'LWP::UserAgent';
> +
> +    sub get_basic_credentials {
> +        return Bugzilla->params->{sentry_username}

Should we check that both are filled in?

Comment 3

[:glob ✱]

this is no longer required - the http auth requirements have been removed from report submission.