Bug 13045

Summary: NGINX - оптимизация файлов конфигурации nginx.conf conf.d/*
Product: [ROSA-based products] ROSA Fresh Reporter: AleXandr <a.avdonin>
Component: Net (ssh, samba, ssl, NM...)Assignee: Mikhail Novosyolov <m.novosyolov>
Status: VERIFIED FIXED QA Contact: ROSA Linux Bugs <bugs>
Severity: normal    
Priority: Normal CC: a.proklov, pastordidi, v.potapov
Version: ServerFlags: v.potapov: qa_verified+
a.proklov: published+
Target Milestone: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Platform: --- ROSA Vulnerability identifier:
RPM Package: ISO-related:
Bad POT generating: Upstream:

Description AleXandr 2022-12-22 15:50:38 MSK
ОС: РОСА Fresh/Хром 12
Ядро: 5.15.77-generic-1rosa2021.1-x86_64
Среда: VirtualBox 7

В конфигурационном файле /etc/nginx/nginx.conf сейчас все находится вместе, и настройка самого сервиса http {} и конфигурация виртуальных хостов server {}. При этом есть специализированный каталог conf.d для виртуальных хостов с шаблонами:
rosa2021 nginx # tree 
.
├── conf.d
│   ├── ssl.conf
│   └── virtual.conf
...
├── nginx.conf
├── nginx.conf.default
...
└── win-utf

Логично будет убрать конфигурацию виртуальных хостов в каталог /etc/nginx/conf.d/ инклудом вида в блоке http {}:

include /etc/nginx/conf.d/*.conf;

Такой вариант более логичным и читаемым. 


И еще расскоментировать (тут спорно конечно, но по умолчанию я бы сделал так):
#access_log  logs/access.log  main; Для записи лога в дефолтной конфигурации
и 
#gzip  on; Зависит от требований конкретного веб приложения, но по умолчанию я бы открыл.
Comment 1 Mikhail Novosyolov 2023-10-01 23:49:31 MSK
********* QA ADVISORY *********

angie 1.3.0-1
- обновлено до 1.3.0
- включены все возможные модули, например, добавлен модуль ssl_preread
- убраны конфиги /etc/angie/conf.d
https://abf.io/build_lists/4751172
https://abf.io/build_lists/4751173
https://abf.io/build_lists/4751175
https://abf.io/build_lists/4751177
https://abf.io/build_lists/4751179

nginx 1.24.0-2
- включены все возможные модули, например, добавлен модуль ssl_preread
- убраны конфиги /etc/nginx/conf.d
https://abf.io/build_lists/4751174
https://abf.io/build_lists/4751176
https://abf.io/build_lists/4751178
https://abf.io/build_lists/4751180
https://abf.io/build_lists/4751181

angie.conf/nginx.conf у нас просто апстримные, подумал, что лучше не усложнять схему конфигов и просто убрал неиспользуемые файлы. Философия конфига здесь такова, что его проще самому с нуля написать руками.
Т.к. они %config(noreplace), если у кого-то они отредактированы и используются, при обновлении не должно быть регрессий.
Comment 2 Dmitry Postnikov 2023-10-02 13:48:50 MSK
***************************
The update sent to testings
Comment 3 Vladimir Potapov 2023-10-10 13:13:22 MSK
angie-1.3.0-1
https://abf.io/build_lists/4751172
https://abf.io/build_lists/4751173
https://abf.io/build_lists/4751175
https://abf.io/build_lists/4751177
https://abf.io/build_lists/4751179

nginx-1.24.0-2
https://abf.io/build_lists/4751174
https://abf.io/build_lists/4751176
https://abf.io/build_lists/4751178
https://abf.io/build_lists/4751180
https://abf.io/build_lists/4751181
********************* Advisory ****************************
 обновлено до 1.3.0
- включены все возможные модули, например, добавлен модуль ssl_preread
- убраны конфиги /etc/angie/conf.d
- включены все возможные модули, например, добавлен модуль ssl_preread
- убраны конфиги /etc/nginx/conf.d
************************************************************
QA Verified