pt/dataflow
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix nginx config file permissions after sudo cp

Browse Source 
sudo cp creates the file as root:root 0600, making it unreadable by the
app user. Add sudo chmod 644 after writing so status detection can read
it without sudo, matching how other nginx configs are set up.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Paul Trowbridge 2026-04-05 17:10:59 -04:00
parent 4b8864edd9
commit a3c7be61d0
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manage.py
View File

@ -147,14 +147,12 @@ def ui_build_time():
return None
def nginx_domain(port):
"""Find nginx site proxying to our port. Uses sudo to read root-owned configs."""
"""Find nginx site proxying to our port."""
if not NGINX_DIR.exists():
return None
for f in NGINX_DIR.iterdir():
try:
r = subprocess.run(['sudo', '-n', 'cat', str(f)],
capture_output=True, text=True)
text = r.stdout if r.returncode == 0 else f.read_text()
text = f.read_text()
if f':{port}' in text:
for line in text.splitlines():
if 'server_name' in line:
@ -497,6 +495,7 @@ server {{
if r.returncode != 0:
err(f'Could not write {conf_path} — check sudo permissions')
return
sudo_run(['chmod', '644', str(conf_path)])
ok(f'nginx config written to {conf_path}')
print(' Testing nginx configuration...')