trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bri...@apache.org
Subject trafficserver git commit: TS-3723: Failed remap should log as ERR_INVALID_URL
Date Mon, 29 Jun 2015 07:06:58 GMT
Repository: trafficserver
Updated Branches:
  refs/heads/master a3e83c11d -> 8dbe5601d


TS-3723: Failed remap should log as ERR_INVALID_URL


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/8dbe5601
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/8dbe5601
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/8dbe5601

Branch: refs/heads/master
Commit: 8dbe5601d668e183d69a63f041fe10dacd4e3978
Parents: a3e83c1
Author: Brian Geffon <briang@apache.org>
Authored: Mon Jun 29 00:06:48 2015 -0700
Committer: Brian Geffon <briang@apache.org>
Committed: Mon Jun 29 00:06:48 2015 -0700

----------------------------------------------------------------------
 proxy/http/HttpTransact.cc | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/8dbe5601/proxy/http/HttpTransact.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index c706c20..59ee0fc 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -845,8 +845,10 @@ HttpTransact::EndRemapRequest(State *s)
         // socket when there is no host. Need to handle DNS failure elsewhere.
       } else if (host == NULL) { /* no host */
         build_error_response(s, HTTP_STATUS_BAD_REQUEST, "Host Header Required", "request#no_host",
NULL);
+        s->squid_codes.log_code = SQUID_LOG_ERR_INVALID_URL;
       } else {
         build_error_response(s, HTTP_STATUS_NOT_FOUND, "Not Found on Accelerator", "urlrouting#no_mapping",
NULL);
+        s->squid_codes.log_code = SQUID_LOG_ERR_INVALID_URL;
       }
       s->reverse_proxy = false;
       goto done;
@@ -859,6 +861,7 @@ HttpTransact::EndRemapRequest(State *s)
 
       SET_VIA_STRING(VIA_DETAIL_TUNNEL, VIA_DETAIL_TUNNEL_NO_FORWARD);
       build_error_response(s, HTTP_STATUS_NOT_FOUND, "Not Found", "urlrouting#no_mapping",
NULL);
+      s->squid_codes.log_code = SQUID_LOG_ERR_INVALID_URL;
 
       s->reverse_proxy = false;
       goto done;


Mime
View raw message