Skip to content
This repository has been archived by the owner on Apr 26, 2021. It is now read-only.

Get rid of api.py temporary directory for sample storage logic #577

Open
jbremer opened this issue Jun 11, 2015 · 1 comment
Open

Get rid of api.py temporary directory for sample storage logic #577

jbremer opened this issue Jun 11, 2015 · 1 comment
Assignees
Labels

Comments

@jbremer
Copy link
Member

jbremer commented Jun 11, 2015

When submitting a task through api.py a temporary directory is created with the submitted file in it. This temporary is used to preserve the filename of the sample. However, as this information is also available in the database, it's not actually required, and thus the logic of the temporary directory can be removed.
The actual bug is the fact that the temporary directory is not actually removed upon deleting the task, thus leaving thousands upon thousands temporary directories around (assuming you have that many tasks).

@botherder
Copy link
Member

It is currently used by both api.py as well as the Django interface. Checking if we can safely get rid of it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants
@botherder @jbremer and others