Closed
Bug 1787929
Opened 2 years ago
Closed 2 years ago
rewrite Breadcrumbs validation
Categories
(Socorro :: Processor, task, P2)
Socorro
Processor
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: willkg)
References
Details
Attachments
(1 file)
There is a processor rule that validates the structure of the Breadcrumbs data. Now that we've got a schema, that should be revisited.
|
Assignee | |
Comment 1•2 years ago
|
||
Assignee: nobody → willkg
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•2 years ago
|
||
|
|
Assignee | |
Comment 3•2 years ago
|
||
I deployed this to production just now in bug #1791381. Marking as FIXED.
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 4•2 years ago
|
||
Reopening this. I should nix the BreadcrumbsRule altogether and use the CopyFromRawCrashRule which handles validation and reducing.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
|
Assignee | |
Comment 5•2 years ago
|
||
Bah--I remember why I can't use CopyFromRawCrashRule. It's because Breadcrumbs may or may not have an additional "values" intermediary key which the BreadcrumbsRule nixes when saving the value to the processed_crash.
So what I implemented is good. Re-marking this as FIXED.
Status: REOPENED → RESOLVED
Closed: 2 years ago → 2 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•