Closed Bug 1100352 Opened 10 years ago Closed 1 year ago

Remove hard-coded allow lists from ProcessedCrash and RawCrash models

Categories

(Socorro :: Webapp, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: adrian, Assigned: willkg)

References

Details

We should rely on our Super Search Fields list and remove the hard-coded white lists of fields in our ProcessedCrash and RawCrash models. However, before doing so, we need to audit our current list to make sure no field was forgotten, so that we don't accidentally remove fields from exposure in our public API.
See Also: → 1396001
Issei: At some point, I want to tackle this, but not quiet, yet. I think this requires some careful architecture work before we can make code changes.
Flags: needinfo?(peterbe)
Thanks for your comment, Will!
I see. I'll find another issue to work on.
Summary: Remove hard-coded white lists from ProcessedCrash and RawCrash models → Remove hard-coded allow lists from ProcessedCrash and RawCrash models

I removed the allow list for the ProcessedCrash API in 683ef8d2.

Still have to do the RawCrash API.

Depends on: 1764395

Grabbing this to do soon.

Assignee: nobody → willkg
Status: NEW → ASSIGNED

The allow list for the RawCrash API was removed in adda44e8.

All done here!

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Depends on: 1626698
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.