fixed git sync app (#1247)

* fixed git sync app * removed shell=True for subprocess call
cvat-ai · Mar 10, 2020 · 7050ad4 · 7050ad4
1 parent ec2fa6e
commit 7050ad4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cvat/apps/git/git.py b/cvat/apps/git/git.py
@@ -271,7 +271,7 @@ def push(self, user, scheme, host, db_task, last_save):
         display_name += " for images" if self._task_mode == "annotation" else " for videos"
         cvat_dumper = AnnotationDumper.objects.get(display_name=display_name)
         dump_name = os.path.join(db_task.get_task_dirname(),
-            "git_annotation_{}.".format(timestamp) + "dump")
+            "git_annotation_{}.xml".format(timestamp))
         dump_task_data(
             pk=self._tid,
             user=user,
@@ -283,7 +283,7 @@ def push(self, user, scheme, host, db_task, last_save):
 
         ext = os.path.splitext(self._path)[1]
         if ext == '.zip':
-            subprocess.call('zip -j -r "{}" "{}"'.format(self._annotation_file, dump_name), shell=True)
+            subprocess.run(args=['7z', 'a', self._annotation_file, dump_name])
         elif ext == '.xml':
             shutil.copyfile(dump_name, self._annotation_file)
         else: