Closed Bug 633103 Opened 15 years ago Closed 15 years ago

Use tempfile when creating XPIs

Categories

(addons.mozilla.org Graveyard :: Add-on Builder, defect)

Product:
addons.mozilla.org Graveyard
Component:
Add-on Builder
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Builder 0.9

People

(Reporter: clouserw, Assigned: zalun)

Details

I thought we already did this, but I noticed that upload_xpi() in apps/jetpack/views.py uses time.time() as a directory name. With multiple boxes writing to this area it's very possible that will conflict. It needs to use the tempfile library.
Assignee: nobody → zaloon
this should actually find it's way to [ftw] upload was borken https://github.com/mozilla/FlightDeck/commit/11be0d0b9ee52cdc9c58b6c47f2ec7f9b04f7a03
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.