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

Merge pull request #892 from pagespeed/crowell-fix-888

Browse Source 
Replace CHECK with return failure and log.
This commit is contained in:
Jeffrey Crowell
2015-01-30 10:28:04 -05:00
parent 001cd2dc4c 81029dafa6
commit 737e9396f5
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
@@ -1661,7 +1661,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);