Reviewer detection sometimes fails due to user being logged out

RESOLVED FIXED

Status

MozReview
General
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mcote, Assigned: mcote)

Tracking

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
The main problem with reviewer detection appears to be this:

Bugzilla error: Logged-out users cannot use the "match" argument to this function to access any user information. (HTTP 500, API Error 226)

I have a feeling it's related to cookie auth.

There's another reviewer-detection problem in bug 1169724, but it's a separate problem, with Bugzilla itself.
Mark and I were talking about this on the walk to breakfast this morning. My ~/.hgrc just has my bugzilla username, and fetches my bugzilla cookie out of my Firefox profile for auth. I've never had reviewer matching work.
(Assignee)

Comment 2

2 years ago
Created attachment 8626335 [details]
MozReview Request: mozreview: Always record cookies in session when logging in (bug 1177454). r?dminor

mozreview: Always record cookies in session when logging in (bug 1177454). r?dminor

The Bugzilla authentication layer was not setting the user cookie variables,
which are saved to the session, when the login was performed with cookies.
This was probably left over from an early attempt to use Bugzilla's cookies
directly when authenticated with the UI.  Now we only use cookies with
API authentication, so we need to percolate them up to the middleware to save
in the session.
Attachment #8626335 - Flags: review?(dminor)

Comment 3

2 years ago
Comment on attachment 8626335 [details]
MozReview Request: mozreview: Always record cookies in session when logging in (bug 1177454). r?dminor

https://reviewboard.mozilla.org/r/12065/#review10481

Ship It!
Attachment #8626335 - Flags: review?(dminor) → review+
(Assignee)

Comment 4

2 years ago
http://hg.mozilla.org/hgcustom/version-control-tools/rev/4d6865755492
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Product: Developer Services → MozReview
You need to log in before you can comment on or make changes to this bug.