Closed
Bug 963431
Opened 11 years ago
Closed 11 years ago
typerror in socorro/cron/jobs/ftpscraper.py in getJsonRelease , line 170
Categories
(Socorro :: Backend, task)
Socorro
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
72
People
(Reporter: rhelmer, Assigned: rhelmer)
Details
(Whiteboard: [qa-])
Attachments
(2 files)
Looks like https://github.com/mozilla/socorro/blob/master/socorro/cron/jobs/ftpscraper.py#L170 is trying to index into the None it's getting back from parseBuildJsonFile() (which it is ok and it should handle)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → rhelmer
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8364856 -
Flags: review?(sdeckelmann)
Assignee | ||
Comment 2•11 years ago
|
||
Comment 3•11 years ago
|
||
Needed to use continue instead of return. Added some tests.
Attachment #8365086 -
Flags: review?(rhelmer)
Updated•11 years ago
|
Attachment #8364856 -
Flags: review?(sdeckelmann)
Comment 4•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/cbc0fb576a8fa1bfd6d338e0d5471aaa1540494f
Fixes bug 963431 handle empty result from getJsonRelease()
https://github.com/mozilla/socorro/commit/067c0342379f53bba250aab6290847bbe7477fb3
Merge pull request #1820 from selenamarie/bug963431-handle-empty-json-result
Fixes bug 963431 handle empty result from getJsonRelease()
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Target Milestone: --- → 72
Updated•11 years ago
|
Attachment #8365086 -
Flags: review?(rhelmer) → review+
Assignee | ||
Updated•11 years ago
|
Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•