Replace CHECK with return failure and log.

Fixes #888 along with https://code.google.com/p/modpagespeed/source/detail?r=4533
apache · Feb 23, 2015 · 83feaaf · 83feaaf
1 parent f4f5ac5
commit 83feaaf
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/ngx_pagespeed.cc b/src/ngx_pagespeed.cc
@@ -1661,7 +1661,10 @@ ngx_int_t ps_resource_handler(ngx_http_request_t* r,
   GoogleString url_string = ps_determine_url(r);
   GoogleUrl url(url_string);
 
-  CHECK(url.IsWebValid());
+  if (!url.IsWebValid()) {
+    ngx_log_error(NGX_LOG_ERR, r->connection->log, 0, "invalid url");
+    return NGX_DECLINED;
+  }
 
   scoped_ptr<RequestHeaders> request_headers(new RequestHeaders);
   scoped_ptr<ResponseHeaders> response_headers(new ResponseHeaders);