support http auth for delivering reports to sentry

RESOLVED WONTFIX

Status

()

--
major
RESOLVED WONTFIX
4 years ago
4 years ago

People

(Reporter: glob, Assigned: glob)

Tracking

Production

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
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.
(Assignee)

Updated

4 years ago
Depends on: 1196095
(Assignee)

Comment 1

4 years ago
Created attachment 8649660 [details] [diff] [review]
1196093_1.patch
Assignee: nobody → glob
Attachment #8649660 - Flags: review?(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?
Attachment #8649660 - Flags: review?(dkl) → review+
(Assignee)

Comment 3

4 years ago
this is no longer required - the http auth requirements have been removed from report submission.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.