1478889 - PhabBugz::User class doesn't handle the condition of a bugzilla user id being invalid.

Status: NEW

(bugzilla.mozilla.org :: Phabricator Integration, defect, P3)

Description

[Dylan Hardison [:dylan] (he/him)]

SSIA

Comment 1

[David Lawrence [:dkl]]

When looking up a user in Phabricator based on a user's bugzilla id or when Phabricator returns an invalid bugzilla user id and we try to convert it to a Bugzilla::User object?

Comment 2

[Dylan Hardison [:dylan] (he/him)]

Not sure I understand the question here. At the moment this isn't a problem so we can back-burner it

Comment 3

[David Lawrence [:dkl]]

The original question I think was what to do in the case that we encounter a changed made by a Phab user that does not
map to a user in Bugzilla and Bugzilla throws an exception. This problem causes the entire feed queue to stall. What would
be a better solution? Do we plug in a bogus BMO user in it's place ([e-mail])?. Do we automatically create a user
in BMO for the Phab user? Or do we continue to just let it fail and when we get notified from DataDog we change the data in
Phabricator to use a real mapped account instead?

Comment 4

[Dylan Hardison [:dylan] (he/him)]

Ah I see! That's a good point.

Generating a new bmo account seems like the most audit-able situation and so that's what I vote for.