antenna should ignore querystring params
Categories
(Socorro :: Antenna, defect, P2)
Tracking
(Not tracked)
People
(Reporter: willkg, Assigned: willkg)
References
Details
Attachments
(2 files)
In bug #1420363, we're trying to put all the annotations in a single JSON-serialized value so then we'd have an "extra" field and zero or more minidump fields. Antenna has a check where it gets fussy if there's an "extra" field and also one or more non-minidump fields.
It seems like that's not working right. The check is probably fine, but either cgi.FieldStorage is spitting out fields that don't exist or something I'm not imaginative to come up with.
This bug covers adding some logging and stuff to figure that out and then eradicating the problem.
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
|
Assignee | |
Comment 3•5 years ago
|
||
We figured out what's going on. Antenna's cgi.FieldStorage usage is pulling both the HTTP POST payload as well as the querystring params. We don't care about the querystirng params--we want to ignore them.
So this bug covers fixing it to ignore querystring params correctly.
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
|
Assignee | |
Comment 5•5 years ago
|
||
|
|
Assignee | |
Comment 6•5 years ago
|
||
I verified this fixes the issue in stage. I'll deploy this to prod on Monday.
|
|
Assignee | |
Comment 7•5 years ago
|
||
This is in prod now. Marking as FIXED.
Description
•