artox/incubator-pagespeed-ngx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Replace CHECK with return failure and log.

Browse Source 
Fixes #888 along with
https://code.google.com/p/modpagespeed/source/detail?r=4533
This commit is contained in:
Jeffrey Crowell
2015-01-30 10:23:00 -05:00
parent 90ac91fe9e
commit 9da85bb9d5
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ngx_pagespeed.cc
+4 -1
View File
@@ -1816,7 +1816,10 @@ ngx_int_t ps_resource_handler(ngx_http_request_t* r,
GoogleString url_string = ps_determine_url(r); GoogleString url_string = ps_determine_url(r);
GoogleUrl url(url_string); 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<RequestHeaders> request_headers(new RequestHeaders);
scoped_ptr<ResponseHeaders> response_headers(new ResponseHeaders); scoped_ptr<ResponseHeaders> response_headers(new ResponseHeaders);