bookpauk / inpx-web

Веб-сервер для поиска по inpx-коллекции
https://lib.omnireader.ru
Creative Commons Zero v1.0 Universal
79 stars 22 forks source link

Адаптация к ОС #28

Closed partizand closed 5 months ago

partizand commented 7 months ago

Предлагаю разделить каталог настроек и каталог данных. Не обязательно делать так как в этом PR, но общий посыл такой. В каждой ОС есть свои стандартные места для расположения данных, логов и настроек. Предлагаю использовать стандартные подходы. Временный каталог должна выдавать сама операционка.

Непонимаю, что в плане защиты дает --unsafe-filter.

Jipok commented 6 months ago

Убрано ежеминутное отображение в лог состояния сервера

Вот эт хорошо, а можно ещё добавить в логи запрос и сколько времен он занял?

partizand commented 6 months ago

Это к автору. Или поставьте реверс прокси.

bookpauk commented 5 months ago

Предлагаю разделить каталог настроек и каталог данных.

Я не сторонник излишнего усложнения, на мой взгляд достаточно app-dir, остальное можно решить симлинками при необходимости

Вот эт хорошо, а можно ещё добавить в логи запрос и сколько времен он занял?

Убрал ежеминутное логирование, для запросов добавил параметр logQueries в конфиг Будет доступно в 1.5.6

partizand commented 5 months ago

Это симлинки усложнение. Не знаю как вы запускаете свой сервис. Но если писать unit для systemd, то нужно разделять, конфиги в etc, данные в var. И права на каталоги нужны. Вы же не под root будете сетевой сервис запускать. Данные могут быть очищены, конфиги не должны при этом страдать. Очень печально, что вы игнорируете базовые подходы.

Jipok commented 5 months ago

Ну честно говоря вы с systemd сами себе проблемы создаёте. Я с tmuxp под юзером всё запускаю и бед не знаю