From 6be6b847fd37824e2df557a2b78ab89f0476f035 Mon Sep 17 00:00:00 2001
From: Joseph Nuthalapati <njoseph@thoughtworks.com>
Date: Mon, 22 Oct 2018 15:55:56 +0530
Subject: [PATCH] backups: Make plinth the owner of the backup archives

Changing the ownership of the backup archives from "root" to "plinth" fixes the
permission errors when downloading backup archives.

Fixes #1402

Signed-off-by: Joseph Nuthalapati <njoseph@thoughtworks.com>
Reviewed-by: James Valleroy <jvalleroy@mailbox.org>
---
 actions/backups | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/actions/backups b/actions/backups
index 2aa62fa8d..e3af4dca0 100755
--- a/actions/backups
+++ b/actions/backups
@@ -24,6 +24,7 @@ import argparse
 import glob
 import json
 import os
+import shutil
 import subprocess
 import sys
 import tarfile
@@ -140,6 +141,8 @@ def subcommand_export(arguments):
         arguments.filename
     ], check=True)
 
+    shutil.chown(arguments.filename, user='plinth', group='plinth')
+
 
 def subcommand_list_exports(arguments):
     """List exported backup archive files."""