This is an automated email from the ASF dual-hosted git repository.
rrm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new fe35df1 Removes TODO (#7027)
fe35df1 is described below
commit fe35df1b5447b0b158877a64d610592b853c2098
Author: Randall Meyer <[email protected]>
AuthorDate: Wed Jul 22 11:18:00 2020 -0700
Removes TODO (#7027)
This has been in the codebase for 10 years and I think the value we've
been using is the value we're sticking with.
---
proxy/http/HttpTransactHeaders.cc | 2 --
1 file changed, 2 deletions(-)
diff --git a/proxy/http/HttpTransactHeaders.cc
b/proxy/http/HttpTransactHeaders.cc
index bd3fc94..52fc136 100644
--- a/proxy/http/HttpTransactHeaders.cc
+++ b/proxy/http/HttpTransactHeaders.cc
@@ -549,8 +549,6 @@ HttpTransactHeaders::generate_and_set_squid_codes(HTTPHdr
*header, char *via_str
// Errors may override the other codes, so check the via string error codes
last
switch (via_string[VIA_ERROR_TYPE]) {
case VIA_ERROR_AUTHORIZATION:
- // TODO decide which one?
- // log_code = SQUID_LOG_TCP_DENIED;
log_code = SQUID_LOG_ERR_PROXY_DENIED;
break;
case VIA_ERROR_CONNECTION:
