fix issue when temp dir does not exists

certsocietegenerale · Mar 28, 2017 · ad30857 · ad30857
1 parent 573d542
commit ad30857
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/web/views/modules.py b/web/views/modules.py
@@ -11,7 +11,7 @@
 from web.views.helpers import get_or_404, requires_permission, file_download, clean_modules, clean_repositories
 from fame.common.config import fame_config
 from fame.common.constants import FAME_ROOT, MODULES_ROOT
-from fame.common.utils import get_class
+from fame.common.utils import get_class, tempdir
 from fame.common.exceptions import MissingConfiguration
 from fame.core.module import ModuleInfo
 from fame.core.config import Config
@@ -445,8 +445,8 @@ def repository_new(self):
     @route('/download', methods=['GET'])
     def download(self):
         # First, create a zip file with the modules
-        f, path = mkstemp(dir=fame_config.temp_path)
-        with ZipFile(os.fdopen(f, 'w'), 'w') as zipf:
+        path = os.path.join(tempdir(), 'modules.zip')
+        with ZipFile(path, 'w') as zipf:
             for root, dirs, files in os.walk(MODULES_ROOT):
                 for filename in files:
                     # Ignore pyc files