[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox
eilandert commented on issue #1749: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1168029872 ok, no sleep! fixed some things. Packages here: http://edge.deb.myguard.nl:/pool/main/n/nginx/

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox
eilandert commented on issue #1749: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1167881999 Thanks @oschaaf , it compiles. However, my build fails thanks to openresty unpatched modules (with the same problem), so I have no packages at all to test.

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox
danidorado commented on issue #1749: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1167132866 @oschaaf. I'm happy to give it a try on my servers, let me know how i can proceed -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-pagespeed-ngx] oschaaf commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox
oschaaf commented on issue #1749: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1167015184 @eilandert & @Lofesa -- https://github.com/apache/incubator-pagespeed-ngx/pull/1750 has an (untested) draft that aims to resolve this. It's still a draft, because

[GitHub] [incubator-pagespeed-ngx] oschaaf opened a new pull request, #1750: [DRAFT] Fixes for NGINX nginx 1.23.0

2022-06-27 Thread GitBox
oschaaf opened a new pull request, #1750: URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1750 As of nginx 1.23 the cache control headers are re-implemented as al linked list. This reworks our code that manipulates this header to work with that. Still a draft: this