1633501 - remove vestigial user_id code

Status: RESOLVED FIXED

(Socorro :: Processor, task, P2)

Description

[Will Kahn-Greene [:willkg] ET needinfo? me]

It looks like there was a UserID annotation in the raw crash and that got pulled into the processed crash as user_id. At some point, someone changed the UserDataRule processor rule to always set user_id to the empty string.

This bug covers removing all UserID and user_id stuff from the processor.

Relatedly, if we're concerned crash reports are coming in with UserID, then we can remove that in the collector so we're not storing it at all anywhere.

Comment 1

[Will Kahn-Greene [:willkg] ET needinfo? me]

Making this a P2 to look into. If this is really a user id, we shouldn't be storing it anywhere.

Comment 2

[Will Kahn-Greene [:willkg] ET needinfo? me]

Grabbing this to do soon. It's pretty straight-forward.

Comment 3

[Will Kahn-Greene [:willkg] ET needinfo? me]

Attachment: pr 5390: bug 1633501: remove vestigal user_id bits in processor

Comment 4

[Will Kahn-Greene [:willkg] ET needinfo? me]

willkg merged PR #5390: "bug 1633501: remove vestigal user_id bits in processor" in e7566b1.

Comment 5

[Will Kahn-Greene [:willkg] ET needinfo? me]

Deployed in bug #1637009. Marking as FIXED.