[gentoo-commits] proj/webapp-config:master commit in: WebappConfig/

["Anthony G. Basile" <blueness@gentoo.org>]

commit:     84767b807e47e38443c7f6107b2d28d8e4595485
Author:     Vincent Brillault <gentoo <AT> lerya <DOT> net>
AuthorDate: Thu Mar 14 02:15:54 2013 +0000
Commit:     Anthony G. Basile <blueness <AT> gentoo <DOT> org>
CommitDate: Thu Mar 14 02:19:06 2013 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/webapp-config.git;a=commit;h=84767b80

WebappConfig/worker.py: correct permissions for server-owned dirs

app-admin/webapp-config uses 'os.makedirs(dst_dir, perm(0x755))'
to create new directories without any 'os.chown' after.  If the
configuration contains 'vhost_perms_serverowned_dir="0775"' or
vhost_perms_serverowned_dir="g+w" those permissions will be filtered
with the default python umask which is '022', resulting in a '0755'
permission instead of a '0775' one.

Thanks Devan Franchini <twitch153 <AT> hotmail.com> for testing.

X-Gentoo-Bug: 434894
X-Gentoo-Bug-URL: https://bugs.gentoo.org/434894
Signed-off-by: Anthony G. Basile <blueness <AT> gentoo.org>

---
 WebappConfig/worker.py |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/WebappConfig/worker.py b/WebappConfig/worker.py
index fbf8f92..07a2f1a 100644
--- a/WebappConfig/worker.py
+++ b/WebappConfig/worker.py
@@ -288,6 +288,8 @@ class WebappAdd:
 
         self.config_protected_dirs = []
 
+        os.umask(0)
+
     def mkdirs(self, directory = ''):
         '''
         Create a set of directories