angristan / nginx-autoinstall

Compile NGINX from source with custom modules and patches on Debian and Ubuntu
GNU General Public License v3.0
636 stars 237 forks source link

nginx-1.23.1 mainline version has been released & Bugfix ngx_headers_… etc #243

Open MaximeMichaud opened 2 years ago

MaximeMichaud commented 2 years ago

Changes with nginx 1.23.0 21 Jun 2022

*) Change in internal API: now header lines are represented as linked
   lists.

*) Change: now nginx combines arbitrary header lines with identical
   names when sending to FastCGI, SCGI, and uwsgi backends, in the
   $r->header_in() method of the ngx_http_perl_module, and during lookup
   of the "$http_...", "$sent_http_...", "$sent_trailer_...",
   "$upstream_http_...", and "$upstream_trailer_..." variables.

*) Bugfix: if there were multiple "Vary" header lines in the backend
   response, nginx only used the last of them when caching.

*) Bugfix: if there were multiple "WWW-Authenticate" header lines in the
   backend response and errors with code 401 were intercepted or the
   "auth_request" directive was used, nginx only sent the first of the
   header lines to the client.

*) Change: the logging level of the "application data after close
   notify" SSL errors has been lowered from "crit" to "info".

*) Bugfix: connections might hang if nginx was built on Linux 2.6.17 or
   newer, but was used on systems without EPOLLRDHUP support, notably
   with epoll emulation layers; the bug had appeared in 1.17.5.
   Thanks to Marcus Ball.

*) Bugfix: nginx did not cache the response if the "Expires" response
   header line disabled caching, but following "Cache-Control" header
   line enabled caching.

Bugfix ngx_headers_more :

https://github.com/openresty/headers-more-nginx-module/commit/91eb0db9ef9ac05c4c514b8f1fdf8a8170cc354e

I edited the file in my browser, I know that HEADERMOD_VER is now unused. I was unsure, If I should have apply the bugfix only, but since the last release of this module is 2017, I consider that this is fine with the current context.

Compilation is now working properly on my side. nginx -V nginx version: nginx/1.22.0 built by gcc 10.2.1 20210110 (Debian 10.2.1-6) built with OpenSSL 1.1.1n 15 Mar 2022 TLS SNI support enabled configure arguments: --prefix=/etc/nginx --sbin-path=/usr/sbin/nginx --conf-path=/etc/nginx/nginx.conf --error-log-path=/var/log/nginx/error.log --http-log-path=/var/log/nginx/access.log --pid-path=/var/run/nginx.pid --lock-path=/var/run/nginx.lock --http-client-body-temp-path=/var/cache/nginx/client_temp --http-proxy-temp-path=/var/cache/nginx/proxy_temp --http-fastcgi-temp-path=/var/cache/nginx/fastcgi_temp --user=nginx --group=nginx --with-cc-opt=-Wno-deprecated-declarations --with-cc-opt=-Wno-ignored-qualifiers --with-threads --with-file-aio --with-http_ssl_module --with-http_v2_module --with-http_mp4_module --with-http_auth_request_module --with-http_slice_module --with-http_stub_status_module --with-http_realip_module --with-http_sub_module --add-module=/usr/local/src/nginx/modules/ngx_brotli --add-module=/usr/local/src/nginx/modules/headers-more-nginx-module

However, before this was not the case unfortunately.

MaximeMichaud commented 2 years ago

ModPageSpeed seems to be not liking Mainline 1.23.0 Fix : https://github.com/apache/incubator-pagespeed-ngx/pull/1751

Kelileis commented 2 years ago

Hello! When is the 1.23 expected to be fixed?

MaximeMichaud commented 2 years ago

Hello! When is the 1.23 expected to be fixed?

Everything is absolutely fine for me.

nginx -V nginx version: nginx/1.23.1 built by gcc 10.2.1 20210110 (Debian 10.2.1-6) built with OpenSSL 1.1.1n 15 Mar 2022 TLS SNI support enabled configure arguments: --prefix=/etc/nginx --sbin-path=/usr/sbin/nginx --conf-path=/etc/nginx/nginx.conf --error-log-path=/var/log/nginx/error.log --http-log-path=/var/log/nginx/access.log --pid-path=/var/run/nginx.pid --lock-path=/var/run/nginx.lock --http-client-body-temp-path=/var/cache/nginx/client_temp --http-proxy-temp-path=/var/cache/nginx/proxy_temp --http-fastcgi-temp-path=/var/cache/nginx/fastcgi_temp --user=nginx --group=nginx --with-cc-opt=-Wno-deprecated-declarations --with-cc-opt=-Wno-ignored-qualifiers --with-threads --with-file-aio --with-http_ssl_module --with-http_v2_module --with-http_mp4_module --with-http_auth_request_module --with-http_slice_module --with-http_stub_status_module --with-http_realip_module --with-http_sub_module --add-module=/usr/local/src/nginx/modules/incubator-pagespeed-ngx-1.13.35.2-stable --add-module=/usr/local/src/nginx/modules/ngx_brotli --add-module=/usr/local/src/nginx/modules/headers-more-nginx-module

MaximeMichaud commented 2 years ago

IDK what CI is doing. ../../nginx-autoinstall.sh: line 760: systemctl: command not found (Compile NGINX / compile (debian:11, MAINLINE, FULL, SYSTEM) (pull_request) Failing after 9m — compile (debian:11, MAINLINE, FULL, SYSTEM) @angristan

muuvmuuv commented 1 year ago

Does this include the PCRE2 Problem with the LUA module, as found here #248 ?

MaximeMichaud commented 1 year ago

Does this include the PCRE2 Problem with the LUA module, as found here #248 ?

No, unfortunately the only time I used LUA with the script (December 2021) it was not working properly.

BrycensRanch commented 1 year ago

hey looking to use this on my server but I need a nginx version bump again

GwynethLlewelyn commented 8 months ago

hey looking to use this on my server but I need a nginx version bump again

Couldn't have said better myself 🤣