search

nginx / unit

NGINX Unit - universal web app server - a lightweight and versatile open source server that simplifies the application stack by natively executing application code across eight different programming language runtimes.
https://unit.nginx.org
Apache License 2.0
5.25k stars 322 forks source link

Broken stdout and stderr logging in Amazon Linux #1178

Closed thesame closed 1 week ago

thesame commented 3 months ago

Systemd unit in package unit-1.32.0-1.amzn2023.ngx.x86_64.rpm contains this line: /usr/sbin/unitd $UNITD_OPTIONS --no-daemon which renders stdout and stderr options of /config/applications/* objects useless, according to this line in the documentation:

When running in --no-daemon mode, application output is always redirected to Unit’s log file.

thresheek commented 3 months ago

For the record, --no-daemon was introduced in https://github.com/nginx/unit/pull/212

I'm not sure if we want to change this to daemonization; or rather provide documentation on how to edit the service file to override the ExecStart= line.

We do run unit on debian with daemonization, so this only applies to rpm-based distros.

Thoughts?

ac000 commented 3 months ago

This has been discussed here, and my feeling on it is here.

thresheek commented 3 months ago

So I guess that's at least +2 for moving back to daemon/forking. I'll prepare a PR with the changes, if noone objects.

BwL1289 commented 2 months ago

Also interested.