Download attachments to tmp file, not into memory

boards/models/attachment/downloaders.py

@@ -1,7 +1,8 b''
 import os
 import re
 
-from django.core.files.uploadedfile import SimpleUploadedFile
+from django.core.files.uploadedfile import SimpleUploadedFile, \
+    TemporaryUploadedFile
 from pytube import YouTube
 import requests
 
@@ -14,7 +15,7 b' HTTP_RESULT_OK = 200'
 HEADER_CONTENT_LENGTH = 'content-length'
 HEADER_CONTENT_TYPE = 'content-type'
 
-FILE_DOWNLOAD_CHUNK_BYTES = 100000
+FILE_DOWNLOAD_CHUNK_BYTES = 200000
 
 YOUTUBE_URL = re.compile(r'https?://www\.youtube\.com/watch\?v=\w+')
 
@@ -38,17 +39,19 b' class Downloader:'
 
         # Download file, stop if the size exceeds limit
         size = 0
-        content = b''
+
+        # Set a dummy file name that will be replaced
+        # anyway, just keep the valid extension
+        filename = 'file.' + content_type.split('/')[1]
+
+        file = TemporaryUploadedFile(filename, content_type, 0, None, None)
         for chunk in response.iter_content(FILE_DOWNLOAD_CHUNK_BYTES):
             size += len(chunk)
             validate_file_size(size)
-            content += chunk
+            file.write(chunk)
 
-        if response.status_code == HTTP_RESULT_OK and content:
-            # Set a dummy file name that will be replaced
-            # anyway, just keep the valid extension
-            filename = 'file.' + content_type.split('/')[1]
-            return SimpleUploadedFile(filename, content, content_type)
+        if response.status_code == HTTP_RESULT_OK:
+            return file
 
 
 class YouTubeDownloader(Downloader):