1.10.33.0 -> 1.10.33.1

config

@@ -29,8 +29,8 @@ if [ "$mod_pagespeed_dir" = "unset" ] ; then
     echo "   You need to separately download the pagespeed library:"
     echo ""
     echo "     $ cd /path/to/ngx_pagespeed"
-    echo "     $ wget https://dl.google.com/dl/page-speed/psol/1.9.32.1.tar.gz"
-    echo "     $ tar -xzvf 1.9.32.1.tar.gz # expands to psol/"
+    echo "     $ wget https://dl.google.com/dl/page-speed/psol/1.10.33.1.tar.gz"
+    echo "     $ tar -xzvf 1.10.33.1.tar.gz # expands to psol/"
     echo ""
     echo "   Or see the installation instructions:"
     echo "     https://github.com/pagespeed/ngx_pagespeed#how-to-build"

src/ngx_pagespeed.cc

@@ -986,7 +986,6 @@ void* ps_create_main_conf(ngx_conf_t* cf) {
       "" /* hostname, not used */,
       -1 /* port, not used */);
   active_driver_factory = cfg_m->driver_factory;
-  active_driver_factory->LoggingInit(ngx_cycle->log, false);
   cfg_m->driver_factory->Init();
   ps_set_conf_cleanup_handler(cf, ps_cleanup_main_conf, cfg_m);
   return cfg_m;
@@ -2479,8 +2478,8 @@ ngx_int_t send_out_headers_and_body(
 
   rc = ngx_http_send_header(r);
 
-  if (rc == NGX_ERROR || rc > NGX_OK || r->header_only) {
-    return rc;
+  if (rc != NGX_OK) {
+    return NGX_ERROR;
   }
 
   // Send the body.
@@ -3052,7 +3051,7 @@ ngx_int_t ps_init_module(ngx_cycle_t* cycle) {
       return NGX_ERROR;
     }
 
-    cfg_m->driver_factory->LoggingInit(cycle->log, true);
+    cfg_m->driver_factory->LoggingInit(cycle->log);
     cfg_m->driver_factory->RootInit();
   } else {
     delete cfg_m->driver_factory;
@@ -3084,7 +3083,7 @@ ngx_int_t ps_init_child_process(ngx_cycle_t* cycle) {
 
   // ChildInit() will initialise all ServerContexts, which we need to
   // create ProxyFetchFactories below
-  cfg_m->driver_factory->LoggingInit(cycle->log, true);
+  cfg_m->driver_factory->LoggingInit(cycle->log);
   cfg_m->driver_factory->ChildInit();
 
   ngx_http_core_main_conf_t* cmcf = static_cast<ngx_http_core_main_conf_t*>(

src/ngx_rewrite_driver_factory.cc

@@ -208,11 +208,10 @@ void NgxRewriteDriverFactory::StartThreads() {
   threads_started_ = true;
 }
 
-void NgxRewriteDriverFactory::LoggingInit(
-    ngx_log_t* log, bool may_install_crash_handler) {
+void NgxRewriteDriverFactory::LoggingInit(ngx_log_t* log) {
   log_ = log;
   net_instaweb::log_message_handler::Install(log);
-  if (may_install_crash_handler && install_crash_handler()) {
+  if (install_crash_handler()) {
     NgxMessageHandler::InstallCrashHandler(log);
   }
   ngx_message_handler_->set_log(log);

src/ngx_rewrite_driver_factory.h

@@ -115,7 +115,7 @@ class NgxRewriteDriverFactory : public SystemRewriteDriverFactory {
     return process_script_variables_;
   }
 
-  void LoggingInit(ngx_log_t* log, bool may_install_crash_handler);
+  void LoggingInit(ngx_log_t* log);
 
   virtual void ShutDownMessageHandlers();
 

test/pagespeed_test.conf.template

@@ -1748,25 +1748,6 @@ http {
           https://www.gstatic.com/psa/static;
     }
 
-    location /mod_pagespeed_test/strip_subresource_hints/default/ {
-      pagespeed DisableFilters add_instrumentation;
-      pagespeed RewriteLevel CoreFilters;
-      pagespeed DisAllow *dontrewriteme*;
-    }
-    location /mod_pagespeed_test/strip_subresource_hints/default_passthrough/ {
-      pagespeed DisableFilters add_instrumentation;
-      pagespeed RewriteLevel PassThrough;
-    }
-    location /mod_pagespeed_test/strip_subresource_hints/preserve_on/ {
-      pagespeed DisableFilters add_instrumentation;
-      pagespeed PreserveSubresourceHints on;
-      pagespeed RewriteLevel CoreFilters;
-    }
-    location /mod_pagespeed_test/strip_subresource_hints/preserve_off/ {
-      pagespeed DisableFilters add_instrumentation;
-      pagespeed PreserveSubresourceHints off;
-      pagespeed RewriteLevel CoreFilters;
-    }
     # $host implicitly tests script variable support. I'd love to test it more
     # directly, but so far this is the best I've come up with and duplicating
     # the test doesn't seem to make sense.