FS#74662 - nginx{,-mainline} PKGBUILD update on "--error-log-path" config directive

Attached to Project: Community Packages
Opened by Fabiano Furtado (fusca) - Friday, 06 May 2022, 15:00 GMT
Last edited by T.J. Townsend (blakkheim) - Friday, 02 June 2023, 14:19 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Massimiliano Torromeo (mtorromeo)
Levente Polyak (anthraxx)
Giancarlo Razzolini (grazzolini)
T.J. Townsend (blakkheim)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
Please, consider update "--error-log-path=stderr" to "--error-log-path=/var/log/nginx/error.log" in PKGBUILD of nginx and nginx-mainline packages. As described in http://nginx.org/en/docs/configure.html:
" --error-log-path=path
sets the name of the primary error, warnings, and diagnostic file. After installation, the file name can always be changed in the nginx.conf configuration file using the error_log directive. By default the file is named prefix/logs/error.log."

Thanks!

Additional info:
* package version(s): all versions of nginx and nginx-mainline packages
* config and/or log files etc.: PKGBUILD
* link to upstream bug report, if any: None

Steps to reproduce:
NONE
This task depends upon

Closed by  T.J. Townsend (blakkheim)
Friday, 02 June 2023, 14:19 GMT
Reason for closing:  Won't implement
Comment by Toolybird (Toolybird) - Friday, 08 July 2022, 07:12 GMT
 FS#38011  implemented current switch back in 2013. Is there are a compelling reason to change it back now?
Comment by Fabiano Furtado (fusca) - Friday, 08 July 2022, 10:24 GMT
Hi... there's not a specific reason or a bug related, but the default value is a log file, not stderr.

Default: error_log logs/error.log error;

http://nginx.org/en/docs/ngx_core_module.html#error_log
Comment by Toolybird (Toolybird) - Friday, 08 July 2022, 23:33 GMT
Ok, so no convincing argument.. But reading further [1] it seems things might have changed since 2013. I also note that both Fedora and Debian configure with "--error-log-path=/var/log/nginx/error.log" (they also provide the directive in a conf file). I'll assign to PM who can make the call :)

[1] https://stackoverflow.com/questions/22541333
Comment by Fabiano Furtado (fusca) - Saturday, 09 July 2022, 22:42 GMT
Thank you!
Comment by T.J. Townsend (blakkheim) - Friday, 02 June 2023, 14:19 GMT
Discussed with another nginx maintainer and we'd prefer to keep it the way it is for now.

Loading...