search

webserver-llc / angie

Angie - drop-in replacement for Nginx
https://en.angie.software
BSD 2-Clause "Simplified" License
1.24k stars 65 forks source link

Добавлен вывод версии nginx в angie -V #71

Closed AdvTechnoKing closed 8 months ago

AdvTechnoKing commented 9 months ago

Есть популярная штука, certbot, хотя вы и так знаете, если судить по домену, так вот, certbot нужен вывод версии nginx.

Собственно про чё я, есть одна противная строка, которая не даёт закостылить certbot под работу с angie, он вызывает angie -V и после в ответе ищет nginx version:, ну соответственно сыпется в ошибку, потому что там вместо неё вписана Angie version:.

Ну я так прикинул, если это форк и там уже есть инфа о версии nginx, то почему бы ему её не вывести. Потестил у себя, работает, ну и вот, предлагаю теперь пулл реквест в массы.

Да и вроде по BSD-2 там должна быть инфа о nginx.

Сам костыль:

ln -s /etc/angie/angie.conf /etc/angie/nginx.conf
certbot --nginx-server-root=/etc/angie --nginx-ctl=angie

Пруф работы:

image

dmitry-j-mikhin commented 9 months ago

В целом разрабы обещали сделать встроенную поддержку ACME без всяких сторонних тулзов. Возможно в итоге тогда этот патч и не понадобиться - https://t.me/angie_support/253 image

AdvTechnoKing commented 9 months ago

Да тут даже не как патч можно рассматривать, а как полезную информацию, ибо кому то может важно видеть, какая сейчас версия nginx под капотом. Но чисто для меня это простенький костыль в certbot.

dmitry-j-mikhin commented 9 months ago

В долгосроке кодовая беза проектов будет все дальше разъезжаться и не факт что базовая версия nginx в итоге что-то полезное будет показывать. Тут кажется что если хочется подружить certbot c Angie, то правильней сделать патч в сам certbot. Но это мое ИМХО и может не отражать виденье разрабов этого проекта )

AdvTechnoKing commented 9 months ago

Изначально хотел залезть туда, но копировать всю папку nginx, что бы в ней потом заменить все слова на angie, было бы крайне тупо. Поэтому посмотрел что он умеет и понял, что самый простой вариант, это добавить 1 строку в angie.

VBart commented 9 months ago

Неплохая идея. Потенциально может оказаться ещё где-то полезной. Когда кодовая база совсем разойдется - уберем.

Сейчас закончим с мерджом свежей версии nginx, а затем примем предложенное изменение. Спасибо.

P.S. Имеет смысл ещё добавить соответствующий feature в docs/xml/angie/changes.xml.