Closed Bug 946875 Opened 11 years ago Closed 11 years ago

processor fails to cleanup temp files during failures

Categories

(Socorro :: Backend, task)

Product:
Socorro
Component:
Backend
Platform:
x86_64
Linux
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: lars, Assigned: lars)

Details

Attachments

(1 file)

PR for adding exception handling to the tempfile contextmanager
44 bytes, text/x-github-pull-request
brandon
: review+
Details | Review 
when invoking stackwalker, the dump file has been written to disk.  If something goes wrong and an exception is raised from within the context scope of the temp file, the temp file is not removed.  fix this
Assignee: nobody → lars
Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/b6fa4726878294541c7f4dab959c1419d00fb541
Merge pull request #1713 from twobraids/delete-the-temp

Fixes Bug 946875 - added exception handling to tempfile contextmanager
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Attachment #8343246 - Flags: review?(bsavage) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: