Closed Bug 1787929 Opened 5 months ago Closed 5 months ago

rewrite Breadcrumbs validation

Categories

(Socorro :: Processor, task, P2)

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: nobody → willkg
Status: NEW → ASSIGNED

I deployed this to production just now in bug #1791381. Marking as FIXED.

Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED

Reopening this. I should nix the BreadcrumbsRule altogether and use the CopyFromRawCrashRule which handles validation and reducing.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

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: 5 months ago5 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.