Ticket #716: fix-permissions-after-transcode.patch

mediagoblin/media_types/video/processing.py

@@ -15 +15 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 from tempfile import NamedTemporaryFile
+import os
 import logging
 import datetime
 
@@ -54 +55 @@
     return False
 
 
+def update_file_permissions(filename):
+    m = os.umask(0)
+    os.umask(m)
+    os.chmod(filename, 0666 & ~m)
+
+
 def process_video(proc_state):
     """
     Process a video entry, transcode the queued media files (originals) and
@@ -117 +124 @@
 
             # Push transcoded video to public storage
             _log.debug('Saving medium...')
+            update_file_permissions(tmp_dst.name)
             mgg.public_store.copy_local_to_storage(tmp_dst.name, medium_filepath)
             _log.debug('Saved medium')
 
@@ -141 +149 @@
 
         # Push the thumbnail to public storage
         _log.debug('Saving thumbnail...')
+        update_file_permissions(tmp_thumb.name)
         mgg.public_store.copy_local_to_storage(tmp_thumb.name, thumbnail_filepath)
         entry.media_files['thumb'] = thumbnail_filepath