mirror of
https://github.com/frappe/bench.git
synced 2024-11-12 00:06:36 +00:00
fix: Change ownership of all log and config files (#787)
Without this, after ` sudo bench setup production ` file ownership of all log files isn't changed and requires human intervention.
This commit is contained in:
parent
3dbb412e3c
commit
2435cb6b2e
@ -1,4 +1,4 @@
|
||||
import os, sys, shutil, subprocess, logging, itertools, requests, json, platform, select, pwd, grp, multiprocessing, hashlib
|
||||
import os, sys, shutil, subprocess, logging, itertools, requests, json, platform, select, pwd, grp, multiprocessing, hashlib, glob
|
||||
from distutils.spawn import find_executable
|
||||
import bench
|
||||
import semantic_version
|
||||
@ -556,16 +556,6 @@ def drop_privileges(uid_name='nobody', gid_name='nogroup'):
|
||||
|
||||
def fix_prod_setup_perms(bench_path='.', frappe_user=None):
|
||||
from .config.common_site_config import get_config
|
||||
files = [
|
||||
"logs/web.error.log",
|
||||
"logs/web.log",
|
||||
"logs/workerbeat.error.log",
|
||||
"logs/workerbeat.log",
|
||||
"logs/worker.error.log",
|
||||
"logs/worker.log",
|
||||
"config/nginx.conf",
|
||||
"config/supervisor.conf",
|
||||
]
|
||||
|
||||
if not frappe_user:
|
||||
frappe_user = get_config(bench_path).get('frappe_user')
|
||||
@ -574,8 +564,9 @@ def fix_prod_setup_perms(bench_path='.', frappe_user=None):
|
||||
print("frappe user not set")
|
||||
sys.exit(1)
|
||||
|
||||
for path in files:
|
||||
if os.path.exists(path):
|
||||
globs = ["logs/*", "config/*"]
|
||||
for glob_name in globs:
|
||||
for path in glob.glob(glob_name, recursive=True):
|
||||
uid = pwd.getpwnam(frappe_user).pw_uid
|
||||
gid = grp.getgrnam(frappe_user).gr_gid
|
||||
os.chown(path, uid, gid)
|
||||
|
Loading…
Reference in New Issue
Block a user